Skip to content

[Bugfix] Display search button in pimcore.helpers.editmode.openLinkEditPanel only when search is available#1056

Merged
kingjia90 merged 1 commit intopimcore:1.7from
jdreesen:fix-openLinkEditPanel
Jan 19, 2026
Merged

[Bugfix] Display search button in pimcore.helpers.editmode.openLinkEditPanel only when search is available#1056
kingjia90 merged 1 commit intopimcore:1.7from
jdreesen:fix-openLinkEditPanel

Conversation

@jdreesen
Copy link
Contributor

The button has been defined twice: once conditionally and once inline.
The inline definition is removed in favor of the conditional one.
Otherwise, the button is show even is no search implementation is available but does nothing on click.

This was an oversight in pimcore/pimcore#13173.

@jdreesen jdreesen changed the base branch from 2.x to 1.7 November 18, 2025 15:53
@sonarqubecloud
Copy link

@jdreesen
Copy link
Contributor Author

jdreesen commented Nov 18, 2025

Now that we finally upgraded to Pimcore 11 and I can use this feature, I noticed that we forgot to support subtypes and classes in pimcore/pimcore#13173 😞

In the hope that it counts as a bugfix (and we don't have to wait again till we upgrade to Pimcore 12), here are the PRs that add it:

@kingjia90 kingjia90 self-assigned this Jan 19, 2026
@kingjia90 kingjia90 added the Bug label Jan 19, 2026
@kingjia90 kingjia90 added this to the 2.2.4 milestone Jan 19, 2026
@kingjia90 kingjia90 merged commit caca07a into pimcore:1.7 Jan 19, 2026
12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 19, 2026
@kingjia90
Copy link
Contributor

kingjia90 commented Jan 19, 2026

Thank you for the follow up PR!
Unfortunately, there would be no 1.7 releases, so this is gonna be merged in 2.2

Ops, messed up (mixed up with #1057 🙅🏻 ), the base branch is still 1.7 on this, but won't be tagged until next LTS then..

@jdreesen jdreesen deleted the fix-openLinkEditPanel branch January 19, 2026 09:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants