![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#9433 closed defect (bug) (fixed)
Patch for #9199 adds extra backslashes to pending-review previews
| Reported by: | ramenboy | Owned by: | |
|---|---|---|---|
| Priority: | high | Milestone: | 2.8 |
| Component: | Administration | Version: | 2.7 |
| Severity: | major | Keywords: | |
| Cc: | Focuses: |
Description
Changesets [10620] and [10621] for #9199 causes broken previews for posts with non-draft status.
Steps to reproduce:
- Posts -> Add New
- Enter a title with quotes and apostrophes
- Enter content with quotes and apostrophes
- In the Publish box next to "Status: Draft", click "Edit"
- Change status to "Pending Review", click "OK"
- Click "Save as Pending"
- Click "Preview"
- Quotes and apostrophes in title and content will have backslashes before them
In addition to mangling the text, this is causing broken images in previews.
Rolling back the change fixes the problem, though it probably reintroduces the original issue #9199.
Change History (17)
#4
@
17 years ago
- Milestone 2.8 → Future Release
Punting due to feature freeze. Reconsider with next release.
#7
@
17 years ago
- Milestone 2.8
- Resolution → fixed
- Status new → closed
seems fixed in trunk. please re-open with feedback if you experience this again.
#8
@
17 years ago
- Resolution fixed
- Status closed → reopened
I just updated to the latest trunk, and I am still able to reproduce this. There have been no changes to post.php since [10620], so I don't see why it would be fixed.
#9
@
17 years ago
weird. I tried it earlier today, and apart from the broken preview link that is highlighted in a separate ticket it seemed to work fine...
#10
@
17 years ago
Unable to reproduce. I get a big fat JS Error on the Add Mew Page with 2.8 Bleeding. Sorry.
#11
@
17 years ago
Fixed my error and tried to reproduce. I was able to reproduce (latest trunk). The slashes article is dispolayed. There is a post _with_ slashes in the database. Looks like somewhere this got slashed like hell before saved in the database.
Save as Pending a second time and then previewing does not have such errors. the old database entry is fixed then (that one that was with slashes inside the db), the new is okay then.
mysterious mircale.
#13
@
17 years ago
- Keywords dev-feedback added; developer-feedback removed
- Milestone → 2.8
Resetting to 2.8 since its a bug thats been confirmed.
Observed in 2.7 and trunk.