Merge branch '5.1.x' into fix-3819

seto1 · Nov 7, 2024 · 20cd81f · 20cd81f
2 parents 94b6bcc + 4ad5093
commit 20cd81f
Show file tree

Hide file tree

Showing 68 changed files with 986 additions and 195 deletions.
diff --git a/.gitignore b/.gitignore
@@ -131,6 +131,8 @@ node_modules
 # db
 /db/*
 !/db/.gitkeep
+# profiler
+/profiler/*
 # etc
 LOCAL_TODO.md
 LOCAL_Q&A.md
diff --git a/composer.json b/composer.json
@@ -108,23 +108,23 @@
         }
     },
     "replace": {
-        "baserproject/baser-core": "5.1.2",
-        "baserproject/bc-admin-third": "5.1.2",
-        "baserproject/bc-blog": "5.1.2",
-        "baserproject/bc-column": "5.1.2",
-        "baserproject/bc-content-link": "5.1.2",
-        "baserproject/bc-custom-content": "5.1.2",
-        "baserproject/bc-editor-template": "5.1.2",
-        "baserproject/bc-favorite": "5.1.2",
-        "baserproject/bc-front": "5.1.2",
-        "baserproject/bc-installer": "5.1.2",
-        "baserproject/bc-mail": "5.1.2",
-        "baserproject/bc-plugin-sample": "5.1.2",
-        "baserproject/bc-search-index": "5.1.2",
-        "baserproject/bc-theme-config": "5.1.2",
-        "baserproject/bc-theme-file": "5.1.2",
-        "baserproject/bc-theme-sample": "5.1.2",
-        "baserproject/bc-uploader": "5.1.2",
-        "baserproject/bc-widget-area": "5.1.2"
+        "baserproject/baser-core": "5.1.3",
+        "baserproject/bc-admin-third": "5.1.3",
+        "baserproject/bc-blog": "5.1.3",
+        "baserproject/bc-column": "5.1.3",
+        "baserproject/bc-content-link": "5.1.3",
+        "baserproject/bc-custom-content": "5.1.3",
+        "baserproject/bc-editor-template": "5.1.3",
+        "baserproject/bc-favorite": "5.1.3",
+        "baserproject/bc-front": "5.1.3",
+        "baserproject/bc-installer": "5.1.3",
+        "baserproject/bc-mail": "5.1.3",
+        "baserproject/bc-plugin-sample": "5.1.3",
+        "baserproject/bc-search-index": "5.1.3",
+        "baserproject/bc-theme-config": "5.1.3",
+        "baserproject/bc-theme-file": "5.1.3",
+        "baserproject/bc-theme-sample": "5.1.3",
+        "baserproject/bc-uploader": "5.1.3",
+        "baserproject/bc-widget-area": "5.1.3"
     }
 }
diff --git a/plugins/baser-core/VERSION.txt b/plugins/baser-core/VERSION.txt
@@ -1,4 +1,4 @@
-5.1.3
+5.1.4
 
 ///////////////////////////////////////////////////////////////////////////////////////////////////////////
 // +---------------------------------------------------------------------------------------------------+ //
@@ -14,6 +14,25 @@
 CHG: 変更内容 / BUG: バグフィックス / NEW: 機能追加
 
 
+[2024-10-24] basercms-5.1.3
+- NEW [BC]  さくらのVPSのSSLに対応
+- CHG [BC]  baserCMS4系で使用していたときのパスワードが引き継げない問題を解決
+- BUG [BC]  プラグイン一覧に開発者の表記にbrタグが入ってる問題を調整
+- BUG [BC]  初期ユーザー以外のシステム管理ユーザーは自分のユーザー情報を編集しようとすると運営ユーザーしか選択できなくなる問題を解決
+- BUG [BC]  利用可能バージョンの取得処理を調整
+- BUG [BC]  サブフォルダ環境でログイン時のリダイレクト先が間違っている問題を改善
+- BUG [BC]  固定ページをエレメントとして読み込めない問題を改善
+- BUG [BC]  メインサイトの管理画面からサブサイト記事にアクセスするとURLが正しく生成されない問題を解消
+- BUG [BC]  カスタムコンテンツ：エラーメッセージに情報不足な部分がある問題を改善
+- BUG [BC]  メインサイトでブログのエイリアスを作成すると、Blog->Posts()で呼び出された記事のURLがすべて1つ目のエイリアスのURLになってしまう問題を改善
+- BUG [BC]  SitesTable::findByUrl() の判定を調整
+- BUG [BC]  サブサイト（独自ドメイン）で作成したページがエイリアス付きサブサイトと混同される問題を改善
+- BUG [BC]  カスタムコンテンツにてファイル項目を含むエントリーをプレビュー時にエラーとなる問題を改善
+- BUG [BC]  エディタをなしに設定しても、ブログ記事の編集画面でCKEditorが選択されてしまう問題を改善
+- BUG [BC]  管理画面テーマの選択の不具合を改善 fix #3884
+- BUG [UL]  フロント側のログインユーザーに公開期間外のアップロードファイルが表示されてしまう問題を改善
+
+
 [2024-09-26] basercms-5.1.2
 - NEW [BC]  JWTキー生成コマンド追加
 - BUG [BC]  BcUploadで作成したフォルダの権限が書き込み可能になっていない問題を改善

diff --git a/plugins/baser-core/config/setting.php b/plugins/baser-core/config/setting.php
@@ -417,6 +417,16 @@
          */
         'enableRootRoutes' => false,
 
