extmenu drag and drop is not available

[rallek]

in ci build 232 is no drag and drop available to change the sort order of the menu items in extmenu.

[cmfcmf]

Duplicate of #665.

[rallek]

a little difference: Now we cant move it, there is nother where to grip the item ;-)

[ghost]

@craigh - did you fix this - something to do with paths to images.

[craigh]

my fixes recently were superficial. I think this is an AJAX response issue? I've never used ext menu 😀

[ghost]

ping @jusuff

[craigh]

this is a javascript error I think. I'm getting errors in the console like:

TypeError: 'undefined' is not an object (evaluating 'draggable.element.down('a').stopObserving')
TypeError: 'undefined' is not an object (evaluating 'this.dropCache.lastElement = ['after',dropOnNode.identify()]')
TypeError: 'undefined' is not an object (evaluating 'this.dropCache.element = dropOnNode')
TypeError: 'undefined' is not an object (evaluating 'this.dropCache.lastElement = ['before',dropOnNode.identify()]')
TypeError: 'undefined' is not an object (evaluating 'this.dropCache.lastElement')
TypeError: 'undefined' is not an object (evaluating 'draggable.element.down('a').stopObserving')
TypeError: 'undefined' is not an object (evaluating 'this.dropCache.lastElement = ['after',dropOnNode.identify()]')

[cmfcmf]

I tested a bit and figured out that there is something wrong with the recursive option in javascript/helpers/Zikula.itemlist.js. Setting recursive to false (in blocks_block_extmenu_modify.tpl of Blocks module) "solves" the problem. Sorting then works fine.
I actually don't know what the recursive parameter is intended for. Is it for creating nested menus?

- a
- b
--- b1 

[espaan]

It says in itemlist:

 * @param {Boolean} [options.recursive=false] Should list be recursivly sortable (allows to create multilevel lists)

In the extmenu is says that you can also use it as multi level menu. Although I have never seen it being used like that and menutree is far better for that I think.

[espaan]

Nice !

[phaidon]

Welcome.

[rallek]

I changed recursive to false in 1.3.5 and it works. Great!

[cmfcmf]

@rallek Use @phaidon's patch (#924) instead (I adopted it for Zikula 1.3.5, see https://gist.github.com/cmfcmf/6329308). Setting recursive to false was more of a workaround.

[rallek]

Thanks to @cmfcmf! I took your adopted patch and it works. Would be great to get it official in #1355. But that is only for security :(