Success

User tests: Successful: Unsuccessful:

avatar beat
beat
24 Jul 2013

Related tracker:
http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=31550

Related discussion:
https://groups.google.com/forum/#!msg/joomlabugsquad/Ag7l3UQhF8M/dhcDpRznxsUJ

This patch also has a joomla-codestyle cleanup for myslq.php and a single line codestyle cleanup for tags observer.

avatar beat beat - open - 24 Jul 2013
avatar ndeet
ndeet - comment - 24 Jul 2013

Hi, great work, the error on MySQL installations where profiling is disabled is now fixed.

avatar beat
beat - comment - 24 Jul 2013

Great, please comment on tracker here:
joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=31550

with a line starting with:

"@test success"

that way, it's visible.

We still need a 2nd tester...

avatar beat
beat - comment - 25 Jul 2013

Ok, I have fixed the bug found by Kyle during 2nd test when using mysql driver (and that I could reproduce here so that I could test fix too) in cmmit 9967dd5 .

I have also merged with latest master this branch, so that pull-request is up-to-date.

Finally, I have done tests with Postgresql in debug and non-debug mode to make sure there are no regressions.

So I think that with 2 successful tests, and fact that without this all J sites in mysql (without i) driver mode are broken with debug on, it's a high priority tracker item.

avatar mbabker
mbabker - comment - 25 Jul 2013

Committed for 3.1.4 packages.

avatar mbabker mbabker - close - 25 Jul 2013

Add a Comment

Login with GitHub to post a comment