Question

在我的处理器类中，我有一个语句，它从db表中获取所有项目并格式化它们以进行显示。此方法不起作用并在getCollection调用时暂停。

class GlobalLinkSettingsProcessor extends modObjectGetListProcessor{

    public function initialize() {
        return parent::initialize();
    }

    public function process() {
        $result = $this->modx->getCollection('ManagerProjects');
        $project_names = array();
        foreach ($result as $row) {
            $projects = unserialize($row->get('manager_projects'));
            foreach($projects as $short_code => $project) {
                $project_names[] = array('project_name' => $project, 'project_short_code' => $short_code);
            }
        }
        return '{"total":' . count($project_names) . ',"results":' . $this->modx->toJSON($project_names) . ',"success":true}';
    }
...
}

使用纯SQL 的代码可以正常工作：

class GlobalLinkSettingsProcessor extends modObjectGetListProcessor{

    public function initialize() {
        return parent::initialize();
    }

    public function process() {
        $leadersql = "SELECT * FROM `modx_manager_projects`";
        $query = $this->modx->query($leadersql);
        $project_names = array();
        while ($row = $query->fetch(PDO::FETCH_ASSOC)) {
            $projects = unserialize($row['manager_projects']);
            foreach($projects as $short_code => $project) {
                $project_names[] = array('project_name' => $project, 'project_short_code' => $short_code);
            }
        };
        return '{"total":' . count($project_names) . ',"results":' . $this->modx->toJSON($project_names) . ',"success":true}';
    }
...
}

我使用类似的方法保存ManagerProjects的第一个并且工作正常，所以我不认为它与模型声明有关。我可以很容易地使用上面的第二种方法，因为它似乎工作，但我想使用最好的方法。

第一种方法有什么问题？

第一种方法是在Modx处理器中实现SQL的正确方法吗？或者，还有更好的方法？

Answer 1

我们可以更轻松地完成这项任务。

@Vasis是对的但我们可以使用base prepareRow 方法而不是重新加载 iterate 方法：

<?php
class GlobalLinkSettingsProcessor extends modObjectGetListProcessor{
    public $classKey = 'ManagerProjects';
    protected $projects = array();

    public function prepareRow(xPDOObject $object) {
        $_projects = unserialize($object->get('manager_projects'));
        foreach($_projects as $short_code => $project) {
            $this->projects[] = array('project_name' => $project, 'project_short_code' => $short_code);
        }
        return parent::prepareRow($object);
    }

    public function outputArray(array $array,$count = false) {
        $count = count($this->projects);
        return parent::outputArray($this->projects,$count);
    }
}
return 'GlobalLinkSettingsProcessor';

在那里我们可以看到一个modx'特征'。在 modObjectGetListProcessor 进程方法中，我们可以看到：

public function process() {
    $beforeQuery = $this->beforeQuery();
    if ($beforeQuery !== true) {
        return $this->failure($beforeQuery);
    }
    $data = $this->getData();
    $list = $this->iterate($data);
    return $this->outputArray($list,$data['total']);
}

getData 方法返回一个对象列表，然后转到 iterate 方法（我们可以检查对象是否可访问，并根据需要更改这些对象的列表）。如果您无法访问某些对象，我们将更改列表。它转到 outputArray 方法，但第二个参数仍然是旧的。所以你应该再算一次。

这是解决方案很好但你试图获取存储在对象字段中的数据。所以 afterIteration 方法将无法在我的处理器版本中进一步扩展。但谁在乎？：）

P.S。：关于您的第一个处理器版本。 modObjectGetList 处理器已准备好进行收集。所以你不必使用 getcollection 方法。只需添加适当的 classKey 属性即可。另一种方法是 modProcessor 扩展。它为您提供了一个基础结构。但你可以制作自己的东西。

Answer 2

因为你做错了！只需看this。正确的方法就是这样：

<?php
class GlobalLinkSettingsProcessor extends modObjectGetListProcessor{
    public $classKey = 'ManagerProjects';

    public function iterate(array $data) {
        $list = array();
        $list = $this->beforeIteration($list);
        $this->currentIndex = 0;
        /** @var xPDOObject|modAccessibleObject $object */
        foreach ($data['results'] as $object) {
            if ($this->checkListPermission && $object instanceof modAccessibleObject && !$object->checkPolicy('list')) continue;
            $projects = unserialize($object->get('manager_projects'));
            foreach($projects as $short_code => $project) {
                $objectArray = array('project_name' => $project, 'project_short_code' => $short_code);
                if (!empty($objectArray) && is_array($objectArray)) {
                    $list[] = $objectArray;
                    $this->currentIndex++;
                }
            }
        }
        $list = $this->afterIteration($list);
        return $list;
    }

}

Modx：getCollection查询不起作用

2 个答案: