Smartqueues configuration should rely on field names not field ids

The patch looks very good, but I think we need an update function to converting existing queues..

Tested the patch and it works great.

Commited to 7.x-2.x. Thanks!

http://drupalcode.org/project/nodequeue.git/commit/a4ea38b

Reopenening this, I think the update function is flawed. It seems to assume that a field always has the same ID as the taxonomy vocabulary. This is certainly not correct! In my case smartqueues ended up attached to the wrong vocab.

This problem happens when upgrading from Drupal 6, when smartqueues were handled via vocabulary ID, we need to add a hook_update_7000() that gather (or create) corresponding taxonomy reference fields and replace vids by field_ids.

Then the conversion from field_id to field_name will be handled by smartqueue_update_7001()

I uploaded a new patch that fixes the issue for me.

Unfortunately, I see no way to fix the issue for people who already made the upgrade.

Also, it may be required to implement some hook in case the field name changes, if it is not there already.

and changing the status.

wasn't quite right, needed hook_update_dependencies

It works when I run the update stand-alone, but not in a complete update. Strange.

As said in #7, this issue now concerns only the upgrade path from D6 to D7, so you should write the update hook as hook_update_7000().

It is generally a bad practice to modify already written update hooks :)

Well, I didn't mess up the initial hook. If you have an idea on how to help people that already did the upgrade, let me know. I have none, since the neccessary information was deleted during the faulty upgrade.

Also, it's just a beta release...