#163: Output of RSS feed should not be parsed as wiki text
-----------------------------------------------------+----------------------
Reporter: earle | Owner: dom
Type: defect | Status: assigned
Priority: normal | Milestone:
Component: openguides | Version: svn
Severity: normal | Resolution:
Keywords: hackfestsummer2007-reviewed, rss plugin |
-----------------------------------------------------+----------------------
Comment(by dom):
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.
--
Ticket URL: <http://dev.openguides.org/ticket/163#comment:8>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#163: Output of RSS feed should not be parsed as wiki text
-----------------------------------------------------+----------------------
Reporter: earle | Owner: dom
Type: defect | Status: assigned
Priority: normal | Milestone:
Component: openguides | Version: svn
Severity: normal | Resolution:
Keywords: hackfestsummer2007-reviewed, rss plugin |
-----------------------------------------------------+----------------------
Changes (by dom):
* owner: Nobody => dom
* status: new => assigned
--
Ticket URL: <http://dev.openguides.org/ticket/163#comment:7>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#10: Bug handling <pre>
-----------------------------------------+----------------------------------
Reporter: dom | Owner: Nobody
Type: defect | Status: new
Priority: normal | Milestone:
Component: openguides | Version:
Severity: normal | Resolution:
Keywords: hackfestsummer2007-reviewed |
-----------------------------------------+----------------------------------
Comment(by dom):
Creole uses
{{{
{{{
}}}
}}}
for pre-formatted text. I'm not sure how we should go about updating our
formatter though.
--
Ticket URL: <http://dev.openguides.org/ticket/10#comment:5>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#228: RSS parse errors should be non-fatal
------------------------+---------------------------------------------------
Reporter: dom | Owner: Nobody
Type: defect | Status: new
Priority: normal | Milestone:
Component: openguides | Version: svn
Severity: normal | Keywords:
------------------------+---------------------------------------------------
On eg
http://oxford.openguides.org/wiki/?id=Phoenix_Picturehouse&version=16
{{{
not well-formed (invalid token) at line 1, column 12, byte 12 at
/usr/lib/perl5/XML/Parser.pm line 187
}}}
This appears to relate to RSS including
http://www.timeout.com/film/listing_rss.php?cinemaID=354 which is now an
HTML 404 page.
This error should be handled more gracefully.
--
Ticket URL: <http://dev.openguides.org/ticket/228>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#23: Ignore null changes
------------------------+---------------------------------------------------
Reporter: dom | Owner: tgj
Type: defect | Status: assigned
Priority: normal | Milestone:
Component: openguides | Version:
Severity: normal | Resolution:
Keywords: patch |
------------------------+---------------------------------------------------
Comment(by dom):
Patch doesn't work for me:
{{{
t/010_metadata.........................Unhandled error: [Can't use string
("51.911") as an ARRAY ref while "strict refs" in use at /home/dom/working
/wiki-toolkit-2/wiki-toolkit/trunk/blib/lib/Wiki/Toolkit/Store/Database.pm
line 291.
] at /home/dom/working/wiki-toolkit-2/wiki-
toolkit/trunk/blib/lib/Wiki/Toolkit.pm line 847
}}}
error comes from the _checksum method, which is being called by the patch.
Haven't dug further yet.
--
Ticket URL: <http://dev.openguides.org/ticket/23#comment:7>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#263: Credit contributors properly
------------------------+---------------------------------------------------
Reporter: dom | Owner: Nobody
Type: task | Status: new
Priority: normal | Milestone:
Component: openguides | Version: svn
Severity: normal | Keywords:
------------------------+---------------------------------------------------
Many people have contributed to OpenGuides in minor as well as major ways
and we should list them all in a file in the distribution.
--
Ticket URL: <http://dev.openguides.org/ticket/263>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#265: Akismet plugin should submit a diff to akismet
-----------------------------------+----------------------------------------
Reporter: dom | Owner: dom
Type: defect | Status: new
Priority: normal | Milestone:
Component: oxford.openguides.org | Version: svn
Severity: normal | Keywords:
-----------------------------------+----------------------------------------
Rather than the whole page, Akismet should just submit a diff to Akismet;
sometimes it's impossible to make even trivial changes to existing
content, because the node contains content that looks (erroneously) like
spam.
Ideally of course Akismet wouldn't treat such things as spam, but this at
least means that these pages don't get locked forever.
--
Ticket URL: <http://dev.openguides.org/ticket/265>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#23: Ignore null changes
------------------------+---------------------------------------------------
Reporter: dom | Owner: tgj
Type: defect | Status: assigned
Priority: normal | Milestone:
Component: openguides | Version:
Severity: normal | Resolution:
Keywords: patch |
------------------------+---------------------------------------------------
Changes (by tgj):
* keywords: => patch
Comment:
Confirmed that two consecutive versions with no changes do have the same
checksum. So it's not that the checksum includes irrelevant metadata, but
rather that the checksum isn't checked.
Attached patch is just a simple check for an identical checksum in
Wiki::Toolkit::Store::Database. If the checksum is identical, the change
is aborted.
The included test works okay. Haven't tested the patch's interoperation
with openguides, but it looks like the (null) change will be silently
rejected (no indication given to user). To improve this would need an API
change for Wiki Toolkit to indicate that a change has been declined and
provide a reason why. Still, this patch might be an improvement on the
current situation.
--
Ticket URL: <http://dev.openguides.org/ticket/23#comment:6>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project
#242: clean target doesn't
------------------------+---------------------------------------------------
Reporter: dom | Owner: Nobody
Type: defect | Status: new
Priority: normal | Milestone:
Component: openguides | Version: svn
Severity: normal | Keywords:
------------------------+---------------------------------------------------
t/prefs.db (at least) is left behind. Should review clean target to ensure
it does clean up everything it should.
--
Ticket URL: <http://dev.openguides.org/ticket/242>
OpenGuides <http://openguides.org/>
The OpenGuides city guides project