Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

html entity weirdness #259

Open
GoogleCodeExporter opened this issue May 31, 2015 · 3 comments
Open

html entity weirdness #259

GoogleCodeExporter opened this issue May 31, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

Steps to reproduce:
1. Edit your plan. Write this: "&. &. &. &."
2. Save plan. Look at plan. You should see "&. &. &. &."
3. Edit your plan again. Notice that the edit text now says "&. &. &. &." 

What is the expected output? What do you see instead?
I would expect the function converting input text to display text, and the 
reverse, to be a bijection.

Other pertinent information (browser version, revision number, etc):
Chromium/Linux 29.0.1547.57 Debian jessie/sid (217859)

Original issue reported on code.google.com by [email protected] on 13 Sep 2013 at 5:00

@GoogleCodeExporter
Copy link
Author

Looks like edit.php needs to call htmlentites() on the edit_text before putting 
it in the textbox. There's some fun to be had here - for example, put "<!--" in 
your plan's text, and it'll break the edit page in a way that would be very 
difficult for an average user to fix. Fun!


Original comment by [email protected] on 13 Sep 2013 at 5:08

@GoogleCodeExporter
Copy link
Author

Lulz, I've actually used this bug as a feature when I wanted to pad text using 
non-breaking spaces. I write  's in my update and then do two roundtrips 
through the edit form. Anyway I have a better way of doing it now and we should 
probably fix this. Thanks for reporting it.

Original comment by umhecbaa on 13 Sep 2013 at 5:19

@GoogleCodeExporter
Copy link
Author

Oh, this should probably be merged with issue 54 if it's general to all html 
entities.

(Umhecbaa (?), you can also use U+2008 to pad text in a plan without the round 
trip. I don't get what's up with whitespace on plans at all.)

Original comment by [email protected] on 13 Sep 2013 at 6:15

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant