Decisions needs to have the node_info array changed to make it do something in 5.0. This will make it work in 5.0 again. It is relativly trivial, but does help.

CommentFileSizeAuthor
decisonsn5.patch.txt595 bytesAnonymous (not verified)

Comments

ChrisKennedy’s picture

ChrisKennedy commented
Status: Active » Needs work

Thanks for working on this - it looks solid although it seems that you reversed the order of the files in your diff.

Also, I would prefer to keep the cvs code compatible with 4.7 at least until the first drupal 5.0 release candidate. What do you think about keeping the 'base' property in there for backwards compatibility and leaving a comment explaining why (so we don't forget)?

ChrisKennedy’s picture

ChrisKennedy commented

Apparently the interface hook_node_info() is not yet finalized: http://drupal.org/node/78549

So we may want to hold off until that API issue is resolved.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Postponed

Ooops... I was just testing out Drupal 5 and testing some contrib modules. Its a still a bit of a moving target though it is in feature freeze, so Drupal should be relativly final. Should be possible to so one the right way next timne.

anarcat’s picture

anarcat commented
Status: Postponed » Fixed

Since 5.x is now in feature freeze, we are attempting to port to 5.0. This particular issue was fixed in issue #92107, but a lot of issues remain, especially with the display of decision forms, for which I will file a seperate bug report.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)