自定义组件不会返回值

Question

好，这是情况。 在我的beforeSave函数中，我想操作一些$this->request->data条目。

这是我的组件：

<?php
App::uses('Component', 'Controller');
class GetStationComponent extends Component {
    public function getStationId ($station) { 
        $stationInstance = ClassRegistry::init('Station');

        $conditions = array('OR' => array(
            array('Station.code LIKE' => $station),
            array('Station.naam LIKE' => $station),
            array('Station.naam_overig LIKE' => $station)
            ));

        $result = $stationInstance->find('list', array('conditions'=>$conditions));

        $value = array_values($result);
        $value = $value[0];

        return $value;
    }
}
?>

这是我的Controller中的beforeSave函数：

public function beforeSave($options = array()) {
        if (!empty($this->request->data['Experience']['vertrekstation']) && !empty($this->request->data['Experience']['aankomststation'])) {
            $this->request->data['Experience']['vertrekstation'] = $this->GetStation->getStationId($this->request->data['Experience']['vertrekstation']);
            $this->request->data['Experience']['aankomststation'] = $this->GetStation->getStationId($this->request->data['Experience']['aankomststation']);
        }
        return true;
    }

它应该返回站名称的ID。 但是，名称本身被存储（由用户填写），而不是ID。 我需要在组件中进行哪些更改（我想...）以返回正确的值？

（PS组件中的查询本身返回一个ID，因为起初我将'beforeSave'直接放入保存数据的函数中，但是随后我的验证错误说这不是一个正确的值。这是正确的...）

Answer 1

补充其他答案； 要获取单个字段的值，请使用Model::field()

return $stationInstance->field('id', $conditions);

最好在此语句中添加排序顺序，以确保始终以相同的顺序返回结果：

return $stationInstance->field('id', $conditions, 'code ASC');

由于您只对Model执行一个查询，此后不执行任何操作，因此您甚至不需要中间的$stationInstance变量。 您的代码可以进一步简化为：

return ClassRegistry::init('Station')->field('id', $conditions, 'code ASC');

一些观察

• 由于工作站名称上的“模糊”匹配，第一个结果可能并不总是用户想要的工作站，因此最好在前端提供“自动完成”功能并让用户选择正确的工作站（例如为了当用户表示 Den Haag HS时防止选择Den Haag ）
• 如果站点与站点完全不匹配，则应显示一条警告，提示找不到该站点
• 您没有在LIKE查询中用% LIKE搜索词。 如果要搜索“名称”包含“ ，则应使用'%' . $station . '%' '%' . $station . '%' '%' . $station . '%' 。 对于“开头为”，请使用$station . '%' $station . '%'
• 如@mark建议; beforeSave()是Model的回调，应该位于那里。 也; 验证完成后会触发beforeSave() ，因此可能为时已晚。 beforeValidate()是对此的最佳回调
• 如果“ Experience模型已经连接到“ Station模型，则无需使用组件，因为可以直接访问“ Station模型。 最好将搜索方法放入Station模型中。

全部移到正确的（*）位置

*）其他选项总是可能的，这只是一种可能的方法

在站点模型中添加“搜索”方法；

app / Model / Station.php

public function getStationIdByName($name)
{
    $name = trim($name);

    if (empty($name)) {
        return null;
    }

    $name = '%' . $name . '%';

    $conditions = array(
        'OR' => array(
            array($this->alias . '.code LIKE' => $name),
            array($this->alias . '.naam LIKE' => $name),
            array($this->alias . '.naam_overig LIKE' => $name),
        )
    );

    return $this->field('id', $conditions, 'code ASC');
}

..并将其用于体验模型

app / Model / Experience.php

public function beforeValidate(array $options = array())
{
    if (
        !empty($this->data[$this->alias]['vertrekstation'])
        && !empty($this->data[$this->alias]['aankomststation'])
    ) {
        // Directly access the Station-model from within the Experience Model
        $this->data[$this->alias]['vertrekstation'] 
            = $this->Station->getStationIdByName($this->data[$this->alias]['vertrekstation']);
        $this->data[$this->alias]['aankomststation']
            = $this->Station->getStationIdByName($this->data[$this->alias]['aankomststation']);
    }

    // Call parent-callback after setting the values
    return parent::beforeValidate($options);
}