+        /**
+         * bc_formのテンプレートを指定
+         * config/bc_form.phpを差し替える         
+         *  プラグイン記法 (プラグイン名.テンプレート名)
+         */
+        //basercms/plugins/baser-core/src/View/AppView.php
+        'AppFormTemplate' => 'BaserCore.bc_form',
+        //basercms/plugins/baser-core/src/View/BcAdminAppView.php
+        'AdminFormTemplate' => 'BaserCore.bc_form',
+
         /**
          * システムナビ
          *

diff --git a/plugins/baser-core/src/BaserCorePlugin.php b/plugins/baser-core/src/BaserCorePlugin.php
@@ -151,7 +151,7 @@ public function bootstrap(PluginApplicationInterface $app): void
         }
 
         /**
-         * プラグインロード
+         * テーマ・プラグインロード
          */
         if (!filter_var(env('USE_DEBUG_KIT', true), FILTER_VALIDATE_BOOLEAN)) {
             // 明示的に指定がない場合、DebugKitは重すぎるのでデバッグモードでも利用しない
@@ -164,19 +164,19 @@ public function bootstrap(PluginApplicationInterface $app): void
         }
 
         if (BcUtil::isTest()) $app->addPlugin('CakephpFixtureFactories');
-        $app->addPlugin('Authentication');
-        $app->addPlugin('Migrations');
 
-        $this->addTheme($app);
-
-        $plugins = BcUtil::getEnablePlugins();
-        if ($plugins) {
-            foreach($plugins as $plugin) {
-                if (BcUtil::includePluginClass($plugin->name)) {
-                    $this->loadPlugin($app, $plugin->name, $plugin->priority);
-                }
-            }
-        }
+        // 利用可能なテーマを取得
+        $themes = $this->getAvailableThemes();
+        // プラグインを追加する前にテーマが保有するプラグインのパスをセット
+        $this->setupThemePlugin($themes);
+        // テーマが保有するプラグインも含めてプラグインを読み込む
+        $this->addPlugin($app);
+        // ======================================================
+        // テーマはプラグインの後に読み込む
+        // テーマもプラグインとして扱う場合があるため、
+        // その場合に、テーマでプラグインの設定等を上書きできるようにする
+        // ======================================================
+        $this->addTheme($app, $themes);
 
         /**
          * デフォルトテンプレートを設定する
@@ -194,42 +194,66 @@ public function bootstrap(PluginApplicationInterface $app): void
         $event->on(new BcAuthenticationEventListener());
     }
 
+    /**
+     * プラグインを追加する
+     * @param PluginApplicationInterface $app
+     * @return void
+     * @checked
+     * @noTodo
+     */
+    public function addPlugin(PluginApplicationInterface $app): void
+    {
+        $app->addPlugin('Authentication');
+        $app->addPlugin('Migrations');
+
+        $plugins = BcUtil::getEnablePlugins();
+        if(!$plugins) return;
+        foreach($plugins as $plugin) {
+            if (!BcUtil::includePluginClass($plugin->name)) continue;
+            $this->loadPlugin($app, $plugin->name, $plugin->priority);
+        }
+    }
+
     /**
      * テーマを追加する
      *
-     * テーマ内のプラグインも追加する
-     *
      * @param PluginApplicationInterface $application
      * @noTodo
      * @checked
      */
