? ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
7 Oct 2017

Pull Request for Issue #18259 .

Summary of Changes

added the missing db selection option

Testing Instructions

enable redis session storage
and select a different redis database for session storage ex. 2
then connect to redis from console redis-cli -n 2
and list KEYS *

screenshot from 2017-10-07 11-55-18

Expected result

you are able to store php session on different from default 0 redis database
screenshot from 2017-10-07 11-43-49

Actual result

only default 0 db

avatar joomla-cms-bot joomla-cms-bot - change - 7 Oct 2017
Category Administration com_config Libraries
avatar alikon alikon - open - 7 Oct 2017
avatar alikon alikon - change - 7 Oct 2017
Status New Pending
avatar alikon alikon - change - 7 Oct 2017
The description was changed
avatar alikon alikon - edited - 7 Oct 2017
avatar alikon alikon - change - 8 Oct 2017
Labels Added: ?
avatar alikon alikon - change - 8 Oct 2017
The description was changed
avatar alikon alikon - edited - 8 Oct 2017
avatar simbus82 simbus82 - test_item - 11 Oct 2017 - Tested successfully
avatar simbus82
simbus82 - comment - 11 Oct 2017

I have tested this item successfully on 5e1b04c


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

avatar csthomas csthomas - test_item - 16 Oct 2017 - Tested successfully
avatar csthomas
csthomas - comment - 16 Oct 2017

I have tested this item successfully on 5e1b04c


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 17 Oct 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 17 Oct 2017

RTC after two successful tests.

avatar mbabker mbabker - change - 19 Oct 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-10-19 21:52:26
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 19 Oct 2017
avatar mbabker mbabker - merge - 19 Oct 2017

Add a Comment

Login with GitHub to post a comment