? Success

User tests: Successful: Unsuccessful:

avatar alikon
alikon
15 Dec 2014

this PR fix two issue (a, b)

Steps to reproduce the issue

(a) On administration side
go to User -> User Manager and click to sort for Last Visit Date or Registration Date column

(b) On administration side
go to User -> User Manager and click on search tools and filter on Registration Date

Expected result

(a) The user list is sorted by registration or visit date
(b) The user list is filtered by register date

Actual result

(a) but it throws a Database Error
query failed (error # %s): %s SQL=SELECT a.* FROM "#_users" AS a ORDER BY a.registerDate ASC LIMIT 20

(b)but it throws a Database Error
query failed (error # %s): %s SQL=SELECT a.* FROM "#_users" AS a WHERE a.registerDate >= '2014-09-15 19:43:46' AND a.registerDate <='2014-12-15 19:43:46' ORDER BY a.name asc LIMIT 20

System information

Joomla! 3.4.0-alpha
Postgres 9.3.5
PHP 5.5.9

Additional comments

Postgres is a little bit different than mysql on mixed case columns name , so this is is only a temporary fix
should be better if all joomla fields became lowercase, the user table is the only table that have mixed case fields

avatar alikon alikon - open - 15 Dec 2014
avatar jissues-bot jissues-bot - change - 15 Dec 2014
Labels Added: ?
avatar brianteeman brianteeman - change - 15 Dec 2014
Category Postgresql
avatar waader
waader - comment - 15 Dec 2014

@test works!

Thank you @alikon for the prompt fixing! I tested with mysql, postgresql and mssql.

avatar zero-24
zero-24 - comment - 15 Dec 2014

@alikon here is a PR that fix the codestyle that should make travis happy :+1: alikon#1 Thanks!

avatar alikon alikon - change - 16 Dec 2014
The description was changed
avatar wilsonge
wilsonge - comment - 27 Dec 2014

@test works here on postgres as well. Merging.

avatar wilsonge wilsonge - close - 27 Dec 2014
avatar wilsonge wilsonge - change - 27 Dec 2014
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2014-12-27 17:55:51

Add a Comment

Login with GitHub to post a comment