Make WordPress Core

Opened 17 years ago

Closed 14 years ago

Last modified 14 years ago

#10498 closed defect (bug) (fixed)

RSS widget filters some characters

Reported by: pavelevap Owned by: rmccue
Priority: normal Milestone: 3.4
Component: Widgets Version: 2.8.2
Severity: normal Keywords: simplepie has-patch dev-feedback
Cc: Focuses:

Description

Hello, when inserting this RSS (​http://servis.idnes.cz/rss.asp?c=mfd_k) into RSS widget then some characters are removed, for example č, í, ý, etc.

Attachments (1)

10498.diff​ (871 bytes ) - added by solarissmoke 16 years ago.

Download all attachments as: .zip

Change History (8)

#1 @azaozz
17 years ago

  • Milestone 2.9 → 3.0

No patch, moving to 3.0

#2 @dd32
17 years ago

  • Keywords simplepie needs-patch added
  • Milestone 3.0 → Future Release

The root cause of this is SimplePie, It seems to be striping the characters instead of converting them to html entities.

Moving to future release due to no patch.

#3 @solarissmoke
16 years ago

  • Keywords has-patch dev-feedback added; needs-patch removed

The problem is that SimplePie tries to use mb_convert_encoding() without checking whether the encoding is supported. In this case, Windows-1250 isn't supported, so you lose all the special characters. You can check what is supported by calling mb_list_encodings().

Someone has reported the issue here: ​https://github.com/rmccue/SimplePie/issuesearch?state=open&q=encoding#issue/121 (second bit). But upstream development doesn't look very active.

Last edited 16 years ago by solarissmoke (previous) (diff)

@solarissmoke
16 years ago

#4 @rmccue
16 years ago

  • Owner changed from azaozz to rmccue
  • Status new → assigned

I'm the SimplePie maintainer, so I'll look into this and report back.

#5 @rmccue
16 years ago

Pushed up to ​https://github.com/simplepie/simplepie/commit/cce56bb6809de9301909204c40c8a4f13ec322fe

Should be find to merge into WordPress' hacked up version. ;)

#6 @pavelevap
14 years ago

  • Resolution → fixed
  • Status assigned → closed

SimplePie 1.2.1 in WordPress 3.4 works well for this issue!

Tested on some websites - WordPress 3.3.2 shows this problem, but 3.4 is fine.

Fixed, thank you!

#7 @nacin
14 years ago

  • Milestone Future Release → 3.4
Note: See TracTickets for help on using tickets.