Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add ContentObjectProvider for content services #215

Open
alexander-schranz opened this issue Oct 15, 2021 · 0 comments
Open

Add ContentObjectProvider for content services #215

alexander-schranz opened this issue Oct 15, 2021 · 0 comments
Labels
DX Only affecting the end developer Performance Problems with performance
Milestone
Stable Release

Comments

@alexander-schranz
Copy link
Member

alexander-schranz commented Oct 15, 2021 •
edited
Loading

Currently there are a lot of content bundle services which loads the content rich entity. This should completely be replaced by a ContentObjectProviderInterface:

  • ContentRouteDefaultsProvider
  • PreviewContentObjectProvider
  • DimensionCollectionRepository
  • ContentReindexProvider
  • ContentSitemapProvider
  • ContentTeaserProvider
  • SmartContent ContentDataProvider

This are currently all service which access Infrastructure directly and should be replaced by using the ContentObjectProviderInterface which every Entity need to provide and can efficient load the entity how it requires in the specific case.

Also related to #192
@alexander-schranz alexander-schranz added this to the Stable Release milestone Oct 15, 2021
@alexander-schranz alexander-schranz added DX Only affecting the end developer Performance Problems with performance labels Oct 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
DX Only affecting the end developer Performance Problems with performance
Projects
None yet
Milestone
Stable Release
Development

No branches or pull requests
1 participant
@alexander-schranz