? Error

User tests: Successful: Unsuccessful:

avatar pollen8
pollen8
30 Sep 2013

This patch checks the actual db collation first, and if not found falls back to the previous method of querying the first db table's first column's collation.
Issue with the previous method is that if you had a table with a field whose collation was not the same as the db's default collation the method would return the wrong result.

avatar pollen8 pollen8 - open - 30 Sep 2013
avatar brianteeman
brianteeman - comment - 13 Oct 2013

Thanks for your contribution - At this time we are only using github as the place to submit code fixes, the actual reporting of issues and testing fixes is still taking place on Joomlacode.

Please can you:

1) Open an item on the Joomlacode tracker in the appropriate area.

CMS Bug Reports: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8103

CMS Feature Requests: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemBrowse&tracker_id=8549

2) After submitting the item to the Joomlacode tracker, add a link to the Joomlacode tracker item here and make sure that you add a link to this GitHub issue or pull request on the joomlacode tracker item.

avatar allrude
allrude - comment - 11 Jul 2014

I looked at this request, but the file libraries/joomla/database/driver/mysqli.php seem to be changed since october 2013 so i can't test it. can you please update your PR

avatar Bakual
Bakual - comment - 11 Jul 2014

I'm closing this PR since it is against master instead of staging, has no tracker item and doesn't apply anymore according to previous comment.

@pollen8 If this is still an issue with current master, can you please open a new (updated) PR against the staging branch?

avatar Bakual Bakual - close - 11 Jul 2014

Add a Comment

Login with GitHub to post a comment