API: Unified attachment_* and model_* for Attachments

This commit is contained in:
Geoffrey “Frogeye” Preud'homme
2020-02-08 11:51:04 +01:00
parent cebc0d7568
commit e81ce3317c
2 changed files with 10 additions and 10 deletions

View File

@@ -57,8 +57,8 @@ class AttachmentStoreRequest extends Request
'filename' => $this->string('filename'), 'filename' => $this->string('filename'),
'title' => $this->string('title'), 'title' => $this->string('title'),
'notes' => $this->nlString('notes'), 'notes' => $this->nlString('notes'),
'model' => $this->string('model'), 'model' => $this->string('attachable_type'),
'model_id' => $this->integer('model_id'), 'model_id' => $this->integer('attachable_id'),
]; ];
} }
@@ -77,14 +77,14 @@ class AttachmentStoreRequest extends Request
str_replace('FireflyIII\\Models\\', '', TransactionJournal::class), str_replace('FireflyIII\\Models\\', '', TransactionJournal::class),
] ]
); );
$model = $this->string('model'); $model = $this->string('attachable_type');
return [ return [
'filename' => 'required|between:1,255', 'filename' => 'required|between:1,255',
'title' => 'between:1,255', 'title' => 'between:1,255',
'notes' => 'between:1,65000', 'notes' => 'between:1,65000',
'model' => sprintf('required|in:%s', $models), 'attachable_type' => sprintf('required|in:%s', $models),
'model_id' => ['required', 'numeric', new IsValidAttachmentModel($model)], 'attachable_id' => ['required', 'numeric', new IsValidAttachmentModel($model)],
]; ];
} }
} }

View File

@@ -52,8 +52,8 @@ class AttachmentUpdateRequest extends Request
'filename' => $this->string('filename'), 'filename' => $this->string('filename'),
'title' => $this->string('title'), 'title' => $this->string('title'),
'notes' => $this->nlString('notes'), 'notes' => $this->nlString('notes'),
'model' => $this->string('model'), 'model' => $this->string('attachable_type'),
'model_id' => $this->integer('model_id'), 'model_id' => $this->integer('attachable_id'),
]; ];
} }