多言語対応のLocale追加やページ設定でサイト名を上書きできるようにしました。#2306

nagasheep · 2025-11-22T15:17:02Z

ロケール（Locale）が jp と en のみだったものを国ごとの方言も含めて増やしました。
詳しくは \app\Enums\ConnectLocale.php を確認してください。
多言語設定が有効な際、html の head タグ内に link rel="alternate" で対応する多言語ページのURLを出力するようにしました。
ページ設定でサイト名を上書きできるようにしました。

概要

同じサイトに他の言語でのページを作りたかったため。
以下のサイトが参考になると思います。
https://tatsumi-shinonome.jp/multi-lunguage

レビュー完了希望日

参考

DB変更の有無

有り

チェックリスト

プルリクエストにわかりやすいタイトルとラベルを付けました。https://github.com/opensource-workshop/connect-cms/wiki/Pull-requests-Rule

・ロケール（Locale）が jp と en のみだったものを国ごとの方言も含めて増やしました。詳しくは \app\Enums\ConnectLocale.php を確認してください。・多言語設定が有効な際、html の head タグ内に link rel="alternate" で対応する多言語ページのURLを出力するようにしました。・ページ設定でサイト名を上書きできるようにしました。

masaton0216

お疲れ様です。
ご対応ありがとうございます。
デバッグのトコ、あまり重要ではないのですが、クエリの部分がパフォーマンス観点で気になっています。

masaton0216 · 2025-11-23T01:57:18Z

app/Http/Middleware/MultiLanguage.php

+        // 言語ルートのデバッグ表示
+        //echo $language_root . "<br />";
+        //echo strpos($page['permanent_link'], '/', 1) . "<br />";


デバッグコードとして必要であればLog::debug()に置き換えるのはどうでしょう？

masaton0216 · 2025-11-23T01:57:44Z

app/Http/Middleware/MultiLanguage.php

+        // 言語ルートをキーにして多言語設定を絞り込み
+        $configs = $request->attributes->get('configs');
+        $configs_multilanguage = $configs->where('category', 'language')->where('additional1', $language_root);
+        //print_r($configs_multilanguage);


デバッグコードとして必要であればLog::debug()に置き換えるのはどうでしょう？

同上です。

masaton0216 · 2025-11-23T01:57:52Z

app/Http/Middleware/MultiLanguage.php

+            $alternates[str_replace('/', '', $page->additional1)] = $page->permanent_link;
+        }
+        $request->attributes->add(['alternates' => $alternates]);
+        //print_r($alternates);


デバッグコードとして必要であればLog::debug()に置き換えるのはどうでしょう？

同上です。

masaton0216 · 2025-11-23T02:42:54Z

app/Http/Middleware/MultiLanguage.php

+        $multilanguage_pages = Page::join('configs', function ($join) use($no_lang_path) {
+            $join->on('configs.category', '=', DB::raw("'language'"))
+                 ->on('pages.permanent_link', 'LIKE', DB::raw("CONCAT(configs.additional1, '" . $no_lang_path . "')" ));
+        })
+        ->select('pages.*', 'configs.additional1')
+        ->get();


懸念

LIKE検索にconcat()組み合わせるとインデックスが効かない（フルスキャンになる）

言語設定数 * ページ数の組み合わせで走査範囲が広がる

当該処理は毎リクエスト実行される

という観点から、パフォーマンス懸念があります。
JOINを廃止して、WHERE句での完全一致検索に変更する案はどうでしょう？
※但し、言語数分だけクエリが実行されます。
（通常は数十もの言語を１つのサイトに設定することはない（２～せいぜい５ぐらい？）という想定が前提です。これに限らない場合はもう少し要件踏まえて調整してもいいかもしれません。）

調整コード

// Configsテーブルから全言語設定を取得 ※後で文字列結合しやすくする $language_roots = $configs->where('category', 'language') ->pluck('additional1') // additional1カラムのみ抽出 ※イメージ）['/', '/en-GB', '/es-ES'] ->map(function($root) { // 前後の "/" を除去 ※イメージ）['', 'en-GB', 'es-ES'] return trim($root, '/'); }) ->filter() // 空文字を除外 ※イメージ）['en-GB', 'es-ES'] ->values() // インデックスを振り直し ※イメージ）[0 => 'en-GB', 1 => 'es-ES'] ->toArray(); // 配列に変換 ※イメージ）['en-GB', 'es-ES'] if (empty($language_roots)) { return $next($request); } // 検索結果を格納するコレクション $multilanguage_pages = collect(); // 各言語ルートに対してループ foreach ($language_roots as $root) { // 検索対象のパスを構築 $search_path = '/' . $root . $no_lang_path; // 完全一致検索（インデックスが効く） $page_found = Page::where('permanent_link', $search_path)->first(); if ($page_found) { // 言語ルート情報を追加 $page_found->language_root = '/' . $root; // コレクションに追加 $multilanguage_pages->push($page_found); } } // 多言語のページ一覧を画面に渡すためにリクエストに設定 $alternates = []; foreach($multilanguage_pages as $page_obj) { $locale = trim($page_obj->language_root, '/'); if (empty($locale)) { // デフォルト言語の場合、設定から取得 $default_locale = $configs->where('category', 'language') ->where('additional1', '/') ->first(); $locale = $default_locale ? $default_locale->value : 'ja'; } $alternates[$locale] = $page_obj->permanent_link; } $request->attributes->add(['alternates' => $alternates]); return $next($request);

サンキューです。
LIKEやめたいですね。
設定言語数が多いときにクエリをたくさん発行するのも微妙。とかも思うので、見直したところ、
wherein で取得できるかなと思えてきたので、見直してみます。

