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.

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

"All" content selection not working on custom content portlets #16587

Closed
wezell opened this issue May 16, 2019 · 5 comments

Comments

@wezell
Copy link
Contributor

commented May 16, 2019

Create a custom content portlet that has uses blogs, news as content types.

Try to use and search by "all" content types. It does not work.

custom-content-types

@wezell wezell added this to the Rex Current milestone May 16, 2019

@erickgonzalez erickgonzalez self-assigned this May 21, 2019

erickgonzalez added a commit that referenced this issue May 22, 2019

@erickgonzalez

This comment has been minimized.

Copy link
Contributor

commented May 22, 2019

PR: #16610

dsilvam added a commit that referenced this issue May 22, 2019

@dsilvam dsilvam added the Merged label May 22, 2019

@dsilvam dsilvam added the Needs QA label May 22, 2019

@dsilvam dsilvam modified the milestones: 051419_REX, Rex Current May 28, 2019

dsilvam added a commit that referenced this issue May 31, 2019

@nollymar nollymar self-assigned this May 31, 2019

@bryanboza bryanboza added this to REX in QA May 31, 2019

@nollymar

This comment has been minimized.

Copy link
Contributor

commented May 31, 2019

Tests Results: The issue persists in master. However, in 5.1.6 it is working correctly

@erickgonzalez

This comment has been minimized.

Copy link
Contributor

commented Jun 3, 2019

The issue is because the changes in the ES Analyzer, the OR clause is not working.
Ticket to follow up #16141 (comment)

@bryanboza

This comment has been minimized.

Copy link
Contributor

commented Jun 3, 2019

Fixed, tested on the release branch // FF

@bryanboza

This comment has been minimized.

Copy link
Contributor

commented Jun 10, 2019

Fixed, tested on master // postgres // FF

@bryanboza bryanboza added the Passed QA label Jun 10, 2019

@bryanboza bryanboza moved this from REX to Done in QA Jun 10, 2019

@wezell wezell closed this Jun 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.