Thread

I need to use some conditional PHP to add a class of ‘current’ to links in a menu ive made when your on their page. Currently im doing this with {if last_segment}, which looks for a url value and adds a class if the right one exists. This works fine but it will break if the page url changes in the future. Is there a better future proof way of doing things? Thanks

Hi James,

Could you possibly give a little more information as to exactly what you’re doing here. I understand what you’re trying to do but don’t quite follow why when a url_title changes that it would break?

If I’m following correctly then I take it that you have a single entry page :

http://www.example.com/template-group/template-name/entry-title-here

and on that page you have all the usual output of that entry and then in your menu you want to show that page with a different class, yes?

So you would have something like this in the menu :

{exp:channel:entries
    channel="default_site"
    dynamic="no"
    disable="member_data|trackbacks|categories|category_fields|pagination"}

<a href="http://{url_title_path=%22template-group/template-name%22}" title="{title}" class="current">{title}</a>

{/exp:channel:entries}

This would then show the entry that you’re currently viewing differently to the rest. Is that what you’re doing here?

If so then if the url_title of the single entry changes then this method should still work.

Please let me know if I’m wrong in what you’re trying to achieve here though.

Thanks,

Mark

Agggh the forums just stripped out the conditional I had placed in the code.

Give me a minute and I’ll upload it somewhere instead.

Cheers,

Mark

Okay I’ve uploaded the code here - http://d.pr/Dokn.

Sorry about that but seems that the forum didn’t like the conditional within the link like that. Probably a protection thing.

The conditional on it’s own would be like this :

{if last_segment == url_title} class="current"{/if}

Thanks,

Mark

Ah, I hadn’t thought about using url_title, i just used a static value. I think I can figure this out. Thanks

Hi James,

Just wanted to see if you’re good to go on this one now?

Is it okay for me to close out the thread?

Thanks,

Mark

Im going to revisit this later but yes, I think I can do it with the method you suggested. Thanks

Hi James,

Excellent. I’m going to go ahead and close this thread then but if anything else crops up then please don’t hesitate to post again as needed.

Cheers,

Mark