[riasvdv]

Description

POST {cpTrigger}/actions/entries/save-entry returned a 500 when entryId arrived as ordinary form data, because the string went straight to Entries::getEntryById(int $entryId):

TypeError: Entries::getEntryById(): Argument #1 ($entryId) must be of type int, string given

StoreEntryController now reads the IDs with $request->integer(), the same way sibling controllers read theirs:

- $entryId = $this->request->input('canonicalId')
-     ?? $this->request->input('sourceId')
-     ?? $this->request->input('entryId');
+ $entryId = $this->request->integer('canonicalId')
+     ?: $this->request->integer('sourceId')
+     ?: $this->request->integer('entryId');

Evidence

StoreEntryControllerTest › "can update an existing entry" now posts entryId as a string, the way a browser form does. The test client had been sending it as an int, so the bug went unnoticed.

• Before: post(save-entry, ['entryId' => '123', ...]) → 500, TypeError … must be of type int, string given
  After: → redirect, and the entry's title is updated

[github-actions]

📚 Storybook previews

@craftcms/ui — open Storybook

No changed components detected in this Storybook.

resources/js — open Storybook

No changed components detected in this Storybook.