フォームのファイルフィールドの問題を修正 (#2957)

baserproject · Dec 20, 2023 · b23715c · b23715c
1 parent 611448a
commit b23715c
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 7 deletions.
diff --git a/plugins/baser-core/src/Utility/BcFileUploader.php b/plugins/baser-core/src/Utility/BcFileUploader.php
@@ -493,6 +493,11 @@ public function moveFileSessionToTmp($data, $fieldName)
         $uploadInfo['uploadable'] = true;
         $uploadInfo['ext'] = BcUtil::decodeContent($fileType, $fileName);
         $uploadedFile[$fieldName] = $uploadInfo;
+
+        // 一時ファイルを一度に複数扱う場合に1ファイルしかアップロードされない問題への対応
+        // 例: メールフォームのファイルフィールドなど
+        $uploadedFile = array_merge($this->getUploadingFiles($data['_bc_upload_id']), $uploadedFile);
+
         $this->setUploadingFiles($uploadedFile, $data['_bc_upload_id']);
         return true;
     }

diff --git a/plugins/baser-core/src/View/Helper/BcUploadHelper.php b/plugins/baser-core/src/View/Helper/BcUploadHelper.php
@@ -157,6 +157,10 @@ public function fileLink($fieldName, $entity, $options = [])
             $basePath = '/baser-core/uploads/tmp/';
         }
 
+        if (is_array($value)) {
+            return false;
+        }
+
         /* ファイルのパスを取得 */
         /* 画像の場合はサイズを指定する */
         if (isset($settings['saveDir'])) {

diff --git a/plugins/bc-mail/src/Service/MailMessagesService.php b/plugins/bc-mail/src/Service/MailMessagesService.php
@@ -120,14 +120,13 @@ public function getIndex(array $queryParams = [])
      */
     public function create(EntityInterface $mailContent, $postData)
     {
-        if (!$postData instanceof EntityInterface) {
-            // newEntity() だと配列が消えてしまうため、エンティティクラスで直接変換
-            $entity = new MailMessage($postData, ['source' => 'BcMail.MailMessages']);
-        } else {
-            $entity = $postData;
+        $entity = $this->MailMessages->newEntity($postData);
+        foreach ($postData as $postKey => $postValue) {
+            if (is_array($postValue)) {
+                $entity->$postKey = $postValue;
+            }
         }
-        $validateEntity = $this->MailMessages->patchEntity($this->MailMessages->newEmptyEntity(), $entity->toArray());
-        if (!$validateEntity->getErrors()) {
+        if (!$entity->getErrors()) {
             $mailFieldsTable = TableRegistry::getTableLocator()->get('BcMail.MailFields');
             $mailFields = $mailFieldsTable->find()->where(['MailFields.mail_content_id' => $mailContent->id, 'MailFields.use_field' => true])->all();
             $this->MailMessages->convertToDb($mailFields, $entity);