? ? Failure

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
21 Sep 2016

Summary of Changes

Updates the session package to the latest development head and makes use of the new validators API. Since JInput does not currently extend Joomla\Input\Input the validators are copied from the Framework and updated to use the CMS' class for now.

This also namespaces JSessionStorageJoomla.

Testing Instructions

Sessions continue working.

Documentation Changes Required

N/A; the new API's documented in the Framework repo's docs and is a transparent change to the CMS' user base.

avatar mbabker mbabker - open - 21 Sep 2016
avatar mbabker mbabker - change - 21 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2016
Category External Library Installation Libraries
avatar joomla-cms-bot joomla-cms-bot - change - 21 Sep 2016
Labels Added: ? ?
avatar zero-24
zero-24 - comment - 21 Sep 2016

hmm looks like it is not defined or do i miss something?
Error displaying the error page: A Joomla\Session\SessionInterface object has not been set.: Class 'Joomla\Cms\Session\Storage\JoomlaStorage' not found

avatar mbabker
mbabker - comment - 21 Sep 2016

I'd say you're missing something. I just checked all 3 apps and it's working right here. You've got the right files, right? libraries/src/Cms/Session/Storage/JoomlaStorage.php should have that class.

avatar zero-24 zero-24 - test_item - 21 Sep 2016 - Tested successfully
avatar zero-24
zero-24 - comment - 21 Sep 2016

I have tested this item successfully on addcc0b

yup it was on my end. login and navigation in backend and frontend still works.


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

avatar wilsonge wilsonge - change - 22 Sep 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-09-22 00:14:50
Closed_By wilsonge
avatar wilsonge
wilsonge - comment - 22 Sep 2016

Yay I love unexpected benefits of this :D

Add a Comment

Login with GitHub to post a comment