我的问题可能如下所示:
yii2 - model load function does not set some model attributes
然而,由于通过联结表处理多对多关系,这里的情况不同。
例如,我有三个表,Jobs
,Eqtypes
和联结表Eqtype_jobs
。我想使用简单Eqtypes
使用多个select Job
将一些activeform
与当前dropDownList
相关联。以下是我所拥有的代码,控制器和视图:
//the controller code
public function actionEqtypeCreate($id)
{
$eqtypes = \app\modules\crud\models\Eqtype::find()->all();
$model = Job::findOne(['id' => $id]);
if ($model->load(Yii::$app->request->post())){
var_dump($model->eqtypes);
die(); // for debuging <<<<<<<<<<*>>>>>>>>
foreach ($eqtypes as $eqt){
$model->eqtypes->id = $eqt->id;
$model->eqtypeJobs->eqtype_id = $eqt->eqtype_id;
$model->save();
}
return $this->redirect(['index']);
}
return $this->render('eqtype-create', ['model' => $model, 'eqtypes' => $eqtypes]);
}
这里的观点:
//The view code i.e Form
<?php
//use Yii;
use yii\helpers\Html;
use yii\widgets\ActiveForm;
use yii\helpers\ArrayHelper;
$form = ActiveForm::begin([
'enableClientValidation' => true,
]);
echo $form->field($model, 'eqtypes')->dropDownList(ArrayHelper::map($eqtypes,'id','title'), ['multiple' => true]);
echo Html::submitButton(
'<span class="glyphicon glyphicon-check"></span> ' .
($model->isNewRecord ? Yii::t('cruds', 'Create') : Yii::t('cruds', 'Save')),
[
'id' => 'save-' . $model->formName(),
'class' => 'btn btn-success'
]
);
$form->end();
我遇到了一个问题:控制器代码中var_dump($model->eqtypes)
之前的die()
,的输出始终返回空数组array(0) { }
。
的确,是什么让我尝试使用die()
进行调试,没有调试,我遇到以下错误:
间接修改重载属性 app \ modules \ crud \ models \ Job :: $ eqtypes在第149行没有效果
在我的情况下,第149行是控制器代码中foreach
语句之后的第一行:
$model->eqtypes->id = $eqt->id;
所有模型均使用yii2-giiant创建。但是,以下是工作模型的部分副本:
<?php
// This class was automatically generated by a giiant build task
// You should not change it manually as it will be overwritten on next build
namespace app\modules\crud\models\base;
use Yii;
abstract class Job extends \yii\db\ActiveRecord
{
/**
* @inheritdoc
*/
public static function tableName()
{
return 'jobs';
}
/**
* @inheritdoc
*/
public function rules()
{
return [
[['machine_id', 'product_id', 'title', 'qty'], 'required'],
[['machine_id', 'product_id', 'qty', 'updated_by' ,'created_by'], 'integer'],
[['created', 'started', 'completed'], 'safe'],
[['desc'], 'string'],
[['title', 'speed'], 'string', 'max' => 255],
[['product_id'], 'exist', 'skipOnError' => true, 'targetClass' => \app\modules\crud\models\Product::className(), 'targetAttribute' => ['product_id' => 'id']],
[['machine_id'], 'exist', 'skipOnError' => true, 'targetClass' => \app\modules\crud\models\Machine::className(), 'targetAttribute' => ['machine_id' => 'id']]
];
}
/**
* @inheritdoc
*/
public function attributeLabels()
{
return [
'id' => Yii::t('app', 'ID'),
'machine_id' => Yii::t('app', 'Machine ID'),
'created' => Yii::t('app', 'Created'),
'started' => Yii::t('app', 'Started'),
'completed' => Yii::t('app', 'Completed'),
'product_id' => Yii::t('app', 'Product ID'),
'title' => Yii::t('app', 'Title'),
'qty' => Yii::t('app', 'Qty'),
'speed' => Yii::t('app', 'Speed'),
'created_by' => Yii::t('app', 'Created By'),
'desc' => Yii::t('app', 'Desc'),
];
}
public function getEqtypeJobs()
{
return $this->hasMany(\app\modules\crud\models\EqtypeJob::className(), ['job_id' => 'id']);
}
/**
* @return \yii\db\ActiveQuery
*/
public function getEqtypes()
{
return $this->hasMany(\app\modules\crud\models\Eqtype::className(), ['id' => 'eqtype_id'])->viaTable('eqtype_jobs', ['job_id' => 'id']);
}
由于var_dump
的输出导致表单中的POST数据未能加载,或者我的代码中缺少其他内容,我确实不知道是不是问题?!
所以,我想知道,为什么load
不能按预期工作? 换句话说, var_dump($model->eqtypes)
打印合并表中的存储值而不是表单中提交的值,然后如果我们能够解决此,为什么有关间接修改的错误消息?
答案 0 :(得分:1)
您遇到的错误是“间接修改重载属性”是因为您正在尝试修改“__get”魔术方法返回的属性。看看“yii \ db \ BaseActiveRecord”的“__get”,简而言之,它首先检查你的模型是否有属性“eqtypes”,如果没有检查你的模型关系。
如您所知,您无法在帖子中加载模型中的关系,只需保存即可。 我认为在您的解决方案中,您正在超越问题并使您的代码过于复杂。
我建议你使用这个模块:https://github.com/cornernote/yii2-linkall 安装此操作并将操作更改为:
$model = Job::findOne(['id' => $id]);
if ($model->load(Yii::$app->request->post())){
$postData = Yii::$app->request->post();
$eqtypesIds = $postData['Job']['eqtypes'];
$eqtypes = \app\models\Eqtype::findAll($eqtypesIds);
$transaction = Yii::$app->db->beginTransaction();
$extraColumns = []; // extra columns to be saved to the many to many table
$unlink = true; // unlink tags not in the list
$delete = true; // delete unlinked tags
try {
$model->linkAll('eqtypes', $eqtypes, $extraColumns, $unlink, $delete);
$model->save();
$transaction->commit();
} catch (Exception $e) {
$transaction->rollBack();
}
return $this->redirect(['index']);
}
$eqtypes = \app\models\Eqtype::find()->all();
return $this->render('eqtype', ['model' => $model, 'eqtypes' => $eqtypes]);
并添加您的工作模型:
public function behaviors()
{
return [
\cornernote\linkall\LinkAllBehavior::className(),
];
}
如果您不想使用此模块,则应覆盖模型的默认load()和save()方法,以包括加载和保存关系。
答案 1 :(得分:0)
检查你的模型验证rulse和post参数,你的模型是验证ruls?
if($model->load(Yii::$app->request->post()) && $model->validate())
{
echo "<pre>";
print_r($_POST); // check the post parameter
exit;
}
else {
echo "<pre>";
print_r($model->getErrors());
exit;
}
答案 2 :(得分:0)
我找到了一个解决方案,但我认为这不是直接的解决方案,也不是更好的解决方案。它没有应用ORM或activerecords的假定好处。解决方案大纲如下:
1.直接从POST中选择多个选择下拉列表中的eqytpes id,如控制器操作中的以下内容:
public function actionEqtypeCreate($id)
{
$eqtypes = \app\modules\crud\models\Eqtype::find()->all();
$model = Job::findOne(['id' => $id]);
if ($model->load(Yii::$app->request->post())){
// <<<<<<<<<<<<<<<<<<<<<<,*>>>>>>>>>>>>>>>>>>>
// Direct Collect the IDs
$model->eqtypesIds = Yii::$app->request->post()['Job']['eqtypes'];
// <<<<<<<<<<<<<<<<<<<<<*>>>>>>>>>>>>>>>>
$model->save();
return $this->redirect(['index']);
}
return $this->render('eqtype-create', ['model' => $model, 'eqtypes' => $eqtypes]);
}
在上一步中,我必须向名为eqtypesIds
的作业模型添加新属性,以便临时存储ids。
在作业模型中,我必须使用afterSave
回调在结合表eqtypes
中执行保存选定的eqtype_jobs
ID,如下所示:
8787
public function afterSave($insert, $changedAttributes)
{
// recal the saved values eqtype_id list to check the absence items of it from the submited and then delete the absence
$ee = $this->hatEqtypeIdsArray(); // method returns avaialble eqtype_id as array
//die(var_dump($ee));
foreach ($ee as $a){
if (!in_array($a, $this->eqtypesIds)){
$eq = \app\modules\crud\models\EqtypeJob::findOne(['job_id' => $this->id, 'eqtype_id' => $a]);
$eq->delete();
}
}
// adding the ids submitted to the conjunction table.
foreach ($this->eqtypesIds as $eqtype) {
$tag = \app\modules\crud\models\EqtypeJob::findOne(['job_id' => $this->id, 'eqtype_id' => $eqtype]);
if ($tag) {
}
else{
$tag = new \app\modules\crud\models\EqtypeJob();
$tag->job_id = $this->id;
$tag->eqtype_id = $eqtype;
$tag->save();
}
}
parent::afterSave($insert, $changedAttributes);
}
beforeSave
处理程序,如下所示:545
public function beforeSave($insert) {
parent::beforeSave($insert);
if(is_null($this->eqtypesIds)){// i.e it has no values.
$this->eqtypesIds = $this->hatEqtypeIdsArray();// get stored values for it
}....
但是,正如我上面所说,这个解决方案很脏。它可能会重复并消耗更多资源。例如,假设您只想编辑作业标题,这将导致选择相关的作业eqtypes,然后重新更新它。