修正してみました。
LIKEはやめて、whereInで指定できるように考えました。
これで、LIKEによるSQLクエリと、言語数ループする処理のパフォーマンス問題はクリアできたかなと思います。
ついでに処理全体もわかりやすくしようと思って修正しています。

masaton0216

ご対応ありがとうございます。
もう少しだけお付き合いください。

masaton0216 · 2025-11-27T10:54:32Z

app/Http/Middleware/MultiLanguage.php

+        // \Log::debug(strpos($page['permanent_link'], '/', 1));
+
+        // 言語ルートをキーにして多言語設定を絞り込み（現在のページが多言語設定の対象か判断するため）
+        $configs = $request->attributes->get('configs');


34行目と重複しているので、こちらの行は削除してよさそうです。

masaton0216 · 2025-11-27T10:59:10Z

app/Http/Middleware/MultiLanguage.php

+        // \Log::debug(print_r($multilanguage_ins, true));
+
+        // Pageテーブルから現ページと同じ意味を持つ多言語ページのpermanent_linkを取得する。
+        $multilanguage_pages = Page::select('pages.*')


pagesテーブルは行列それなりに多いテーブルなので、取得する列数絞るとデータ転送＆メモリ量は減らせそうです。

$multilanguage_pages = Page::select('permanent_link')

masaton0216 · 2025-11-27T11:20:44Z

app/Http/Middleware/MultiLanguage.php

+        // 言語ルートをキーにして多言語設定を絞り込み（現在のページが多言語設定の対象か判断するため）
+        $configs = $request->attributes->get('configs');
+        $configs_multilanguage = $configs->where('category', 'language')->where('additional1', $language_root);
+        // \Log::debug(print_r($configs_multilanguage, true));


伝え方が曖昧ですいません。
意図はコメントアウト外してデバッグコードとして残す、という意味です。
結論コードは下記です。.envでdebugレベルを変更すれば適宜、.logファイルに出力されます。

\Log::debug($configs_multilanguage);

失礼。
バグがあった際などは、改めて考えると思うので、このデバッグコードはそこまで残しておくものではないです。
ということで、今回のデバッグコードは削除しますね。

masaton0216 · 2025-11-27T11:21:11Z

app/Http/Middleware/MultiLanguage.php

+
+        // 設定されている各言語のルートパスをconfigから絞り込む
+        $configs_multilanguage_roots = $configs->where('category', 'language');
+        // \Log::debug(print_r($configs_multilanguage_roots, true));


意図はコメントアウト外してデバッグコードとして残す、という意味です。

同上です。

masaton0216 · 2025-11-27T11:21:20Z

app/Http/Middleware/MultiLanguage.php

+            // '//'の変換は、configの多言語設定URLでトレイリングスラッシュのアリ/ナシをどちらも許容しているため。
+            $multilanguage_ins[str_replace('/', '', $configs_multilanguage_root->additional1)] = str_replace('//', '/', $multilanguage_in);
+        }
+        // \Log::debug(print_r($multilanguage_ins, true));


意図はコメントアウト外してデバッグコードとして残す、という意味です。

同上です。

masaton0216 · 2025-11-27T11:21:28Z

app/Http/Middleware/MultiLanguage.php

+            $alternates[$this->getLocale2Path($page->permanent_link)] = $page->permanent_link;
+        }
+        $request->attributes->add(['alternates' => $alternates]);
+        // \Log::debug(print_r($alternates, true));


意図はコメントアウト外してデバッグコードとして残す、という意味です。

同上です。

nagasheep · 2025-11-27T12:47:45Z

こちらこそ、付き合っていただき、感謝感謝です。
すぐ内容を検討して作業したいのですが、基本的にプログラミングが夜しかできないので（笑）、今日はもうギブアップかも。
時間見つけて続きしますので、引き続きよろしくお願いいたします。

nagasheep · 2025-11-27T22:34:20Z

修正しましたー。

masaton0216 · 2025-11-30T11:23:39Z

対応お疲れ様でした！

masaton0216 reviewed Nov 23, 2025

View reviewed changes

nagasheep added 2 commits November 24, 2025 10:39

LIKE検索をwhereInに変更した他、処理を見直しました。

be650cc

phpcsのチェックがあったので修正しました。

3939e38

masaton0216 reviewed Nov 27, 2025

View reviewed changes

処理効率化及び不要な部分を削除しました。

036bae4

masaton0216 approved these changes Nov 30, 2025

View reviewed changes

masaton0216 assigned nagasheep Nov 30, 2025

masaton0216 added the enhancement 機能強化 label Nov 30, 2025

masaton0216 merged commit fc389f4 into master Nov 30, 2025
1 check passed

masaton0216 deleted the Multi-Language_rel=alternate branch November 30, 2025 11:23

nagasheep restored the Multi-Language_rel=alternate branch January 4, 2026 05:22

nagasheep mentioned this pull request Jan 4, 2026

多言語設定の修正で、HTMLヘッドエリアのTITLEタグの対応が漏れていたことへの修正です。 #2333

Open

Uh oh!

Conversation

nagasheep commented Nov 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

概要

レビュー完了希望日

関連Pull requests/Issues

参考

DB変更の有無

チェックリスト

Uh oh!

masaton0216 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

懸念

調整コード

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

masaton0216 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nagasheep commented Nov 27, 2025

Uh oh!

nagasheep commented Nov 27, 2025

Uh oh!

Uh oh!

masaton0216 commented Nov 30, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

nagasheep commented Nov 22, 2025 •

edited

Loading