laravel topics indexページを画像付きで表示したい（1：多）

Question

開発環境：Laravel5.5 , MySQL5.7
にて実装しています。

トピック（新着情報）のリストページを作成しているのですが、
簡単なことでエラーとなりました。

トピック情報を掲載するのに、
データベース テーブルは2つあります。

topics テーブル
id , title , content , updated_at

topic_images テーブル
id , topic_is(外部キー), img_at , updated_at

です。
このテーブルは、
topics : topic_images = 1 : 多
となります。

Laravel クエリービルダーでこの情報をコントローラで読み込み際に
$topic_datas = DB::table('topics')
->join('topic_images', 'topics.id', '=', 'topic_images.topic_id')
->select('topics.id', 'topics.title', 'topics.content', 'topic_imags.img_at')
->groupBy('topics.id')
->get()

と実装しますと、当然かもしれませんが img_at が複数あるために どれが選択して良いか不明となるためエラーになってしまいます。

SELECT list is not in GROUP BY clause and contains nonaggregated column ‘（フィールド名）’ which is not functionally dependent on columns in GROUP BY clause

img_at の選択は、topic_images.id の 1番若い番号を選出する様にしたいのですが、
どのようなクエリー分になるか、わかりますでしょうか？

単純な質問で申し訳ありません。
不足情報は記述しますので、どなたかアドバイスをお願いしたいです。

Answer 1

DBファサードの代わりにEloquentを使っても良いのではないかと思いました。
その場合は、以下のような感じにするのはいかがでしょうか。

Topicモデル

namespace App;

use Illuminate\Database\Eloquent\Model;

class Topic extends Model
{
    public function topicImages()
    {
        return $this->hasMany(TopicImage::class);
    }
}

参考: １対多

TopicImageモデル

namespace App;

use Illuminate\Database\Eloquent\Model;

class TopicImage extends Model
{
    //
}

コントローラのアクション

    public function index()
    {
        // Topicを取得して、topicImagesを遅延ロードする
        $topics = \App\Topic::with(['topicImages' => function ($query) {
            $query->orderBy('id', 'desc'); // idが若い順に取得
        }])->get();

        return view(ビュー名, compact('topics'));
    }

参考: Eagerロードへの制約

Bladeビュー

    <table class="table">
        ...
        <tbody>
            @foreach($topics as $topic)
                <tr>
                    <td>{{ $topic->id }}</td>
                    <td>{{ $topic->title }}</td>
                    <td>{{ $topic->content }}</td>
                    <td>
                        {{-- topicImagesを取得して最初のものを選出して表示 --}}
                        <img
                            src="{{ $topic->topicImages->first()->img_at }}"
                            alt="{{ $topic->topicImages->first()->id }}"
                        >
                    </td>
                </tr>
            @endforeach
        </tbody>
    </table>

参考: first()

Comments

[Suzuki-Yamato]

ご支援ありがとうございます。

実は、今回 Laravel-Admin というライブラリーを採用しております。
（前担当者が実装していました。）

Eloquent で実装すると、なぜか エラーになる
（どうも Roleの箇所が不適切の様でエラーの対策ができずに クエリービルダーで対応していました。）

やはりEloquent で実装できる方法が良いのかもしれませんね。

アドバイス適切であります。ご連絡ありがとうございます。

[Suzuki-Yamato]

protected static function boot()
{
parent::boot();

static::addGlobalScope('owner_id', function (Builder $builder) {
if (Admin::user()->inRoles(['super_admin', 'system_admin'])) {
$builder->whereNotNull('id')->orderBy('updated_at', 'desc');
}
});
}

Laravel-Admin （管理画面からの投稿）でこのグルーバルスコープにてラーになってしまいます。

Call to a member function inRoles() on null

良きアドバイスがありましたら助かります。

[Lulucom]

Admin::user() の戻り値がnullになっているようですね。何かお心当たりはありますか？

[Suzuki-Yamato]

正しく、そこであります。
管理画面（Auth認証で、ログイン後 Topicsモデルを編集するために、この関数があると思いますが、
viewで表示させるためでしたら、ここは不要なはずなのですが、
ここを、どの様に編集して良いか。。。実は困っていたところです。

管理画面のコントロールでは、
$grid = new Grid(new Topic);
と使用されていますが、実は Laravel-Admin の仕様も不明で困っていました。

そのため、Eloquentを使用せず、クエリービルダーで対応していたところです。

深みにハマる質問ですみません。

[Lulucom]

私もLaravel-Adminは殆ど知見がないのですが、
・先ほどのbootメソッドは何のモデルのものでしょうか
・Admin::user()のAdminとはどのようなクラスでしょうか

[Suzuki-Yamato]

ご連絡ありがとうございます。

Topicモデルになります。
全文を記載させていただきます。

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Database\Eloquent\Builder;
use Encore\Admin\Facades\Admin;

class Topic extends Model
{
/**
* モデルの「初期起動」メソッド
*
* @return void
*/
protected static function boot()
{
parent::boot();

static::addGlobalScope('owner_id', function (Builder $builder) {
if (Admin::user()->inRoles(['super_admin', 'system_admin'])) {
$builder->whereNotNull('id')->orderBy('updated_at', 'desc');
}
});
}

/**
* ユーザーに関連する情報を取得
*/
public function user()
{
return $this->hasOne(config('admin.database.users_model'), 'id', 'owner_id');
}

public function topic_category()
{
return $this->belongsTo(TopicCategory::class);
}

public function topic_images()
{
return $this->hasMany(TopicImage::class,'topic_id');
}
}

もし一般的にviewへの適用の際には、無効にするような方法があれば良いのですが、、、

[Lulucom]

Admin::user()は、認証済みの管理者ユーザーがアクセスしてきたときだけ、管理者ユーザーのインスタンスが返されるのかもしれないですね。モデルの中で認証状況を判定するのってどうなのかな？という疑問はありますが、その部分でoptional()を使って以下のようにしてみるとどうでしょうか。
if (optional(Admin::user())->inRoles([ ...
参考: https://readouble.com/laravel/5.5/ja/helpers.html#method-optional

[Suzuki-Yamato]

なるほど！
やってみます。

おっしゃる通りで、前任者が
モデルの中で認証状況をした理由がわからず、どの様に変更すれば良いかが困っていたところです。

[Lulucom]

または、そこはそのままにしておき、代わりに、Laravel-admin以外でTopicを取得するときには必ず withoutGlobalScope('owner_id') を呼ぶようにしても良いと思います。
例:
$topics = \App\Topic::withoutGlobalScope('owner_id')
->with(['topicImages' => function ($query) {
$query->orderBy('id', 'desc');
}])
->get();

[Lulucom]

または、それをグローバルスコープにするのではなく、ローカルスコープにしてLaravel-adminでTopicを取得するときだけ適用するのが良いと思います。

[Suzuki-Yamato]

なるほどです！！！
やってみます。