cakePHP3でユーザ情報の変更ページの実装

Question

authコンポーネントを利用して、ログインシステムを作っていますが、
ユーザ情報ページでニックネームやパスワード変更などの仕組みを作ろうとしていますが、
意外とよくわからず苦戦しております。

Authorizedメソッドをオーバーライドするとかいろいろ書いてありますがどうなのでしょうか。
Entityの使いかたも気になっています。

まず、アルゴリズム的な確認なのですが、sessionからidを受け取って、それ元にデータベースに接続して、その他の
情報を読み込み、inputタグなどのvalue情報に埋め込んでいくという形でよいでしょうか？

次のプログラムでは何がいけないのか教えてください。
controller

Usersクラス{

  public function settings(){
    $id = $this->request->session()->read('Auth.User.id');
    $user = $this->Users->find('all')->where(['id' => $id]);
    $this->set(compact('user'));
  }
}

settings.ctpの方

<?= h($user->id); ?>
<?= h($user->nickname); ?>

エラーは次のようにでます。

Notice (8): Undefined property: Cake\ORM\Query::$id [APP/Template/Users/settings.ctp, line 24]

宜しくお願い致します。

Answer 1

APP/Template/Users/settings.ctpの24行目にある$idが未定義らしいです。
24行目が質問文にあるのかは分かりませんが、未定義かどうか怪しいところはissetでチェックしてやるといいと思います。

Comments

[yajin]

issetでチェックしてみたところされていませんと出てきました。
なぜされていないのでしょうか。

[YashaWedyue]

24行目は質問に載っている箇所ですか？
されていませんというのがよく分かりません。

[yajin]

言葉足らずで申し訳ございません。
issetすると定義されていませんとのことです。
if(isset($user->id))あるいはif(isset($user->nickname))とするとfalseが返ってきますということです。

[YashaWedyue]

定義されていない理由は定義していないからです。
$user配列にデータを入れる処理ができていないんだと思います。

[yajin]

ありがとうございます。
コントローラ側のせいだなということはわかっておりますが、コントローラの書き方がいまいちよくわかりません。上記のコントローラの内容ではなぜいけないのでしょうか。

[YashaWedyue]

APP/Template/Users/settings.ctpの24行目を見せていただかないとなんとも言えないのです・・・
$userを宣言している部分と、APP/Template/Users/settings.ctpの24行目を質問に追記していただけますか？

[yajin]

恐れ入ります。

24行目は
<?= h($user->id); ?>
<?= h($user->nickname); ?>
です。

$userを宣言している部分

$user = $this->Users->find('all')->where(['id' => $id]);
$this->set(compact('user'));

この2行だけではだめでしょうか。

[YashaWedyue]

$this->Users->find('all')->where(['id' => $id]);だと
$user['id']の形になってないみたいですね。
$userを宣言した後に$this->log($user);を追加してログを見てみてください。

[yajin]

2016-07-11 14:44:54 Error: SELECT Users.id AS `Users__id`,
Users.email AS `Users__email`,
Users.ukey AS `Users__ukey`,
Users.nickname AS `Users__nickname`,
Users.password AS `Users__password`,
Users.status AS `Users__status`,
Users.created AS `Users__created`,
Users.modefied AS `Users__modefied` FROM users Users WHERE id = :c0

logsのerror.logをみたところこのような形となっておりました。

[YashaWedyue]

$user = $this->Users->find('all')->where(['id' => $id]);を
$user = $this->Users->find()->where(['id' => $id])->all();
に変えてもう一度ログを見てみてください。

[yajin]

今度は、確かにカラムの内容が入ってきました。

2016-07-11 15:03:49 Error: [{"id":1,"email":"[e-mail]",
"ukey":"ランダムな文字列",
"nickname":"hogehoge",
"password":"ハッシュ化されたもの",
"status":1,"created":"2016-06-28T09:08:37+0900","modefied":null}]

でもなぜエラーログに吐かれているのでしょうか。

[YashaWedyue]

んー
<?= h($user['id']); ?>とか
<?= h($user[0]['id']); ?>を試してみていただけますか？

[yajin]

Error: Cannot use object of type Cake\ORM\ResultSet as array
File /var/www/html/cakephp/projectname/src/Template/Users/settings.ctp
Line: 21

ですね。オブジェクト型じゃないと利用できないみたいです。

[coba-coba]

横からすみません。

場当たり的に試すよりは、debug($this->request->session()->read()); debug($user);で変数の中身を確認した方がいいと思いますよ。

あとSQLはforeachやall()・toArray()メソッドで実行されるため、それらによって実行結果を得てからでないと表示できません。

[yajin]

debug($this->request->session()->read());
は以下のようになります。
[
'Config' => [
'time' => (int) 1468282582
],
'Auth' => [
'User' => [
'id' => (int) 1,
'email' => '[e-mail]',
'ukey' => 'ランダムなキー',
'nickname' => 'hogehoge',
'status' => (int) 1,
'created' => object(Cake\I18n\FrozenTime) {

'time' => '2016-06-28T09:08:37+09:00',
'timezone' => 'Asia/Tokyo',
'fixedNowTime' => false

},
'modefied' => null
]
],