fix #3143 【5.1】URL形式以外の文字も入力できてしまいます【コンテンツ管理＞リンク編集】

baserproject · Feb 22, 2024 · ac553f5 · ac553f5
1 parent 9aae998
commit ac553f5
Show file tree

Hide file tree

Showing 2 changed files with 45 additions and 2 deletions.
diff --git a/plugins/bc-content-link/src/Model/Table/ContentLinksTable.php b/plugins/bc-content-link/src/Model/Table/ContentLinksTable.php
@@ -64,8 +64,10 @@ public function validationDefault(Validator $validator): Validator
         ->requirePresence('id', 'update');
 
         $validator
-        ->scalar('url')
-        ->notEmptyString('url', __d('baser_core', 'リンク先URLを入力してください。'), 'update');
+            ->scalar('url')
+            ->notEmptyString('url', __d('baser_core', 'リンク先URLを入力してください。'), 'update')
+            ->regex('url', '/^http|^\/.*/', __d('baser_core', 'リンク先URLはURLの形式を入力してください。'))
+            ->maxLength('url', 255, __d('baser_core', 'リンク先URLは255文字以内で入力してください。'));
 
         return $validator;
     }

diff --git a/plugins/bc-content-link/tests/TestCase/Model/Table/ContentLinksTableTest.php b/plugins/bc-content-link/tests/TestCase/Model/Table/ContentLinksTableTest.php
@@ -79,6 +79,47 @@ public function testValidationDefault()
         ], $contentLink->getErrors());
     }
 
+    /**
+     * Test testValidationURL
+     */
+    public function testValidationURL()
+    {
+        $validator = $this->ContentLinks->getValidator('default');
+        //エラー場合、
+        //スペースだけ入力
+        $errors = $validator->validate([
+            'url' => '        '
+        ]);
+        //戻り値を確認
+        $this->assertEquals('リンク先URLはURLの形式を入力してください。', current($errors['url']));
+        //スラッシュがない場合
+        $errors = $validator->validate([
+            'url' => 'あああああ'
+        ]);
+        //戻り値を確認
+        $this->assertEquals('リンク先URLはURLの形式を入力してください。', current($errors['url']));
+
+        //長いURLを入力場合
+        $errors = $validator->validate([
+            'url' => '/' . str_repeat('a', 255)
+        ]);
+        //戻り値を確認
+        $this->assertEquals('リンク先URLは255文字以内で入力してください。', current($errors['url']));
+
+        //正常場合、
+        $errors = $validator->validate([
+            'url' => '/test'
+        ]);
+        //戻り値を確認
+        $this->assertCount(0, $errors);
+
+        $errors = $validator->validate([
+            'url' => 'https://basercms.net/'
+        ]);
+        //戻り値を確認
+        $this->assertCount(0, $errors);
+    }
+
     /**
      * test beforeCopyEvent
      */