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.

 
You must be logged in to post
Search Forums:


 






Minimum search word length is 4 characters – Maximum search word length is 84 characters
Wildcard Usage:
*  matches any number of characters    %  matches exactly one character

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

Post
New Member

stevemagruder

posts 4

12:33 am September 6, 2009

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

New Member

stevemagruder

posts 4

11:14 am September 15, 2009

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

Steve Magruder, Freelance Web Programmer

Admin

WordPress Professional

Raleigh, NC

posts 442

3:33 pm September 15, 2009

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

New Member

stevemagruder

posts 4

6:43 pm December 17, 2009

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

Admin

hallsofmontezuma

Cary, NC

posts 202

4:29 am December 18, 2009

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?

New Member

stevemagruder

posts 4

11:34 am December 18, 2009

Great!  Thank you!

Steve Magruder, Freelance Web Programmer

New Member

aquanauts

posts 1

9:55 am June 17, 2010

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?


About the WordPress Developer Forum

Forum Timezone: America/New_York

Most Users Ever Online: 205

Currently Online:
13 Guests

Currently Browsing this Topic:
1 Guest

Forum Stats:

Groups: 5
Forums: 23
Topics: 1543
Posts: 3447

Membership:

There are 6357 Members

There are 4 Admins
There are 3 Moderators