port favorite_nodes to 6.x

subscribe

If I spent time to update this module to D6, is there interest in a patch?

Of course there is.

Go ahead and port it, test it, create a properly formatted patch and attach it here.

Thanks in advance.

Keen. It's gone rather well so far, but I haven't touched the views port yet, as it's only just gone into beta.

Alright, here's the patch I generated - it's off of svn's diff, so I hope it works. On the off chance it doesn't, I've included a tarball as well.

Again, I haven't touched the views stuff - in fact, I just commented that all out. I'm interested in looking at it later, but I did this at work as we needed the module for a site.

We need some testing for the D6 version in the previous comment before we can commit it.

Two or three positive confirmations that it does work should be nice.

The views stuff can wait till we have a views release.

Some bugs found
- favorite_nodes_xmlrpc (maps a nid as parameter but _add accepts node)
- list of nodes is not possible favorites/view disabled
--favorite_nodes/view/%user/%favorite_nodes_nodetype there's no need to put uid here better make parameter optional (look at node.module hook_menu (node/add and +types)
--there's no menu 'Favorites'

Hmmn, #269377: Port to Drupal 6.x with a different patch. Now there is a 6.x dev release. Updating this issue.