Pending

User tests: Successful: Unsuccessful:

avatar pborreli
pborreli
17 Jul 2012
avatar pborreli pborreli - open - 17 Jul 2012
avatar infograf768
infograf768 - comment - 17 Jul 2012

Please create a tracker on joomlacode, reference here and explain the issue with testing instructions.

avatar pborreli pborreli - close - 17 Jul 2012
avatar willdurand
willdurand - comment - 17 Jul 2012

Seriously?

avatar lolautruche
lolautruche - comment - 17 Jul 2012

Openness fail

avatar clawfire
clawfire - comment - 17 Jul 2012

"please user 45 differents tools before submiting some fix to our apps, 'cause time wasted to patch our code is not enough. Cheers" #sarcasm

avatar bdunogier
bdunogier - comment - 17 Jul 2012

I'm not a Joomla expert, but testing this might prove a tad difficult...

avatar elinw
elinw - comment - 17 Jul 2012

I don't what it has to do with openness since the process and the fact that the tracker not git hub is the point of truth for issues is documented in a million places. The way you test manually is to apply it and make sure that nothing changes, which obviously will be true in this case since this is just a code quality fix. Also the pull would need to be revised since it includes a platform change unless you've sent that change already to the platform repo.

avatar Rydgel
Rydgel - comment - 17 Jul 2012

PHP.

avatar gries
gries - comment - 17 Jul 2012

I would say that THIS is the right case to choose pragmatism over process...

avatar digitalkaoz
digitalkaoz - comment - 17 Jul 2012

rly? lol!

avatar realityking
realityking - comment - 17 Jul 2012

It would be really helpful if you could submit the fix for libraries/joomla/database/driver/pdo.php to joomla/joomla-platform. We treat the platform like an upstream library and we'll pick up the fix with the next merge. The platform accepts pull requests BTW ;)

avatar pborreli
pborreli - comment - 17 Jul 2012
avatar realityking
realityking - comment - 17 Jul 2012

@pborreli
Thanks. I added tiny comment, just waiting for the unit tests to pass (http://developer.joomla.org/pulls/) and I'll merge it.

Add a Comment

Login with GitHub to post a comment