Skip to content

[NC3移行] NC3からの移行処理で多言語ページのページ権限が登録されない不具合を修正#2218

Merged
masaton0216 merged 40 commits into
masterfrom
OW-2586
Jul 8, 2025
Merged

[NC3移行] NC3からの移行処理で多言語ページのページ権限が登録されない不具合を修正#2218
masaton0216 merged 40 commits into
masterfrom
OW-2586

Conversation

@masaton0216

@masaton0216 masaton0216 commented Jul 1, 2025

Copy link
Copy Markdown
Contributor

概要

NC3移行機能において、多言語ページに対してページ権限が正しく登録されるよう修正しました。
また、テストが存在しなかったのでテストを追加しています。

変更内容

本旨対応:多言語ページ権限登録の修正

d18274e

  • 多言語対応のページすべてに権限を設定するロジックを追加
    • マイグレーションマッピングテーブルへのレコード登録処理を追加('target_source_table' => 'source_pages_lang'
    • JSON形式で格納された言語情報を活用したページ権限管理
  • setPageRolesForMultiLanguagePages() メソッドを新規作成
    • マイグレーションマッピングテーブルから言語別ページを検索し、権限を設定

付随対応:テストケース追加

  • テストクラスを追加
  • 主要なメソッドに対してテスト77ケースを追加
# php artisan test tests/Unit/Migration/MigrationNc3ExportTraitTest.php
~略~
  Tests:  77 passed
  Time:   47.43s

修正の背景

  • NC3移行時に多言語サイトのページ権限が適切に移行されない問題がありました。
    • 具体的には、日英とあった場合の英語ページに権限が一切設定されない。

レビュー完了希望日

なし

関連PR/Issues

なし

参考情報

なし

DB変更の有無

なし(ロジック修正とテストコードのみ)

チェックリスト

  • 多言語ページ権限が正しく設定される
  • 既存の単一言語ページ権限に影響なし
  • テストが正常に実行される
  • コードスタイルガイドに準拠

@masaton0216 masaton0216 self-assigned this Jul 1, 2025
@masaton0216 masaton0216 added the bug バグ・不具合連絡 label Jul 1, 2025
@masaton0216 masaton0216 changed the title [マイグレーション] NC3からの移行処理で多言語対応を実装 [NC3移行] NC3からの移行処理で多言語ページのページ権限が登録されない不具合を修正 Jul 1, 2025
masaton0216 added 26 commits July 1, 2025 17:13
@masaton0216 masaton0216 merged commit e31e512 into master Jul 8, 2025
1 check passed
@masaton0216 masaton0216 deleted the OW-2586 branch July 10, 2025 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug バグ・不具合連絡

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant