Yii2 Gridview error

Question

I want to display column from another table in gridview.

In my controller actionIndex:

public function actionIndex()
{     
    $user_id = \Yii::$app->user->identity->id;
    $queryFile = PowerGen::find();
    $queryFile ->select(['submitted_by','filename.file_name','filename.submitted_by'])
        ->joinWith('filename')//Tells Yii to use the complains relation that we define below. By default it is an inner join
        ->where(['filename.submitted_by' => $this->user_id]);

    $dataProvider= new ActiveDataProvider([
        'query' => $query,
        'pagination' => false,
    ]);

    return $this->render('index', ['dataProvider4'=>$dataProvider]);
}

In my model:

public function fileName()
{
    $user_id = \Yii::$app->user->identity->id;
    return $this->hasMany(Filename::className(), 'submitted_by' => $this->user_id);
}

Error is:

Error
PHP Parse Error – yii\base\ErrorException
syntax error, unexpected '=>' (T_DOUBLE_ARROW)

What is the error in the line..

Thank you

score 1 · Answer 1 · answered Jul 03 '18 at 04:03

1

Use hasMany() like

public function fileName()
{
    $user_id = \Yii::$app->user->identity->id;
    return $this->hasMany(Filename::className(), ['submitted_by' => $this->user_id]);
}

answered Jul 03 '18 at 04:03

vishuB

4,173
5
31
49

Oh, i see you added brackets. let me try – Mr Second Jul 03 '18 at 05:23
for "$this->user_id", how can I tag the logged user? – Mr Second Jul 03 '18 at 05:30
Using `$this->user_id` in relation definition is incorrect. – rob006 Jul 03 '18 at 18:41

score 1 · Answer 2 · answered Jul 03 '18 at 16:21

First i think your function must be like this:

public function fileName()
{
    return $this->hasMany(Filename::className(), ['submitted_by' => 'user_id']);
}

And your query like this:

$user_id = \Yii::$app->user->identity->id;
$queryFile = PowerGen::find();
$queryFile ->select(['submitted_by','filename.file_name','filename.submitted_by'])
                ->joinWith('filename')
                ->where(['filename.submitted_by' => $user_id]);

You are declaring the variable $user_id

$user_id = \Yii::$app->user->identity->id;

but you are not using it anywhere.

I need to display only the files submitted by the logged user. — Mr Second, Jul 05 '18 at 08:02

score 0 · Answer 3 · answered Jul 03 '18 at 18:41

0

You're defining your relation in a wrong way:

Method name should be prefixed by get, like getFileName().
Second argument of hasMany() should be array with map of column names. You should not use dynamic values here or model attributes.
You should not use Yii::$app->user->identity->id or POST/GET data in your model. It breaks MVC pattern and will create problems if you will try to use this relation in console for example.

public function getFileName() {
    return $this->hasMany(Filename::className(), ['submitted_by' => 'user_id']);
}

answered Jul 03 '18 at 18:41

rob006

21,383
5
53
74

I need to display the files submitted by the logged user. – Mr Second Jul 05 '18 at 08:02
Then you should add this condition in controller. Model should not be responsible for such magic. – rob006 Jul 05 '18 at 08:32

Yii2 Gridview error

3 Answers3