[MGNLRSSAGG-48] Superfluous spaces in feed ftl's Created: 27/Apr/11 Updated: 04/Nov/15 Resolved: 04/Nov/15 |
|
| Status: | Closed |
| Project: | Magnolia RSS Aggregator Module |
| Component/s: | None |
| Affects Version/s: | 1.1.3, 1.2.1 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Neutral |
| Reporter: | Felix Rabe | Assignee: | Felix Rabe |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||
| Issue Links: |
|
||||||||||||
| Template: |
|
||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||
| Date of First Response: | |||||||||||||
| Description |
|
My fixes for The original solution tackled the case of "... paragraph.</p><h1>Heading" becoming "... paragraph.Heading". Solution proposal: Change: ?replace("</?[^>]*(>|$)| ?|&#?[0-9A-Za-z]*$", " ", "r")
into: ?replace("</[^>]*><[^/>]*>| ?", " ", "r")?replace("</?[^>]*(>|$)|&#?[0-9A-Za-z]*$", "", "r")
such that "</x><y>" and " " become " " (one space character), whereas any other tags, and end-of-string (partial) entities become "" (i.e. removed). |
| Comments |
| Comment by Michael Mühlebach [ 04/Nov/15 ] |
|
Given the thousands of other issues we have open that are more highly requested, we won't be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes. |