New RSS handler

[DarTar]

The current RSS handler should be replaced with a more flexible tool like the one tested here:

 http://wikkawiki.org/RSSHandler

As soon as we adopt a better RSS handler and we are able to produce multiple feeds for global changes, local changes, comments (#335) or other modifications (#143), a config option (#249) could be used to determine what kind of feeds must be generated and made auto-discoverable.

Related tickets

• #143
• #249
• #335
• #484

[DarTar]

Updating description with ref to #335 (closed as duplicate).

[DarTar]

[367] adds the 3rdparty feedcreator class required by the new handlers.

[DarTar]

[368] adds docblock and SVN id keyword

[DarTar]

[412] adds preliminary code for feed generation handlers for recent changes and recent comments using FeedCreator.

[DotMG]

Bug in your code: comments.xml.php line 71 (replace plugins by core).

[DarTar]

[422] fixes the above issues, thanks.

[DarTar]

adding related tickets

[DarTar]

Retargeting to 1.3. Code for this ticket may have already been committed to trunk, from which 1.3 will be branched. Consider backporting urgent issues to 1.2.X

[BrianKoontz]

(In [1512]) Ported feedcreator code from trunk to 1.3. Refs #250.

[DotMG]

Actually, test could not pass because of error_reporting(E_ALL i suppose) and PHP Notices.

<b>Notice</b>:  Undefined property: Wakka::$method in <b>/home/mahefa/homemade/wikka/branches/1.4/handlers/recentchanges.xml/recentchanges.xml.php</b> on line <b>104</b><br /> 
<br /> 
<b>Notice</b>:  Use of undefined constant FEED_IMAGE_URL - assumed 'FEED_IMAGE_URL' in <b>/home/mahefa/homemade/wikka/branches/1.4/handlers/recentchanges.xml/recentchanges.xml.php</b> on line <b>109</b><br /> 

Have to tackle some part of #38 first...

[DotMG]

Added error_reporting(E_ALL ^{E_NOTICE); on top of my working copy to allow me to test anyway and the test failed. Links on the RSS become something like}

wikka.php?wakka=rakoto&time=2010-04-23+20%3A56%3A32 instead of

wikka.php?wakka=rakoto&time=2010-04-23+20%3A56%3A32, thus the parameter time is ignored.

On the xml source, it's written &time=

[DotMG]

(In [1635]) Refs #250

• Removes Notices and error messages to avoid that the resulting XML feed is invalid, done in double step (Fixed FEED_IMAGE_URL not defined, replaced $this->method with $this->handler) (Added ob_start() and ob_end_clean() to catch all other possible error messages
• Fixed double escaping of ampersands in URL

[DotMG]

Taking ownership of the ticket ...

[BrianKoontz]

Tested with both rewrite_mode enabled and disabled. Links work correctly.

[BrianKoontz]

Updated milestone to 1.3.1