Per http://drupal.org/update/modules/6/7#drupal_behaviors

nodequeue_dragdrop.js hadn't been updated to use the new way to create Drupal.behaviors. Patch forthcoming.

CommentFileSizeAuthor
#7 994996-followup_update_javascript_to_drupal_7.patch971 bytesamateescu
#4 nodequeue_dd_js_994996_1.patch6.41 KBbblake
#1 nodequeue_dd_js_994996.patch6.43 KBbblake

Comments

bblake’s picture

bblake commented
StatusFileSize
new nodequeue_dd_js_994996.patch6.43 KB
bblake’s picture

bblake commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, nodequeue_dd_js_994996.patch, failed testing.

bblake’s picture

bblake commented
Status: Needs work » Needs review
StatusFileSize
new nodequeue_dd_js_994996_1.patch6.41 KB

Updated to successfully patch against HEAD

slashrsm’s picture

slashrsm commented
Status: Needs review » Reviewed & tested by the community

Code looks fine. All features seems to work.

merlinofchaos’s picture

merlinofchaos commented
Status: Reviewed & tested by the community » Fixed

The patch didn't apply, so I basically recreated it. If you did anything other than just add attach: and the jQuery wrapper, that didn't make it in.

amateescu’s picture

amateescu commented
Status: Fixed » Needs review
StatusFileSize
new 994996-followup_update_javascript_to_drupal_7.patch971 bytes

The nodequeueClearTitle behavior was not updated. Here is a patch to fix this issue and #899864: "Enter the title of a node to add it to the queue" does not clear when adding new nodes to queues

amateescu’s picture

amateescu commented
Status: Needs review » Fixed

Commited to 7.x.

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

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.