Skip to content
Snippets Groups Projects
Commit d3246716 authored by Christian Kuhn's avatar Christian Kuhn
Browse files

[TASK] Clean up DefaultDataProvider

Backend layout DefaultDataProvider receives some love:
* Use DI and declare public:true since single backend
  layout data providers are created using GU::makeInstance().
* Relocate @internal BackendLayoutView::getDefaultColumnLayout()
  to a private constant of DefaultDataProvider.
* Annotate DefaultDataProvider @internal since it is an
  interface implementing leaf class that should not be extended
  but exchanged using existing API.
* Declare DefaultDataProvider readonly since it is now
  a stateless service.
* Inline tableName property (a de facto constant) that is
  already hard coded at other places.
* Avoid a silent catch that would trigger an exception at another
  place and add a @todo hinting about an indirect ext:backend
  cross dependency to ext:frontend.

Resolves: #105255
Releases: main
Change-Id: I0b1fc42c6e802198eae6d0f1089679b28f7feac4
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/86536


Tested-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: default avatarChristian Kuhn <lolli@schwarzbu.ch>
Tested-by: default avatarStefan Bürk <stefan@buerk.tech>
Reviewed-by: default avatarOliver Bartsch <bo@cedev.de>
Tested-by: default avatarcore-ci <typo3@b13.com>
Tested-by: default avatarOliver Bartsch <bo@cedev.de>
Reviewed-by: default avatarStefan Bürk <stefan@buerk.tech>
parent c649800b
Branches
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment