Title attribute setting issue (and a fix, believe it or not) All in One SEO Pack Troubleshooting Forum

The support forums have a new home! Please visit All in One SEO Pack's new residence at Semper Plugins to purchase a support plan.

Please consider registering
guest

Lost password?

Home

Forum

All in One SEO Pack Troubleshooting

Title attribute setting issue (and a fix, believe it or not)

Topic RSS

Title attribute setting issue (and a fix, believe it or not)

September 6, 2009
12:33 am

stevemagruder

New Member

Forum Posts: 4

Member Since:
September 6, 2009

Offline

First of all, excellent plugin! Love it!

I did run into a minor issue, though, with how it sets the title attribute in the menu for a page if I have left the AIOSEOP title attribute field blank for the page. The aioseop_filter_callback function was actually setting the anchor in the menu list item to have no title at all, when I expected that it would just set it to what was there already.

So I changed the second $filtered equation as follows:

$filtered = '<li class="page_item page-item-'.$postID.$matches[2].'"><a href="'.$matches[3].'" title="'.$matches[4].'">'.$menulabel.'</a>';

This fixed the issue.

Steve Magruder, Freelance Web Programmer

September 15, 2009
11:14 am

stevemagruder

New Member

Forum Posts: 4

Member Since:
September 6, 2009

Offline

Could this fix be added to the official plugin? Surely, nobody intends to override a title with a blank.

Steve Magruder, Freelance Web Programmer

September 15, 2009
3:33 pm

WordPress Professional

Admin

Forum Posts: 442

Member Since:
October 14, 2008

Offline

You must be a subscriber to view administrator posts. Click here to purchase a support subscription.

For what shall it profit a man if he shall gain the whole world and lose his own soul

December 17, 2009
6:43 pm

stevemagruder

New Member

Forum Posts: 4

Member Since:
September 6, 2009

Offline

I don't mean to be impatient, but I have been holding up upgrading this plugin on multiple (and growing) WordPress sites due to this fix not being in there yet. I don't have the time to patch the code for all the sites every time there is an upgrade.

I hope that the fix is included soon.

Steve Magruder, Freelance Web Programmer

December 18, 2009
4:29 am

hallsofmontezuma

Admin

Forum Posts: 202

Member Since:
June 8, 2009

Offline

You must be a subscriber to view administrator posts. Click here to purchase a support subscription.

For what shall it profit a man, if he shall gain the whole world, and lose his own soul?

December 18, 2009
11:34 am

stevemagruder

New Member

Forum Posts: 4

Member Since:
September 6, 2009

Offline

Great! Thank you!

Steve Magruder, Freelance Web Programmer

June 17, 2010
9:55 am

aquanauts

New Member

Forum Posts: 1

Member Since:
June 17, 2010

Offline

I've run into this same problem and am using 1.6.11.

I've compared the code in 1.6.11 with the code fix stevenmacgruder supplied and it does not match. Was your fix implemented?

Having said that, I subbed in Steven's code and my problem is still there. I don't want to leave the AIOSEOP rewrite title field blank. But I don't want it to destroy my nav bar, which worked properly with Page List Plus plugin.

Anyway to resolve this story the TITLE in the browser bar gets rewritten and NOT the WP_List_Pages entry I use in my nav bar?

All RSS

Forum Timezone: America/New_York

Most Users Ever Online: 205

Currently Online:
34 Guest(s)

Currently Browsing this Page:
1 Guest(s)