? Failure

User tests: Successful: Unsuccessful:

avatar chrisdavenport
chrisdavenport
25 Mar 2016

Found a way to make it work, I think.

Should fix #9534 and #9582.

avatar chrisdavenport chrisdavenport - open - 25 Mar 2016
avatar chrisdavenport chrisdavenport - change - 25 Mar 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Mar 2016
Labels Added: ?
avatar brianteeman
brianteeman - comment - 25 Mar 2016

Can you write some test instructions please.

avatar mbabker
mbabker - comment - 25 Mar 2016

That works for PDO MySQL, but other PDO connections will fail. The unit tests fail here because the function doesn't exist on SQLite (which is the engine most of the unit tests are running off of).

avatar chrisdavenport
chrisdavenport - comment - 25 Mar 2016

I had a feeling something like that might be the case. Looks like the problem is in PDO itself and may not be fixable in Joomla without some hacky workarounds.

avatar wilsonge
wilsonge - comment - 26 Mar 2016

Can we not just put this in the PDOMySQL driver file then Chris?

avatar wilsonge wilsonge - change - 26 Mar 2016
Milestone Added:
avatar chrisdavenport
chrisdavenport - comment - 26 Mar 2016

I didn't know that file even existed! I've never used PDO before. It looks like it will need specific fixes for MSSQL and possibly others too, which surely defeats the object of using PDO in the first place.

avatar brianteeman brianteeman - change - 26 Mar 2016
Category Libraries
avatar Fedik Fedik - test_item - 26 Mar 2016 - Tested successfully
avatar Fedik
Fedik - comment - 26 Mar 2016

I have tested this item :white_check_mark: successfully on 78c57c9

it also fixes #9572


This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9601.

avatar mbabker
mbabker - comment - 26 Mar 2016

Are we sure this isn't either a PHP bug or a Joomla implementation bug? Seems odd that a method in a PHP extension would fail so catastrophically.

avatar chrisdavenport
chrisdavenport - comment - 26 Mar 2016

Closing in favour of #9610.

avatar chrisdavenport chrisdavenport - change - 26 Mar 2016
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2016-03-26 17:14:50
Closed_By chrisdavenport
avatar chrisdavenport chrisdavenport - close - 26 Mar 2016
avatar chrisdavenport chrisdavenport - close - 26 Mar 2016
avatar chrisdavenport chrisdavenport - head_ref_deleted - 26 Mar 2016
avatar zero-24
zero-24 - comment - 26 Mar 2016

can someone please remove the wrong milestone here? @brianteeman @wilsonge @mbabker

avatar wilsonge wilsonge - change - 26 Mar 2016
Milestone Removed:
avatar wilsonge
wilsonge - comment - 26 Mar 2016

done - thanks :)

Add a Comment

Login with GitHub to post a comment