Summary: | XHTML correction in authority summary | ||
---|---|---|---|
Product: | Koha | Reporter: | Fridolin Somers <fridolin.somers> |
Component: | Staff interface | Assignee: | Fridolin Somers <fridolin.somers> |
Status: | CLOSED FIXED | QA Contact: | Bugs List <koha-bugs> |
Severity: | minor | ||
Priority: | P3 | CC: | chris, gmcharlt, kyle.m.hall, paul.poulain |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Attachments: |
Proposed patch
Proposed patch (updated) BUG 5312 XHTML correction in authority summary [SIGNED-OFF] BUG5312 XHTML correction in authority summary |
Description
Fridolin Somers
2010-10-19 07:53:21 UTC
Created attachment 2693 [details] [review] Proposed patch (Correction of comment #0) > When an authority summary contains a "\n",it is replaced by a HTML's "<br>". > XHTML new line is "<br />". Patch is correct. needs signoff status forgotten, setting it assigning to Fridolyn, as he sent the patch Change looks good to me but patch is not in a valid format: Patch format detection failed. Fridolyn, could you resend as git patch please? Created attachment 5085 [details] [review] Proposed patch (updated) recreated with git format-patch Created attachment 9997 [details] [review] BUG 5312 XHTML correction in authority summary Created attachment 9999 [details] [review] [SIGNED-OFF] BUG5312 XHTML correction in authority summary Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> QA comment: tiny patch that produces valid XHTML, passes QA of course ! worth porting to 3.8 Thanks everyone. I've learned a bit of XHTML since : new line is "<br/>" (no space). Patch is correct. There have been no further reports of problems so I am marking this bug resolved. |