Feeling Lucky
?
[#8322] - [3.5.0 Beta] Fix permissons editing with the PDO Driver
- Closed
- 9 Nov 2015
- Medium
- Build: staging
- # 8322
- Diff
- zero-24:fix_com_config_pdo
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
What is changed
Not allways we get a boolean if a SQL is true. e.g. on PDO we get a PDO object.
How to test
- install beta 1 (PDO Database Driver)
- go to the global config.
- try to change the config
- you got an "undefined" error
- apply the patch
- it works.
- try it again with the mysql driver.
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
| Milestone |
Added: |
||
wilsonge
- comment
- 8 Nov 2015
I've tested this and it works - but I'd much prefer the solution that I've commented in the code - which does the same thing in far less code :)
Fedik
- comment
- 8 Nov 2015
I have tested this item 
successfully on 8f311e9
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8322.
joomla-cms-bot
- comment
- 8 Nov 2015
This PR has received new commits.
CC: @Fedik
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8322.
wilsonge
- comment
- 8 Nov 2015
I have tested this item successfully on 93ff193
Works successfully with PDO and MySQLi drivers
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8322.
Fedik
- comment
- 9 Nov 2015
I have tested this item successfully on 93ff193
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8322.
| Status | Pending | ⇒ | Ready to Commit |
zero-24
- comment
- 9 Nov 2015
| Labels |
Added:
?
|
||
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-09 11:42:50 |
| Closed_By | ⇒ | wilsonge |
| Labels |
Removed:
?
|
||
I can confirm, that this pull fixes part of #8321 issue