As soon as Drupal 6 is out we should be there with a 6 version of FeedAPI.

Converting 5.x modules to 6.x

CommentFileSizeAuthor
#17 feedapi-update6x_remainings.patch2.78 KBmustafau
#16 feedapi-update6x_remainings.patch1.53 KBmustafau

Comments

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented

+1. We should definitely provide Drupal 6 version before the official release of Drupal 6.
Suggested way:
- decide on all features / tasks in the issue queue - is it essential for feedapi 5.x?
- implement the must-have features in 5.x
- do a feature-freeze for 5.x After this point __only__ bug fixes should get into the code.
- upgrade feedapi for 6
- maintain 5.x and 6.x, but new features are only for 6.x

Do you think it's acceptable?
It 'd be so good to avoid the endless cycle of porting features between branches.

alex_b’s picture

alex_b commented

Freezing 5.x development will be hard as we are using feedapi on a series of 5 pages. So at least for a little while we will need to maintain 2 branches. Lets see how things develop. I think this issue shouldn't keep us from upgrading early.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented

Subscribing.

alex_b’s picture

alex_b commented

Good news: Jose Reyero will upgrade feedapi to 6 :)

We should use the upgrade to break out some modules and let them live in their own space.

Let's upgrade

* feedapi_inherit
* feedapi_node
* feedapi_node_views
* parser_common_syndication
* parser_simplepie
* feedapi

and break out

* feedapi_aggregator

into its own project.

Jose will review code as he upgrades. Outstanding tests and bug fixes for 5 will be upgraded as we go.

kpaul’s picture

kpaul commented

Thanks guys. Bumping this to the top with D6 being released recently.

ilbeppe’s picture

ilbeppe commented

Any news???

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Assigned: Unassigned » aron novak
Priority: Normal » Critical

Yes, I'm assigned to work on this. You can expect the first DRUPAL-6 release this week.

kpaul’s picture

kpaul commented

Thanks for the update. I may be able to help test/debug.

designerbrent’s picture

designerbrent commented

subscribing

ilbeppe’s picture

ilbeppe commented

News?

cosmicdreams’s picture

cosmicdreams commented

Thanks for your ongoing work Aron. I hope to talk with you in irc about how I can help you best.

Will I see you in there this week? Or are you busy with ramping up for the Feed API talk at Boston?

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Title: Upgrade to 6 » Try out 6.x-dev
Version: 5.x-1.x-dev » 6.x-1.x-dev

Please try out 6.x-dev package! The CVS contains DRUPAL-6--1 branch. The dev package will be available shortly.
@cosmicdreams : i won't attend DrupalCon Boston.

cosmicdreams’s picture

cosmicdreams commented
Version: 6.x-1.x-dev » 5.x-0.x-dev

Great news! I won't have to worry about backporting my module to Drupal 5 now.

cosmicdreams’s picture

cosmicdreams commented

Bah, sorry. I can't see the new 6.x version in the Version drop down.

alex_b’s picture

alex_b commented
Version: 5.x-0.x-dev » 6.x-1.x-dev

don't worry :)

mustafau’s picture

mustafau commented
Status: Active » Needs review
StatusFileSize
new feedapi-update6x_remainings.patch1.53 KB

Arguments to feedapi_inherit_help() were not updated to 6.x.

Patch is attached.

mustafau’s picture

mustafau commented
StatusFileSize
new feedapi-update6x_remainings.patch2.78 KB

Updated patch.

Additionally coder says:

Translations moved from ./po to ./translations
aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Component: Code » Code (general)
Status: Needs review » Closed (fixed)

Old ticket, closing it now. it does not makes sense anymore.