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

SimpleSearch ignoring context setting #32

Open
BigBlockStudios opened this issue Apr 23, 2016 · 0 comments
Open

SimpleSearch ignoring context setting #32

BigBlockStudios opened this issue Apr 23, 2016 · 0 comments

Comments

@BigBlockStudios
Copy link

There is a bug in the generated SQL for simplesearch context settings as well as unpublished resources:

SELECT DISTINCT modResource.id AS modResource_id, etc-etc-etc modResource.properties AS modResource_properties
FROM modx_site_content AS modResource
WHERE ( (
modResource.pagetitle LIKE '%car%' OR modResource.longtitle LIKE '%car%' OR modResource.alias LIKE '%car%' OR modResource.description LIKE '%car%' OR modResource.introtext LIKE '%car%' OR modResource.content LIKE '%car%'
OR modResource.pagetitle LIKE '%seat%' OR modResource.longtitle LIKE '%seat%' OR modResource.alias LIKE '%seat%' OR modResource.description LIKE '%seat%' OR modResource.introtext LIKE '%seat%' OR modResource.content LIKE '%seat%'
OR modResource.pagetitle LIKE '%safety%' OR modResource.longtitle LIKE '%safety%' OR modResource.alias LIKE '%safety%' OR modResource.description LIKE '%safety%' OR modResource.introtext LIKE '%safety%' OR modResource.content LIKE '%safety%'

AND modResource.published = 1
AND modResource.searchable = 1
AND modResource.deleted = 0
AND modResource.context_key IN ('English') )
AND ( modResource.id != 0 AND modResource.id != 0 ) )

the issue is with the order of the AND/OR conditions ~ it should be:

WHERE ( (
modResource.pagetitle LIKE '%car%' OR etc - etc - etc LIKE '%safety%' ) <!-- or conditions terminated here --> ANDmodResource.published= 1 ANDmodResource.searchable= 1 ANDmodResource.deleted= 0 ANDmodResource.context_keyIN ('English') AND (modResource.id!= 0 ANDmodResource.id` != 0 ) )

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

No branches or pull requests

1 participant