Skip to content
This repository has been archived by the owner on Aug 15, 2023. It is now read-only.

OCQuery - Alfresco can't search on only TYPE #1273

Closed
ghost opened this issue Dec 22, 2016 · 6 comments
Closed

OCQuery - Alfresco can't search on only TYPE #1273

ghost opened this issue Dec 22, 2016 · 6 comments
Assignees
Milestone

Comments

@ghost
Copy link

ghost commented Dec 22, 2016

Previously, Alfresco was able to parse very simple queries and could search on TYPE only. Recent changes to AbstractAlfrescoEmbQueryImpl have removed this functionality, and it is causing issues across HPI. The following query will now fail when it was previously working:

screen shot 2016-12-22 at 10 33 32 am

@ghost ghost added this to the HPI 2.5 milestone Dec 22, 2016
@gsteimer
Copy link
Contributor

Related to #1271. If that issue is fixed by adding some sort of select statement, we should test that those changes can be reverted once this is fixed in OC.

@mikeblum
Copy link
Contributor

We'll want to maintain the functionality of searching on multiple types:

(QUERY AND (TYPE:A Or TYPE:B OR TYPE:C))

We can add a case to AlfrescoEmbQueryImpl where the query is just a type, not a query + type.

@ZachGerth
Copy link

The issue is just related to empty parentheses at the beginning of the query where we generate Alfresco queries. If we simply make sure to not add this empty pair of parentheses, we avoid the issue entirely.

@mikeblum
Copy link
Contributor

yep - sounds reasonable. I blindly put a starting ( and a closing ) before the TYPE clause to avoid a leaky expression.

@ZachGerth
Copy link

ZachGerth commented Dec 22, 2016 via email

@rosenoam
Copy link

rosenoam commented Dec 22, 2016

OC Revision #15715

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

No branches or pull requests

4 participants