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

Errors in Smart search component on PostgreSQL #13848

Closed
csthomas opened this issue Feb 1, 2017 · 4 comments
Closed

Errors in Smart search component on PostgreSQL #13848

csthomas opened this issue Feb 1, 2017 · 4 comments

Comments

@csthomas
Copy link
Contributor

csthomas commented Feb 1, 2017

Steps to reproduce the issue

After applied PR #12313 there appears new errors.

Click on index and wait until indexing complete.

  1. Then try to search some word in search field.
  • ERROR: operator does not exist: timestamp without time zone ~ unknown LINE 4: ...e%' OR "l"."url" LIKE '%lime%' OR "l"."indexdate" LIKE '%li... ^ HINT: No operator matches the given name and argument type(s). You might need to add explicit type casts
  1. Click on Content Maps on the left panel. administrator/index.php?option=com_finder&view=maps
  • ERROR: column "s.count_published" must appear in the GROUP BY clause or be used in an aggregate function LINE 1: SELECT a.,s.count_published,s.count_unpublished,COUNT(b.id)... ^ ERROR: column "s.count_published" must appear in the GROUP BY clause or be used in an aggregate function LINE 1: SELECT a.,s.count_published,s.count_unpublished,COUNT(b.id)... ^ ERROR: column "s.count_published" must appear in the GROUP BY clause or be used in an aggregate function LINE 1: SELECT a.*,s.count_published,s.count_unpublished,COUNT(b.id)... ^

Expected result

No errors.

Actual result

As above.

System information (as much as possible)

PHP 7.0.13-0ubuntu0.16.04.1
PostgreSQL 9.5.5

@csthomas
Copy link
Contributor Author

csthomas commented Feb 2, 2017

I'm working on point 2.

@alikon
Copy link
Contributor

alikon commented Feb 2, 2017

for let com_finder work on postgresql we need to fix some more issues i'm afraid
#12348 , #12417

@alikon
Copy link
Contributor

alikon commented Feb 2, 2017

@csthomas #12348 should solve point 1.

@csthomas
Copy link
Contributor Author

csthomas commented Feb 6, 2017

I have a ready PR #13854 to test.

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

No branches or pull requests

4 participants