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 11 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 10 months ago #

    Hi jamescosta,

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

  3. jamescosta
    Member
    Posted 10 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 10 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 10 months ago #

    Peter!

    Thank you so much for this - seriously!

    Have an awesome day!

  6. acroplex
    Member
    Posted 10 months ago #

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

  7. meanstreak
    Member
    Posted 10 months ago #

    Worked for me too! Thanks!

  8. acroplex
    Member
    Posted 10 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 10 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 10 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 10 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 10 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 10 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