? ? Pending

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
2 Jan 2017

Summary of Changes

There are many places using the database name to determine if an action can be taken. Since 3.5 there is a getServerType() method that can be used to do the same. Use it.

Testing Instructions

Methods taking different actions based on the database engine should still work and produce the same result.

Documentation Changes Required

N/A

avatar mbabker mbabker - open - 2 Jan 2017
avatar mbabker mbabker - change - 2 Jan 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jan 2017
Category Administration com_admin com_finder Installation Libraries Front End Plugins
avatar mbabker mbabker - change - 2 Jan 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 2 Jan 2017
Category Administration com_admin com_finder Installation Libraries Front End Plugins Administration com_admin com_finder Installation Libraries Front End Plugins Unit Tests
avatar alikon
alikon - comment - 3 Jan 2017

I have tested this item successfully on cd8a38f

mysqli & postgresql


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

avatar alikon alikon - test_item - 3 Jan 2017 - Tested successfully
avatar anibalsanchez
anibalsanchez - comment - 4 Jan 2017

I have tested this item successfully on cd8a38f

Test OK


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

avatar anibalsanchez anibalsanchez - test_item - 4 Jan 2017 - Tested successfully
avatar jeckodevelopment jeckodevelopment - change - 4 Jan 2017
Status Pending Ready to Commit
avatar jeckodevelopment
jeckodevelopment - comment - 4 Jan 2017

RTC


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

avatar rdeutz rdeutz - change - 5 Jan 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-01-05 10:31:41
Closed_By rdeutz
Labels Added: ?
avatar rdeutz rdeutz - close - 5 Jan 2017
avatar rdeutz rdeutz - merge - 5 Jan 2017

Add a Comment

Login with GitHub to post a comment