Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Here's a patch which removes the need to escape quotes whenever possible, either by swapping the quotes, or else by expanding contractions.
There are still some possessives which I can't do anything about because they're in the same sentence as links.
Partial patch. Will get to the rest when I get home.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | fix-escaped-quotes_2.patch | 39.61 KB | webchick |
| #7 | fix-escaped-quotes_1.patch | 39.03 KB | webchick |
| #5 | drupal_67.patch | 28.3 KB | ac |
| #3 | drupal_66.patch | 28.3 KB | ac |
| #2 | fix-escaped-quotes_0.patch | 21 KB | webchick |
Comments
Comment #1
drummThere are some t() additions hiding in here.
Comment #2
webchickBah! Thanks, drumm.
Here is a re-rolled version up to but not including menu.module. Putting this here in case someone interested in mind-numbing tasks wants to take it from here. ;)
Comment #3
acI have removed the unnecessary escapes like you asked webchick. Is this what you wanted?
Comment #4
webchickJust glancing through quickly, looks good to me!
Comment #5
acSlight mistake fixed
Comment #6
webchickLooks good, but I totally forgot about \" :P Rolling some more...
Comment #7
webchickLet's try this.
Comment #8
webchickOuch. Lots of failed hunks. Re-rolling.
Comment #9
webchickLet's try this again.
It's worth noting that this and the Move HTML out of strings wherever possible patch are going to collide in many places, so the sooner one of them gets committed, the sooner the other can get cleaned up.
Comment #10
dries commentedCommitted to CVS HEAD. Thanks.
Comment #11
(not verified) commented