Fix drag-n-drop after folder move/create (#1489648)

Aleksander Machniak

2014-07-25 2c0d3e1dd0a3df01c8adea0de5f2826f0bcb9434

 program/js/treelist.js

@@ -46,7 +46,7 @@
    scroll_speed: 20,
    save_state: false,
    keyboard: true,
    check_droptarget: function(node){ return !node.virtual }
    check_droptarget: function(node) { return !node.virtual; }
  }, p || {});

  var container = $(node),
@@ -67,6 +67,9 @@
    searchfield,
    tree_state,
    ui_droppable,
    ui_draggable,
    draggable_opts,
    droppable_opts,
    list_id = (container.attr('id') || p.id_prefix || '0'),
    me = this;

@@ -83,6 +86,7 @@
  this.drag_end = drag_end;
  this.intersects = intersects;
  this.droppable = droppable;
  this.draggable = draggable;
  this.update = update_node;
  this.insert = insert;
  this.remove = remove;
@@ -108,7 +112,11 @@
    e.stopPropagation();
  });

  container.on('click', 'li', function(e){
  container.on('click', 'li', function(e) {
    // do not select record on checkbox/input click
    if ($(e.target).is('input'))
      return true;

    var node = p.selectable ? indexbyid[dom2id($(this))] : null;
    if (node && !node.virtual) {
      select(node.id);
@@ -226,15 +234,24 @@
  function select(id)
  {
    if (selection) {
      id2dom(selection).removeClass('selected').removeAttr('aria-selected');
      id2dom(selection, true).removeClass('selected').removeAttr('aria-selected');
      if (search_active)
          id2dom(selection).removeClass('selected').removeAttr('aria-selected');
      selection = null;
    }

    var li = id2dom(id);
    if (!id)
      return;

    var li = id2dom(id, true);
    if (li.length) {
      li.addClass('selected').attr('aria-selected', 'true');
      selection = id;
      // TODO: expand all parent nodes if collapsed

      if (search_active)
        id2dom(id).addClass('selected').attr('aria-selected', 'true');

      scroll_to_node(li);
    }

@@ -455,7 +472,7 @@
  /**
   *
   */
  function reset()
  function reset(keep_content)
  {
    select('');

@@ -463,7 +480,24 @@
    indexbyid = {};
    drag_active = false;

    container.html('');
    if (keep_content) {
      if (draggable_opts) {
        if (ui_draggable)
          draggable('destroy');
        draggable(draggable_opts);
      }

      if (droppable_opts) {
        if (ui_droppable)
          droppable('destroy');
        droppable(droppable_opts);
      }

      update_data();
    }
    else {
      container.html('');
    }

    reset_search();
  }
@@ -702,6 +736,7 @@
  {
    var domid = p.id_encode ? p.id_encode(id) : id,
      suffix = search_active && !real ? '--xsR' : '';

    return $('#' + p.id_prefix + domid + suffix, container);
  }

@@ -807,8 +842,8 @@
    else if (dir < 0 && !from_child && next.children('ul[role=group]:visible').length) {
      next.children('ul').children('li:last').find('a:first').focus();
    }
    else if (next.length && next.find('a:first')) {
      next.find('a:first').focus();
    else if (next.length && next.find('a:first').focus().length) {
        // focused
    }
    else {
      parent = li.parent().closest('li[role=treeitem]');
@@ -844,6 +879,11 @@
   */
  function drag_start()
  {
    if (drag_active)
      return;

    drag_active = true;

    var li, item, height,
      pos = container.offset();

@@ -851,7 +891,6 @@
    list_scroll_top = container.parent().scrollTop();
    pos.top += list_scroll_top;

    drag_active = true;
    box_coords = {
      x1: pos.left,
      y1: pos.top,
@@ -914,6 +953,9 @@
   */
  function drag_end()
  {
    if (!drag_active)
      return;

    drag_active = false;
    scroll_timer = null;

@@ -944,7 +986,7 @@
  }

  /**
   * Determine if the given mouse coords intersect the list and one if its items
   * Determine if the given mouse coords intersect the list and one of its items
   */
  function intersects(mouse, highlight)
  {
@@ -964,8 +1006,8 @@
    }

    // check intersection with visible list items
    var pos, node;
    for (var id in item_coords) {
    var id, pos, node;
    for (id in item_coords) {
      pos = item_coords[id];
      if (mouse.x >= pos.x1 && mouse.x < pos.x2 && mouse.top >= pos.y1 && mouse.top < pos.y2) {
        node = indexbyid[id];
@@ -1018,7 +1060,24 @@
   */
  function droppable(opts)
  {
    var my_opts = $.extend({ greedy: true, hoverClass: 'droptarget', addClasses:false }, opts);
    if (!opts) opts = {};

    if ($.type(opts) == 'string') {
      if (opts == 'destroy') {
        ui_droppable = null;
      }
      $('li:not(.virtual)', container).droppable(opts);
      return this;
    }

    droppable_opts = opts;

    var my_opts = $.extend({
        greedy: true,
        tolerance: 'pointer',
        hoverClass: 'droptarget',
        addClasses: false
      }, opts);

    my_opts.activate = function(e, ui) {
      drag_start();
@@ -1041,6 +1100,45 @@
    };

    $('li:not(.virtual)', container).droppable(my_opts);

    return this;
  }

  /**
   * Wrapper for jQuery.UI.draggable() activation on this widget
   *
   * @param object Options as passed to regular .draggable() function
   */
  function draggable(opts)
  {
    if (!opts) opts = {};

    if ($.type(opts) == 'string') {
      if (opts == 'destroy') {
        ui_draggable = null;
      }
      $('li:not(.virtual)', container).draggable(opts);
      return this;
    }

    draggable_opts = opts;

    var my_opts = $.extend({
        appendTo: 'body',
        revert: 'invalid',
        iframeFix: true,
        addClasses: false,
        cursorAt: {left: -20, top: 5},
        create: function(e, ui) { ui_draggable = ui; },
        helper: function(e) {
          return $('<div>').attr('id', 'rcmdraglayer')
            .text($.trim($(e.target).first().text()));
        }
      }, opts);

    $('li:not(.virtual)', container).draggable(my_opts);

    return this;
  }
}