[Bug] Lost spaces in summary (#Plot) after words with length 4 and less (1 Viewer)

[arion_p]

Just a note:   html entity is a non-breaking space, so is 0xA0 character code in Unicode. So technically this (HtmlDecode) is working as expected. We need to make sure all non-breaking spaces are converted to regular spaces before importing to DB

 

[ShIvADeSt]

So may be rename topic in "Lost spaces in Summary (#Plot) if it is non-break space"? Or "Convert non-break spaces into regular spaces"

 

[ShIvADeSt]

Same trouble for some titles.

 

[Seidelin]

Just a note:   html entity is a non-breaking space, so is 0xA0 character code in Unicode. So technically this (HtmlDecode) is working as expected. We need to make sure all non-breaking spaces are converted to regular spaces before importing to DB

Should we add this to mantis then?

 

[arion_p]

I would say yes. Though I am not yet sure of the proper solution.

 

[Seidelin]

I marked the thread as "Bug" and put the issue in mantis.

 

[elliottmc]

Bumping this thread. Do we have a solution yet?

Mark

 

[Deda]

Yes I have it for a long time but can't verify it on kinopoisk examples because it changed HTML code to   which HTMLDecode correctly change to a space so I can't verify patch (I tested on only on user database and patch corrects those 0xA0).

 

[elliottmc]

Yes I have it for a long time but can't verify it on kinopoisk examples because it changed HTML code to   which HTMLDecode correctly change to a space so I can't verify patch (I tested on only on user database and patch corrects those 0xA0).

Is there any chance of regression for anyone else? What is your general feeling on this? Should the fix be committed?

Thanks,

Mark

 

[Deda]

Patch itself is harmless so it will do a transform for problematic non-breaking space and some XML special chars or string will just pass the code and nothing happens. Maybe it will be a good for the future to place html cleaning there so if something slips script, here will be converted to a normal char but it would need little attention because music plugin also uses this for converting strings with single quotas to a string suitable for sql lite database.