-    public function addTheme(PluginApplicationInterface $application)
+    public function addTheme(PluginApplicationInterface $application, array $themes): void
     {
         $application->addPlugin(Inflector::camelize(Configure::read('BcApp.coreAdminTheme'), '-'));
         $application->addPlugin(Inflector::camelize(Configure::read('BcApp.coreFrontTheme'), '-'));
         if (!BcUtil::isInstalled()) return;
-        $sitesTable = TableRegistry::getTableLocator()->get('BaserCore.Sites');
-        try {
-            $sites = $sitesTable->find()->where(['Sites.status' => true]);
-        } catch (MissingConnectionException) {
-            return;
+
+        foreach($themes as $theme) {
+            if(!BcUtil::includePluginClass($theme)) continue;
+            try {
+                $application->addPlugin($theme);
+            } catch (MissingPluginException $e) {
+                $this->log($e->getMessage());
+            }
         }
+    }
 
+    /**
+     * テーマが保有するプラグインのパスを追加する
+     * @param array $themes
+     * @return void
+     * @checked
+     * @noTodo
+     */
+    public function setupThemePlugin(array $themes): void
+    {
+        if (!BcUtil::isInstalled()) return;
+        if(!$themes) return;
         $path = [];
-        foreach($sites as $site) {
-            if ($site->theme) {
-                if(!BcUtil::includePluginClass($site->theme)) continue;
-                try {
-                    $application->addPlugin($site->theme);
-                    $pluginPath = CorePlugin::path($site->theme) . 'plugins' . DS;
-                    if (!is_dir($pluginPath)) continue;
-                    $path[] = $pluginPath;
-                } catch (MissingPluginException $e) {
-                    $this->log($e->getMessage());
-                }
-            }
+        foreach($themes as $theme) {
+            $pluginsPath = CorePlugin::path($theme) . 'plugins' . DS;
+            if (!is_dir($pluginsPath)) continue;
+            $path[] = $pluginsPath;
         }
-        // テーマプラグインを追加
         if($path) {
             Configure::write('App.paths.plugins', array_merge(
                 Configure::read('App.paths.plugins'),
@@ -238,6 +262,32 @@ public function addTheme(PluginApplicationInterface $application)
         }
     }
 
+    /**
+     * 利用可能なテーマを取得する
+     * @return array
+     * @checked
+     * @noTodo
+     */
+    public function getAvailableThemes(): array
+    {
+        if (!BcUtil::isInstalled()) return [];
+        $sitesTable = TableRegistry::getTableLocator()->get('BaserCore.Sites');
+        try {
+            $sites = $sitesTable->find()->where(['Sites.status' => true]);
+        } catch (MissingConnectionException) {
+            return [];
+        }
+        $themes = [];
+        foreach($sites as $site) {
+            if ($site->theme) {
+                if (!is_dir(CorePlugin::path($site->theme))) continue;
+                if(in_array($site->theme, $themes)) continue;
+                $themes[] = $site->theme;
+            }
+        }
+        return $themes;
+    }
+
     /**
      * デフォルトテンプレートを設定する
      * @checked

diff --git a/plugins/baser-core/src/Command/ComposerCommand.php b/plugins/baser-core/src/Command/ComposerCommand.php
@@ -34,6 +34,7 @@ class ComposerCommand extends Command
      * @return \Cake\Console\ConsoleOptionParser
      * @checked
      * @noTodo
+     * @unitTest
      */
     protected function buildOptionParser(\Cake\Console\ConsoleOptionParser $parser): \Cake\Console\ConsoleOptionParser
     {

diff --git a/plugins/baser-core/src/Command/CreateReleaseCommand.php b/plugins/baser-core/src/Command/CreateReleaseCommand.php
@@ -59,6 +59,7 @@ protected function buildOptionParser(\Cake\Console\ConsoleOptionParser $parser):
      * @return int|void|null
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function execute(Arguments $args, ConsoleIo $io)
     {
@@ -109,6 +110,7 @@ public function execute(Arguments $args, ConsoleIo $io)
      * @param string $packagePath
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function clonePackage(string $packagePath, string $branch)
     {

diff --git a/plugins/baser-core/src/Command/SetupInstallCommand.php b/plugins/baser-core/src/Command/SetupInstallCommand.php
@@ -41,6 +41,7 @@ class SetupInstallCommand extends Command
      * @return int|void|null
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function execute(Arguments $args, ConsoleIo $io)
     {

diff --git a/plugins/baser-core/src/Command/SetupTestCommand.php b/plugins/baser-core/src/Command/SetupTestCommand.php
@@ -39,6 +39,7 @@ class SetupTestCommand extends Command
      * @return int|void|null
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function execute(Arguments $args, ConsoleIo $io)
     {

diff --git a/plugins/baser-core/src/Controller/AppController.php b/plugins/baser-core/src/Controller/AppController.php
@@ -195,6 +195,7 @@ public function beforeFilter(EventInterface $event)
      * @return bool
      * @noTodo
      * @checked
+     * @unitTest
      */
     private function checkPermission()
     {

diff --git a/plugins/baser-core/src/Controller/BcFrontAppController.php b/plugins/baser-core/src/Controller/BcFrontAppController.php
@@ -73,6 +73,7 @@ public function beforeFilter(EventInterface $event)
      * @param EventInterface $event
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function beforeRender(EventInterface $event): void
     {

diff --git a/plugins/baser-core/src/Controller/UploadsController.php b/plugins/baser-core/src/Controller/UploadsController.php
@@ -31,6 +31,7 @@ class UploadsController extends AppController
      * @return \Cake\Http\Response
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function tmp()
     {

diff --git a/plugins/baser-core/src/Controller/UsersController.php b/plugins/baser-core/src/Controller/UsersController.php
@@ -33,6 +33,7 @@ class UsersController extends BcFrontAppController
      * @return void
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function initialize(): void
     {

diff --git a/plugins/baser-core/src/Model/Behavior/BcContentsBehavior.php b/plugins/baser-core/src/Model/Behavior/BcContentsBehavior.php
@@ -86,6 +86,7 @@ public function getType(): string
      * アソシエーション時に alias を除外する
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function offAlias(): void
     {
@@ -99,6 +100,7 @@ public function offAlias(): void
      * アソシエーション時に alias を含める
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function onAlias(): void
     {

diff --git a/plugins/baser-core/src/Model/Table/ContentsTable.php b/plugins/baser-core/src/Model/Table/ContentsTable.php
@@ -409,7 +409,7 @@ public function getUniqueName($name, $parentId, $contentId = null)
         $query = $this->find()->where(['name LIKE' => $name . '%', 'site_root' => false]);
         if (isset($parentId)) $query = $query->andWhere(['parent_id' => $parentId]);
         if ($contentId) {
-            $query = $query->andWhere(['id <>' => $contentId]);
+            $query = $query->andWhere(['Contents.id <>' => $contentId]);
         }
         $datas = $query->select('name')->orderBy('name')->all()->toArray();
         $datas = Hash::extract($datas, '{n}.name');
@@ -1221,7 +1221,7 @@ public function getRelatedSiteContents(int $id, array $options = []): array
             ['Sites.status' => true]
         ];
         if ($options['excludeIds']) {
-            $conditions['Contents.site_id <>'] = $options['excludeIds'];
+            $conditions['Contents.site_id NOT IN'] = $options['excludeIds'];
         }
         $conditions = array_merge($conditions, $this->getConditionAllowPublish());
 

diff --git a/plugins/baser-core/src/Model/Table/SitesTable.php b/plugins/baser-core/src/Model/Table/SitesTable.php
@@ -212,7 +212,7 @@ public function pluginExistsByDasherize($value)
      */
     public function getPublishedAll(): ResultSetInterface
     {
-        return $this->find()->where(['status' => true])->all();
+        return $this->find()->where(['Sites.status' => true])->all();
     }
 
     /**
@@ -236,7 +236,7 @@ public function getList($mainSiteId = null, $options = [])
 
         $conditions = [];
         if (!is_null($options['status'])) {
-            $conditions = ['status' => $options['status']];
+            $conditions = ['Sites.status' => $options['status']];
         }
 
         if (!is_null($mainSiteId)) {

diff --git a/plugins/baser-core/src/Service/Admin/PluginsAdminService.php b/plugins/baser-core/src/Service/Admin/PluginsAdminService.php
@@ -206,6 +206,7 @@ public function getViewVarsForAdd()
      * @return bool
      * @checked
      * @noTodo
+     * @unitTest
      */
     public function isPluginsDirWritable()
     {