Sitemap XML casting undefined entity

  • Resolved Tobias Ehlert

    (@tobiasehlert)


    10 years ago

    Hi,

    I run several blogs and all of them use the jetpack sitemap plugin, but there seems to be some html encoding issue with the text “Change frequency” when its translated (in my example to swedish) which in this case would be: <th>Ändra frekvens</th>

    The error says undefined entity and its because the text does contain & in the beginning and that doesn’t seem to be correct. Its on line 93 in my case, if that is of any use.

    / Tobias

    https://ww.wp.xz.cn/plugins/jetpack/

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Jeremy Herve

    (@jeherve)

    Jetpack Mechanic 🚀

    10 years ago

    That’s indeed a known issue, and something we’d like to fix in the next Jetpack release.

    Would you mind testing the patch we’ve worked on, and let me know if that works for you?
    https://github.com/Automattic/jetpack/pull/4056/files

    Let me know how it goes!

    Thread Starter Tobias Ehlert

    (@tobiasehlert)

    10 years ago

    Hi Jeremy,

    Looks to me that there is an issue on line

    It looks to me that the issue is on line 93 at the Change Frequency in Swedish.
    &Auml;ndra frekvens should be Ändra frekvens instead I guess.

    This needs to be fixed somehow
    line 93 <th>’ . esc_html__( ‘Change Frequency’, ‘jetpack’ ) . ‘</th>

    Also posted a comment on the pull request 🙂

    Regards,
    Tobias

    Thread Starter Tobias Ehlert

    (@tobiasehlert)

    10 years ago

    Hi Jeremy,

    I updated the pull request and Alex Kirk has adjusted the code that all strings are encoded correctly. I update the ticket to resolved, since a bugfix is on the way 🙂

    Regards,
    Tobias

    Plugin Author Jeremy Herve

    (@jeherve)

    Jetpack Mechanic 🚀

    10 years ago

    Perfect, thanks for giving the patch a try, and for your comment!

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘Sitemap XML casting undefined entity’ is closed to new replies.

Tags