[更新]使用约定，防止不良行为

在写完前面的示例之后，我注意到您当前的设置中存在一些缺陷。

1. 如果vertrekstation和aankomststation应该持有站点的“外键”（站点ID ），则不会根据CakePHP模型和数据库约定对它们进行命名
2. 由于1)通过将此代码放入beforeValidate() ，在更新现有记录时也会触发该代码。 因为您同时使用aankomststation和vertrekstation字段来保存工作站的名称 （在Form内）和id （在数据库内部），所以模型在更新时将尝试通过id查找station-id。 请注意， 在表单内，您仍将使用vertrekstation和aankomstation作为字段名。 这些字段名称在数据库中不存在，因此将无法直接更新数据库中的数据，这是beforeValidate()回调用于
3. 因为Experience模型需要与Station模型建立两个关系（一次作为departure station （ 'vertrekstation' ，一次是arrival station （ 'aankomststation' ）），所以您将需要Station模型的别名 。 另请： 与同一模型的多个关系

app / Model / Experience.php

class Experience extends AppModel {
    /**
     * Station will be associated to the 'Experience' Model TWICE
     * For clarity, using an 'alias' for both associations
     *
     * The associated Models will be accessible via;
     * $this->DepartureStation
     * $this->ArrivalStation
     *
     * To stick to the CakePHP conventions, name the foreign keys
     * accordingly
     */
    public $belongsTo = array(
        'DepartureStation' => array(
            'className'  => 'Station',
            'foreignKey' => 'departure_station_id',
        ),
        'ArrivalStation' => array(
            'className'  => 'Station',
            'foreignKey' => 'arrival_station_id',
        )
    );

    public function beforeValidate(array $options = array())
    {
        // vertrekstation and aankomststation hold the 'names' of the
        // stations and will only be present if the form has been submitted
        if (
            !empty($this->data[$this->alias]['vertrekstation'])
            && !empty($this->data[$this->alias]['aankomststation'])
        ) {
            // Directly access the Station-model from within the Experience Model
            // using the *aliases*
            $this->data[$this->alias]['departure_station_id'] 
                = $this->DepartureStation->getStationIdByName($this->data[$this->alias]['vertrekstation']);
            $this->data[$this->alias]['arrival_station_id']
                = $this->ArrivalStation->getStationIdByName($this->data[$this->alias]['aankomststation']);

            // Invalidate the vertrekstation and aankomststation fields if lookup failed
            if (empty($this->data[$this->alias]['departure_station_id'])) {
                // Unable to find a station. Mark the Form-field invalid
                $this->invalidate('vertrekstation', __('A station with this name was not found'));
            }
            if (empty($this->data[$this->alias]['arrival_station_id'])) {
                // Unable to find a station. Mark the Form-field invalid
                $this->invalidate('aankomststation', __('A station with this name was not found'));
            }

        }

        // Call parent-callback after setting the values
        return parent::beforeValidate($options);
    }

}

Answer 2

Cake的find（'list'）选项返回一个类似的数组

array( 1 => 'name1',
       3 => 'name2',
       //etc...
)

其中索引是id，值是您在模型上设置的显示字段 。 因此，当您执行$value = array_values($result); ，您正在提取数组的值（即显示字段）。 我假设您没有将id用作displayField，所以这就是为什么它返回名称而不是id的原因。

我不确定为什么要使用find（'list'）而不是find（'first'）或其他替代方法，但是如果您出于某种原因不想修改它，则应该返回第一个id的修复程序通过搜索获得的是

reset($result);  //possibly not needed, but just in case
$value = key($result );

Answer 3

首先，您必须了解Cake的工作原理。模型中没有$this->request 。 它是控制器的一部分。 在您的模型中，您传递的数据将直接在$this->data 。

public function beforeSave($options = array()) {
    parent::beforeSave($options); // you also forgot the parent call

    if (!empty($this->data[$this->alias]['vertrekstation']) && ...)) {
        $this->data[$this->alias]['vertrekstation'] = ...;
    }
    return true;
}

您的查找电话看起来也搞砸了。 我不知道你想做什么。 但是我强烈建议您使用debug（）等来找出返回的内容并相应地更正您的代码。 如果您只对单个值感兴趣，则可能需要find（first）。