[Issue] Include Constructor to be a part of CommandListInterface
API, extend inline documentation
#31102
Labels
Component: Console
feature request
Priority: P2
A defect with this priority could have functionality issues which are not to expectations.
Progress: PR in progress
Severity: S4
Affects aesthetics, professional look and feel, “quality” or “usability”.
Triage: Dev.Experience
Issue related to Developer Experience and needs help with Triage to Confirm or Reject it
Projects
Milestone
This issue is automatically created based on existing pull request: #29762: Include Constructor to be a part of
CommandListInterface
API, extend inline documentationDescription (*)
In my personal opinion usingCommandListInterface
is invalid way of adding new Commands to interface.It is because Constructor is not a part of Interface (Service Contract), thus it is not "guaranteed" part of Interface.
UsingCommandList
is against Magento rules, thus it's implementation is not guaranteed and can change in backwards-incompatible way.I'm extending theCommandList
with@api
to ensure developers that this class won't change backwards-incompatible way and this way - they can either add new classes using:Argument injection toCommandList
classPlugin ongetCommands
onCommandListInterface
.I'd love to get your feedback there.After discussion with @kandy I decided to add the
__construct()
to the API.Answering the main concerns: https://3v4l.org/8ug8i
As it affects you (community) I need your 👍🏻 or comments to pass the change:
Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)
Thanks
The text was updated successfully, but these errors were encountered: