Skip to content

Added prebuild event for extending possibilities to hook into graphql queries#982

Merged
robertSt7 merged 3 commits intopimcore:2.xfrom
valantic-cx-alps:enhancement/pre-build-event
Feb 12, 2026
Merged

Added prebuild event for extending possibilities to hook into graphql queries#982
robertSt7 merged 3 commits intopimcore:2.xfrom
valantic-cx-alps:enhancement/pre-build-event

Conversation

@valantic-cx-alps
Copy link
Copy Markdown
Contributor

No description provided.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 6, 2025

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new GraphQL listing “pre-build” event hook intended to allow extensions to intervene before a listing is constructed and loaded.

Changes:

  • Dispatches a new ListingEvents::PRE_BUILD event at the start of QueryType::resolveListing().
  • Introduces ListingEvents::PRE_BUILD event constant for subscribers.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
src/GraphQL/Resolver/QueryType.php Dispatches the new pre-build event before creating and configuring the object listing.
src/Event/GraphQL/ListingEvents.php Defines the new PRE_BUILD event name/annotation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/GraphQL/Resolver/QueryType.php
Comment thread src/GraphQL/Resolver/QueryType.php
robertSt7 and others added 2 commits January 23, 2026 09:46
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sonarqubecloud
Copy link
Copy Markdown

@robertSt7 robertSt7 added this to the 2.4.0 milestone Feb 12, 2026
@robertSt7 robertSt7 merged commit 0ab5180 into pimcore:2.x Feb 12, 2026
11 of 20 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Feb 12, 2026
@robertSt7
Copy link
Copy Markdown
Contributor

@valantic-cx-alps Thanks for adding the prebuild event

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants