As designed They thought me to use {xen:link} :)

[Vincent]

Hello,

Some of the tips I received when creating add-ons was that I'd best use {xen:link} for linking.
Today I found 2 links in the interface that don't use that

The first one is in PAGE_CONTAINER

<a href="admin.php">{xen:phrase reopen_via_admin_control_panel}</a>

to (I added this suggestion to it, so don't forget to mark as implented )

<a href="{xen:adminlink 'options/list/boardActive'}">{xen:phrase reopen_via_admin_control_panel}</a>

The second one is in moderator_bar

<a href="admin.php" class="acp adminLink"><span class="itemLabel">{xen:phrase admin_control_panel}</span></a>

to

<a href="{xen:adminlink}" class="acp adminLink"><span class="itemLabel">{xen:phrase admin_control_panel}</span></a>

 

[Kier]

Fixed both.

 

[Kier]

I've just reversed out these changes, as Mike pointed out to me that using {xen:adminlink ...} in front-end templates will cost one SQL query. Therefore, leave it as it is.

 

[Vincent]

I've just reversed out these changes, as Mike pointed out to me that using {xen:adminlink ...} in front-end templates will cost one SQL query. Therefore, leave it as it is.

Oh right
Well, noone is perfect

Is the boardActive suggestion still implented?

 

[Kier]

Yes it is