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

Log In

Lost password?
Advanced Search:

— Forum Scope —



— Match —



— Forum Options —




Wildcard usage:
*  matches any number of characters    %  matches exactly one character

Minimum search word length is 4 characters - maximum search word length is 84 characters

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?

Forum Timezone: America/New_York

Most Users Ever Online: 205

Currently Online:
34 Guest(s)

Currently Browsing this Page:
1 Guest(s)

Top Posters:

romiojuliat: 102

yaoming987: 50

jintuy: 50

miki1986: 33

newshow: 28

karen1234: 27

Member Stats:

Guest Posters: 0

Members: 6356

Moderators: 3

Admins: 4

Forum Stats:

Groups: 5

Forums: 23

Topics: 1546

Posts: 3450

Newest Members: carofern@cisco.com, pronto, jessie, pbaylies, ninaeast, freda

Moderators: davecashmore (2), aaron (5), tempadmin (0)

Administrators: WordPress Professional (442), hallsofmontezuma (202), onefinejay (10), stevemortiboy (4)