Another small step, here is a patch to start the port of the syndication.module to Drupal 5.x.

This patch addresses a minor Drupal 4.7.x to 5.x change; module_exist() is now module_exists()

Keep in mind, I'm trying to teach myself PHP so excuse the errors (if they exist).

Still more to do!

Russ

CommentFileSizeAuthor
#9 syndication_5.x_05.patch11.3 KBkiz_0987
#6 syndication_5.x_04.patch7.97 KBandremolnar
#5 syndication_5.x_03.patch7.99 KBandremolnar
#4 syndication_5.x_02.patch5.27 KBandremolnar
syndication_5.x_01.patch2.49 KBrport

Comments

canadrian’s picture

canadrian commented

Would be interested to know when this is ported to 5.x officially

georgedamonkey’s picture

georgedamonkey commented

Me too!!

spazio’s picture

spazio commented

Would be great if the module would be completely ported to 5.x. Could you post your version with patch here in the meantime?

Thank you!

Mario

andremolnar’s picture

andremolnar commented
Category: bug » feature
Status: Needs work » Needs review
StatusFileSize
new syndication_5.x_02.patch5.27 KB

Here's a port which includes settings and fixes module_exist(s).
Also cleans up some line endings that had extra whitespace.

andremolnar’s picture

andremolnar commented
StatusFileSize
new syndication_5.x_03.patch7.99 KB

Here is an updated patch that ports the module to 5.x and also includes the ability to display feeds generated by views.

andremolnar’s picture

andremolnar commented
StatusFileSize
new syndication_5.x_04.patch7.97 KB

Ummm - this time minus the debug.

leoklein’s picture

leoklein commented

Re: syndication_5.x_04.patch

I applied this to the Syndication Module HEAD.

In Administer under "RSS feed syndication", I see the heading "Views:" but none of my Views are listed.

andremolnar’s picture

andremolnar commented

The views functionality will only display views that are generating feeds - it will not create the feeds for you - you will have to configure your views for that.
see:http://drupal.org/node/83597

andre

kiz_0987’s picture

kiz_0987 commented
StatusFileSize
new syndication_5.x_05.patch11.3 KB

Updated patch:
- Fixed user search form
- Fixed display of hierarchy in taxonomy
- Fixed t() calls

andremolnar’s picture

andremolnar commented

I happen to be on vacation at the moment, but I will be rolling a 5.x release of this module when I come back. I will review this patch at that time.

andre

andremolnar’s picture

andremolnar commented

I've gone ahead and created a 5.x branch for the project and have set up the project to create daily development snapshots. Once people have had a chance to test / provide feedback - and needed changes are commited - I'll create a proper 1.0 release.

kiz_0987 - can you create a new patch against the 5.x branch?

andre

andremolnar’s picture

andremolnar commented
Status: Needs review » Closed (fixed)

I'm actually going to close this issue. If there are other issues that could/should have been fixed we can deal with those as issues against the new 5.x version.

Module has been ported to 5.

andre