[Plugin: Register Plus Redux] RPR not escaping data well

  • Resolved webakimbo

    (@webakimbo)


    15 years, 6 months ago

    Noticed some frustrating bugs regarding how the plugin is escaping and saving data.

    1) Quotes are allowed in CSS, most notably when you want to include a font family that isn’t a single word (e.g. “Gill Sans”). RPR escapes those quotes, which makes the CSS invalid.

    2) I created a custom dropdown with a list of countries. If a quote or apostrophe (e.g. Cote D’Ivoire) is found, the list is simply truncated at the offending character’s first appearance. Not good!

    Seems like an easy fix for a serious limitation, maybe a patch exists?

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

    (@radiok)

    15 years, 6 months ago

    I fixed the first bug on the trunk, I’m not quite as sure how to fix the second bug. I’ll have to mull that over, my first attempt to just add slashes to that field didn’t work… Not sure why, but I’ll figure it out soon enough.

    Plugin Author radiok

    (@radiok)

    15 years, 6 months ago

    webakimbo, I finally figured out your second problem, the resolution might surprise you, ok, it probably won’t, but it surprised me. Redux is storing the data just fine, this is a display problem only. When Redux writes these values to HTML, it is with single quotes, so Cote D’Ivoire becomes ‘Cote D’Ivoire’. I think the problem is pretty obvious. So I’m working on putting all these fields into double quotes, it’s just a bit painful to escape all these quotes, ugh. Resolution is forthcoming.

    Plugin Author radiok

    (@radiok)

    15 years, 6 months ago

    This has been resolved in 3.6.21

    Thread Starter webakimbo

    (@webakimbo)

    15 years, 6 months ago

    Thanks! Good sleuthing.

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

The topic ‘[Plugin: Register Plus Redux] RPR not escaping data well’ is closed to new replies.