Opened 12 years ago

Closed 6 years ago

#163 closed defect (duplicate)

Output of RSS feed should not be parsed as wiki text

Reported by: Earle Martin Owned by:
Priority: low Milestone:
Component: openguides Version: svn
Severity: trivial Keywords: migrated hackfestsummer2007-reviewed, rss plugin triage
Cc:

Description

Looks like leading spaces on lines in RSS output are being interpreted as preformatted text wiki formatting by Text::WikiFormat?.

http://london.openguides.org/index.cgi?South_Bank_Centre

Change History (10)

comment:1 Changed 12 years ago by Dominic Hargreaves

Summary: Output of RSS feed embedding appearing as preformatted textOutput of RSS feed should not be parsed as wiki text

comment:2 Changed 12 years ago by Kake

We should do this by changing Wiki::Toolkit::Formatter::UseMod so it accepts "pre_format_macros" and "post_format_macros" as alternative arguments to the current "macros" argument.

comment:3 Changed 12 years ago by Kake

Keywords: hackfestsummer2007-reviewed added

comment:4 Changed 12 years ago by Kake

Owner: changed from Earle Martin to Kake

comment:5 Changed 12 years ago by Kake

Status: newassigned

comment:6 Changed 10 years ago by kake

Owner: changed from Kake to Nobody
Status: assignednew

comment:7 Changed 10 years ago by Dominic Hargreaves

Owner: changed from Nobody to Dominic Hargreaves
Status: newassigned

comment:8 Changed 10 years ago by Dominic Hargreaves

I think it would be better to have the possibility of macros being passed in as hashrefs, so they can contain hashrefs. Since the RSS macros currently reverses the HTML-escaping performed by the formatter, we should take advantage of this to introduce an option for whether to process the macro before the escaping too. This would mean individually formatting each of the macros separately, to allow for all such combinations.

comment:9 Changed 7 years ago by bob

Keywords: triage added
Owner: Dominic Hargreaves deleted
Priority: normallow
Severity: normaltrivial
Status: assignednew

i wonder if anyone actually uses the RSS embedding stuff anymore. it causes issues with installs as it is.

needs checking to see if its still an issue anyway

comment:10 Changed 6 years ago by bob

Keywords: migrated added
Resolution: duplicate
Status: newclosed
Note: See TracTickets for help on using tickets.