WordPress.org

Ready to get started?Download WordPress

Forums

All in One SEO Pack
[resolved] Symbol in post title doesn't show up in title (13 posts)

  1. jamescosta
    Member
    Posted 7 months ago #

    Hey folks!

    When I have a title like "$14,000", All in One SEO removes the "$14" (therefore leave it as ,000") - the only solution is putting a space between the $ and 14.

    Any ideas?

    http://wordpress.org/plugins/all-in-one-seo-pack/

  2. wpsmort
    Member
    Posted 7 months ago #

    Hi jamescosta,

    Do you have a URL of a page where we can see this happening?

  3. jamescosta
    Member
    Posted 7 months ago #

    Hey wpsmort!

    We activated it for a minute to just get you these screenshots and stuff, so hopefully they're okay - let me know if you need any more!

    THANK YOU for your help! :-)

  4. Peter Baylies
    Member
    Plugin Author

    Posted 7 months ago #

    jamescosta,

    Thank you for the report; this was a bug, I've fixed it. You can update by deleting and reinstalling the plugin, or by changing line 1248 of aioseop_class.php to this:

    return preg_replace( '/<title>(.*?)<\/title>/is', '<title>' . preg_replace('/(\$|\\\\)(?=\d)/', '\\\\\1', esc_textarea( strip_tags( $title ) ) ) . '</title>', $content );

  5. jamescosta
    Member
    Posted 7 months ago #

    Peter!

    Thank you so much for this - seriously!

    Have an awesome day!

  6. acroplex
    Member
    Posted 7 months ago #

    This fixed it, but it was line 1548 for me.

  7. meanstreak
    Member
    Posted 7 months ago #

    Worked for me too! Thanks!

  8. acroplex
    Member
    Posted 7 months ago #

    Update: Unfortunately, the problem persists as it now converts double quotes into entities. Eg "test" becomes:

    &αmp;#8220;test&αmp;#8221;

  9. Peter Baylies
    Member
    Plugin Author

    Posted 7 months ago #

    acroplex,

    Is it getting converted twice? When I use double quotes, it shows up as &quot;Test&quot; and renders as "Test". You could try this instead:

    return preg_replace( '/<title>(.*?)<\/title>/is', '<title>' . preg_replace('/(\$|\\\\)(?=\d)/', '\\\\\1', strip_tags( $title ) ) . '</title>', $content );

  10. acroplex
    Member
    Posted 7 months ago #

    hi Peter,

    Both single quotes and double quotes appear as entities in the title.

    Would you please roll out an updated point release?

  11. Peter Baylies
    Member
    Plugin Author

    Posted 7 months ago #

    acroplex,

    Did you try the above code, and did that fix your issue? Also, what browser are you using?

  12. Peter Baylies
    Member
    Plugin Author

    Posted 7 months ago #

    acroplex,

    This was a bug, I've fixed it now; you can update by deleting and reinstalling the plugin.

  13. acroplex
    Member
    Posted 7 months ago #

    Many thanks, Peter, I can confirm the glitch has been fixed. I deleted the plugin and reinstalled it; good to see it retained my settings as well :-)

    Kind regards,
    Theo

Reply

You must log in to post.

About this Plugin

About this Topic