Make WordPress Core

Opened 11 years ago

Closed 8 years ago

Last modified 8 years ago

#10498 closed defect (bug) (fixed)

RSS widget filters some characters

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


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 9 years ago.

Download all attachments as: .zip

Change History (8)

#1 @azaozz
11 years ago

  • Milestone changed from 2.9 to 3.0

No patch, moving to 3.0

#2 @dd32
10 years ago

  • Keywords simplepie needs-patch added
  • Milestone changed from 3.0 to 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
9 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 9 years ago by solarissmoke (previous) (diff)

9 years ago

#4 @rmccue
9 years ago

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

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

#5 @rmccue
9 years ago

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

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

#6 @pavelevap
8 years ago

  • Resolution set to fixed
  • Status changed from assigned to 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
8 years ago

  • Milestone changed from Future Release to 3.4
Note: See TracTickets for help on using tickets.