Uploaded image for project: 'Magnolia RSS Aggregator Module'
  1. Magnolia RSS Aggregator Module
  2. MGNLRSSAGG-47

Improve HTML removal in feed ftl's

    XMLWordPrintable

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • 1.1.3, 1.2.1
    • 1.1.2, 1.2
    • None
    • None

    Description

      I've implemented some improvements to the regular expressions used in those FTLs to remove HTML from the entry description:

      • src/main/resources/info/magnolia/module/rssaggregator/paragraphs/feedList.ftl
      • src/main/resources/info/magnolia/module/rssaggregator/paragraphs/combinedFeed.ftl

      By mistake, I've committed the changes to the issue MGNLRSSAGG-33. Consider the following revisions as belonging to this issue:

      • Trunk:
        • r43516 – MGNLRSSAGG-33 Feed ftl's: Make regexp pattern more reliable
        • r43524 – MGNLRSSAGG Improve feed ftl's
        • r43541 – MGNLRSSAGG-33 Feed ftl's: Make regexp pattern even more reliable
        • r43543 – MGNLRSSAGG-33 Feed ftl's: Remove HTML code once, and before using ?length and ?substring builtins
        • r43571 – MGNLRSSAGG Easier regex
        • r43627 – MGNLRSSAGG-33 Feed ftl's: Remove partial HTML entities again after taking substring of description
      • 1.1 branch:
        • r43517 – MGNLRSSAGG-33 Feed ftl's: Make regexp pattern more reliable (backport of r43516)
        • r43525 – MGNLRSSAGG Improve feed ftl's (backport of r43524)
        • r43542 – MGNLRSSAGG-33 Feed ftl's: Make regexp pattern even more reliable (backport of r43541)
        • r43544 – MGNLRSSAGG-33 Feed ftl's: Remove HTML code once, and before using ?length and ?substring builtins (backport of r43543)
        • r43572 – MGNLRSSAGG Easier regex (backport of r43571)
        • r43628 – MGNLRSSAGG-33 Feed ftl's: Remove partial HTML entities again after taking substring of description (backport of r43627)

      Checklists

        Acceptance criteria

        Attachments

          Issue Links

            Activity

              People

                frabe Felix Rabe
                frabe Felix Rabe
                Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  Checklists

                    Task DoD