Actions
Task #94683
opensetPid on AbstractDomainObject should not force void return type
Status:
New
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2021-08-01
Due date:
% Done:
0%
Estimated time:
TYPO3 Version:
10
PHP Version:
Tags:
Complexity:
Sprint Focus:
Description
I believe it is a good practice to have fluent setters (return type self) for instance in order to have immutable objects. Sure in that case it should be called with*. Nevertheless I don't find it necessary to force void as a return type on AbstractDomainObject->setPid. Also this avoids clean code styles like:
$someService->save((new Entity())->setPid(123));
/**
* Setter for the pid.
*
* @param int $pid
*/
public function setPid(int $pid): void
{
$this->pid = $pid;
}
If TYPO3 does not use this i fully understand but at least it should not force void return type, which makes it impossible as an extension developer to have fluent setters apart from creating a different method like setPageUid().
Kind regards and thank you for the great work
Rozbeh Chiryai Sharahi
Updated by Bastian Bringenberg over 2 years ago
- Project changed from 9 to TYPO3 Core
- TYPO3 Version set to 10
Guessed TYPO3 Version.
Moved Project.
Actions