Skip to content

Conversation

akagane99
Copy link
Contributor

@akagane99 akagane99 commented Jun 21, 2023

概要

  • エラー内容
    • Docker+php8.0でエラー発生しました。(Xampp+php8.0ではエラー発生しませんでした)
      • Xampp+php7.3で再現しました。php8.0系に依存するエラーじゃなかったです。
    • この影響で、管理画面系bladeのみ、php8対応でエラーになっていても、エラーが画面表示されず、正常表示されます。
      • (エラーログには出力されていました)
      • 修正前+php8.0+一般プラグインは、ちゃんとエラー画面表示されました。
  • 対応
  • 対応後
    • 管理画面系bladeでも、エラー画面が表示され、落ちます。 そのため、管理画面系に修正が必要です。
    • 対応済み画面
      • サイト管理>WYSIWYG設定 e29e508
      • サイト管理>meta情報 5dbe54a
      • サイト管理>エラー設定
      • サイト管理>アクセス解析
      • サイト管理>サイト設計書
      • ページ管理>ページ権限一覧
      • ユーザ管理 (所属型を設定時にTrying to get property 'name' of non-object エラー対応)
      • ユーザ管理>一括削除
      • メッセージ管理
      • 外部認証>NetCommons2認証
      • 施設管理>施設登録
      • 施設管理>項目セット登録
      • テーマ管理>カスタムテーマ生成
      • コード管理>表示設定
      • コード管理>検索条件登録
      • コード管理>注釈一覧
      • コード管理>注釈登録
      • コード管理>ダウンロード
      • 祝日管理

修正前エラーログ(サイト管理>WYSIWYG設定 を開いた時)

2023-06-21 17:07:17 [Wed Jun 21 17:07:17.837054 2023] [php:warn] [pid 30] [client 172.20.0.1:49424] PHP Warning:  Invalid callback App\\Plugins\\Manage\\SiteManage\\SiteManage::ccErrorHandler, cannot access protected method App\\Plugins\\Manage\\SiteManage\\SiteManage::ccErrorHandler() in /var/www/html/connect-cms/storage/framework/views/80aec5200fe8bc640e19e8f90fcc74671a814f37.php on line 545, referer: http://localhost/manage/site

レビュー完了希望日

なし

関連Pull requests/Issues

なし

参考

なし

DB変更の有無

なし

チェックリスト

@akagane99 akagane99 added the bug バグ・不具合連絡 label Jun 21, 2023
@akagane99 akagane99 self-assigned this Jun 21, 2023
@akagane99 akagane99 changed the title [コア] php8.0で管理画面系bladeでUndefined variableエラーが起こっても、エラーハンドリングできていなかったバグの修正 [コア] 管理画面系bladeでUndefined variableエラーが起こっても、エラーハンドリングできていなかったバグの修正 Jun 21, 2023
@akagane99 akagane99 changed the title [コア] 管理画面系bladeでUndefined variableエラーが起こっても、エラーハンドリングできていなかったバグの修正 [コア] 管理画面系bladeでUndefined variableエラーが起こっても、エラー画面が表示されないバグの修正 Jun 21, 2023
akagane99 added 20 commits June 21, 2023 21:29
@akagane99
Copy link
Contributor Author

Laravel Dusk Connect-cms-testが通る事を確認しました。
https://github.com/opensource-workshop/connect-cms/actions/runs/5340772818/jobs/9680952001

@akagane99 akagane99 merged commit 8df8fe3 into master Jun 22, 2023
@akagane99 akagane99 deleted the fix-php8-manage branch June 22, 2023 02:14
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