Question

我正在为Prestashop 1.6构建一个非常简单的模块，我添加了一个管理界面，可以列出我的记录，一个表单来添加，编辑和删除。

这可以正常工作，你可以在这里看到： enter image description here

问题是可拖动的重新排序按钮不可拖动。因此重新排序不起作用......

根据official docs，如果在控制器中设置位置选项，则会获得可拖动的功能：

['position'] => 'position', // If set to position, the field will display arrows and be drag and droppable, which will update position in db (optional).

这是我模块控制器的相关部分：

$this->fields_list = array(
        'id_quicklinks' => array(
            'title' => $this->l('ID'),
            'align' => 'center',
            'width' => 25
        ),
        'titulo' => array(
            'title' => $this->l('Titulo'),
            'width' => 'auto'
        )
        , 'lead' => array(
            'title' => $this->l('Subtitulo'),
            'width' => 'auto'
        ), 
        'position' => array(
            'title' => $this->l('Ordem'),
            'filter_key' => 'a!position',
            'position' => 'position',
            'align' => 'center',
            'class' => 'fixed-width-md'
        ),
        'active' => array(
            'title' => $this->l('Publicado'),
            'width' => '25',
            'active' => 'status'
        )
    );

正如您在打印屏幕中看到的那样，显示了手柄，但拖放功能并不起作用。控制台上没有javascript错误，没有什么......我可以在源代码中看到jQuery和jQueryUI被加载。其他具有重新排序功能的管理页面正常工作......

有什么想法吗？感谢。

Answer 1

好吧，如果有人有兴趣，我设法让这个工作。

在我的模块管理控制器中，我已经添加了（在__construct方法之前，在打开类之后）：

protected $position_identifier = 'id_quicklinks';

id_quicklinks是此模块使用的数据库表的主键。

这启用了我正在寻找的拖放功能，但是现在我可以拖放，订单没有保存在数据库中。

为了实现这一点，我添加了另外两种改编自controllers/admin/AdminCarriersController.php和classes/Carrier.php的方法：

public function ajaxProcessUpdatePositions()
    {
        $way = (int)Tools::getValue('way');
        $id_quicklinks = (int)Tools::getValue('id');
        $positions = Tools::getValue('quicklinks');

        if (is_array($positions))
            foreach ($positions as $position => $value)
            {
                $pos = explode('_', $value);

                if (isset($pos[2]) && (int)$pos[2] === $id_velcroquicklinks)
                {
                        if (isset($position) && $this->updatePosition($way, $position, $id_quicklinks))
                            echo 'ok position '.(int)$position.' for id '.(int)$pos[1].'\r\n';
                        else
                            echo '{"hasError" : true, "errors" : "Can not update id '.(int)$id_quicklinks.' to position '.(int)$position.' "}';

                    break;
                }
            }

    }

和

