[#5385] - Users users model - postgresql (case sensitive DDL)
- Closed
- 15 Dec 2014
- Medium
- Build: staging
- # 5385
- Diff
- alikon:patch-9
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Steps to reproduce the issue
On administration side
go to User -> User Manager and click to sort for Last Visit Date or Registration Date column
Expected result
The user list is sorted by registration or visit date
Actual result
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
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
Votes
jissues-bot
- | Labels |
Added:
?
|
||
brianteeman
- | Category | ⇒ | Postgresql |
Shame on me I've tested with 1 user only. I'll review it the
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/5385.
i was doing the correct thing
quote the field name with mixed case (ie."registerDate")
in the wrong place and in the wrong way
now should work on both
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-12-11 19:27:42 |
| Status | Closed | ⇒ | New |
| Status | New | ⇒ | Closed |
| Closed_Date | 2014-12-11 19:27:42 | ⇒ | 2014-12-15 20:10:12 |
Your patch resolves the database error but the sorting in the user manager is impacted in a negativ way - tested with postgres and mysql.
This is the sorting order without your patch (ordered by registration date):
And with the patch applied:
Also note that the arrow indicating the sorting direction disappeared.