Skip to content
This repository was archived by the owner on Apr 20, 2026. It is now read-only.

[Improvement]: Distinguish relation conditions applied as preconditions#249

Merged
kingjia90 merged 7 commits into
pimcore:2.xfrom
dolmit-tanel-paaro:distinguish-relation-conditions-applied-as-preconditions
Jun 12, 2025
Merged

[Improvement]: Distinguish relation conditions applied as preconditions#249
kingjia90 merged 7 commits into
pimcore:2.xfrom
dolmit-tanel-paaro:distinguish-relation-conditions-applied-as-preconditions

Conversation

@dolmit-tanel-paaro

Copy link
Copy Markdown
Contributor

Resolves #248

@dolmit-tanel-paaro dolmit-tanel-paaro force-pushed the distinguish-relation-conditions-applied-as-preconditions branch 3 times, most recently from fafe916 to 9b70410 Compare April 11, 2025 12:05
@dolmit-tanel-paaro

dolmit-tanel-paaro commented Apr 11, 2025

Copy link
Copy Markdown
Contributor Author

I'm not sure if we also should update filters in Findologic and SearchIndex namespaces.

@dolmit-tanel-paaro

Copy link
Copy Markdown
Contributor Author

I'm not sure if we also should update filters in Findologic and SearchIndex namespaces.

Updated those classes too.

@dolmit-tanel-paaro dolmit-tanel-paaro force-pushed the distinguish-relation-conditions-applied-as-preconditions branch from 92e3083 to 84fb3a3 Compare April 11, 2025 12:32
@sonarqubecloud

Copy link
Copy Markdown

@ghost ghost added the Pimcore:ToDo label May 5, 2025
@kingjia90 kingjia90 self-assigned this May 27, 2025

@kingjia90 kingjia90 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you for your PR!

I have Just found this inconsistency after a quick scan, haven't tested it yet all, not very familiar with this topic yet

Comment thread src/FilterService/FilterType/SearchIndex/MultiSelect.php Outdated
@dolmit-tanel-paaro dolmit-tanel-paaro force-pushed the distinguish-relation-conditions-applied-as-preconditions branch from 63ac52a to da0d776 Compare May 28, 2025 06:07
@dolmit-tanel-paaro dolmit-tanel-paaro force-pushed the distinguish-relation-conditions-applied-as-preconditions branch from da0d776 to 666bfe6 Compare May 28, 2025 10:48
@kingjia90 kingjia90 requested a review from Copilot May 28, 2025 15:10

This comment was marked as resolved.

Comment thread src/FilterService/FilterType/SelectRelation.php
Comment thread src/FilterService/FilterType/NumberRangeSelection.php Outdated
Comment thread src/FilterService/FilterType/NumberRangeSelection.php Outdated
Comment thread src/FilterService/FilterType/NumberRange.php Outdated
Comment thread src/FilterService/FilterType/NumberRange.php Outdated
Comment thread src/FilterService/FilterType/SelectRelation.php
@dolmit-tanel-paaro

Copy link
Copy Markdown
Contributor Author

@kingjia90 Pushed a fix for the situation you're stuck in.

@dolmit-tanel-paaro dolmit-tanel-paaro force-pushed the distinguish-relation-conditions-applied-as-preconditions branch from 33fe854 to f45d3c8 Compare May 30, 2025 09:36
@fashxp fashxp added this to the 2.1.0 milestone Jun 10, 2025
@sonarqubecloud

Copy link
Copy Markdown

@kingjia90 kingjia90 merged commit 4a67c08 into pimcore:2.x Jun 12, 2025
14 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 12, 2025
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.

[Improvement]: Distinguish relation conditions applied as preconditions

6 participants