public function updatePosition($way, $position, $id)
    {

        if (!$res = Db::getInstance()->executeS('
            SELECT `id_quicklinks`, `position`
            FROM `'._DB_PREFIX_.'quicklinks`
            ORDER BY `position` ASC'
        ))
            return false;

        foreach ($res as $quicklinks)
            if ((int)$quicklinks['id_quicklinks'] == (int)$id)
                $moved_quicklinks = $quicklinks;

        if (!isset($moved_quicklinks) || !isset($position))
            return false;
        var_dump($moved_quicklinks['position']);
        // < and > statements rather than BETWEEN operator
        // since BETWEEN is treated differently according to databases
        return (Db::getInstance()->execute('
            UPDATE `'._DB_PREFIX_.'quicklinks`
            SET `position`= `position` '.($way ? '- 1' : '+ 1').'
            WHERE `position`
            '.($way
                ? '> '.(int)$moved_quicklinks['position'].' AND `position` <= '.(int)$position
                : '< '.(int)$moved_quicklinks['position'].' AND `position` >= '.(int)$position.'
            '))
        && Db::getInstance()->execute('
            UPDATE `'._DB_PREFIX_.'quicklinks`
            SET `position` = '.(int)$position.'
            WHERE `id_quicklinks` = '.(int)$moved_quicklinks['id_quicklinks']));
    }

我希望这可以帮助有同样问题的人。

Answer 2

感谢您的解决方案。

我花了很长时间才意识到ajaxProcessUpdatePositions()以某种方式单独调用。对于那些努力实现该方法的人，请注意，当在表格中移动元素时调用方法时，echo和var_dump()将不会打印任何内容。

另外，我发现了一些优化代码的方法。

您可以替换：

if (!$res = Db::getInstance()->executeS('
       SELECT `id_quicklinks`, `position`
       FROM `'._DB_PREFIX_.'quicklinks`
       ORDER BY `position` ASC'
))
    return false;

foreach ($res as $quicklinks)
    if ((int)$quicklinks['id_quicklinks'] == (int)$id)
         $moved_quicklinks = $quicklinks;

通过：

if (!$moved_quicklinks  = Db::getInstance()->executeS('
       SELECT `id_quicklinks`, `position`
       FROM `'._DB_PREFIX_.'quicklinks`
       WHERE `id_quicklinks` = ' . (int) $id . '
       LIMIT 1;'
))
    return false;

数据库返回的行（和字段）越少越好。而且，它消除了PHP代码中循环的需要。

您还可以在同一个数据库调用中合并两个更新查询：

return (Db::getInstance()->execute('
            UPDATE `'._DB_PREFIX_.'quicklinks`
            SET `position`= `position` '.($way ? '- 1' : '+ 1').'
            WHERE `position`
            '.($way
                ? '> '.(int)$moved_quicklinks[0]['position'].' AND `position` <= '.(int)$position
                : '< '.(int)$moved_quicklinks[0]['position'].' AND `position` >= '.(int)$position.'
            ') . '; UPDATE `'._DB_PREFIX_.'quicklinks`
            SET `position` = '.(int)$position.'
            WHERE `id_quicklinks` = '.(int)$moved_quicklinks[0]['id_quicklinks']);

数据库调用是昂贵的操作，因此将它们降至最低是必须的。与PHP代码一样，如果第一个查询失败，则不会执行第二个查询。

以下是我的ajaxProcessUpdatePositions()的样子：

public function ajaxProcessUpdatePositions()
{
    $id = (int) Tools::getValue('id');
    $positions = Tools::getValue("priority_level");
    $way = (boolean) Tools::getValue('way');

    //"and $id" because an ID of 0 is invalid (at least if you are using an auto-increment PK)
    if (is_array($positions) and $id)  
    {
        foreach ($positions as $position => $value)
        {
            $pos = explode('_', $value);

            if (isset($pos[2]) && (int) $pos[2] === $id)
            {
                $position = (int) $position;

                if ($priorityLevel = Db::getInstance()->executeS(
                        'SELECT `id_priority_level`, 
                            `priority_level`
                        FROM `pc_job_priority`
                        WHERE `id_priority_level` = ' . $id . '
                        LIMIT 1'
                )) {
                    Db::getInstance()->execute(
                        'UPDATE `pc_job_priority`
                        SET `priority_level` = `priority_level` '. 
                            ($way ? '- 1' : '+ 1') . '
                        WHERE `priority_level`' . (
                            $way ? 
                            ' > '. (int) $priorityLevel[0]['priority_level'] . 
                                ' AND `priority_level` <= '. $position :
                            ' < '. (int) $priorityLevel[0]['priority_level'] . 
                                ' AND `priority_level` >= '. $position 
                        ) . '; 
                        UPDATE `pc_job_priority`
                        SET `priority_level` = ' . $position . '
                        WHERE `id_priority_level` = ' . $id
                    );
                }

                break;
            }
        }
    }
}

Prestashop自定义管理模块可拖动排序/订单无法正常工作

2 个答案: