? bug PR-4.3-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
19 Apr 2023

Pull Request for Issue #40412 .

Summary of Changes

This PR backports several bugfixes from the frameworks Input/Cookie class to the CMS class, making the classes compatible. This fixes the issue of the languagefilter not being able to set the cookie. The original issue is, that our base class for our application, AbstractWebApplication, defines the Input object as the framework class, while the child classes use the CMS class instead.

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

avatar joomla-cms-bot joomla-cms-bot - change - 19 Apr 2023
Category Libraries
avatar Hackwar Hackwar - open - 19 Apr 2023
avatar Hackwar Hackwar - change - 19 Apr 2023
Status New Pending
avatar bayareajenn bayareajenn - test_item - 21 Apr 2023 - Tested successfully
avatar bayareajenn
bayareajenn - comment - 21 Apr 2023

I have tested this item successfully on e33f299

Because there were no test instructions, I went around looking at other links to determine what and how I was supposed to test. Ultimately I applied the patch, added another language, changed Session to Year in the Language Filter Plugin. Tested on the frontend with no error. Thanks.


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

avatar bayareajenn
bayareajenn - comment - 21 Apr 2023

Adding screen shot for the cookie being 1 year out.


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

avatar formfranska formfranska - test_item - 21 Apr 2023 - Tested successfully
avatar formfranska
formfranska - comment - 21 Apr 2023

I have tested this item successfully on e33f299


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

avatar formfranska
formfranska - comment - 21 Apr 2023

Thank you for this patch. Super ?????


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

avatar richard67 richard67 - change - 21 Apr 2023
Status Pending Ready to Commit
Labels Added: bug PR-4.3-dev
avatar richard67
richard67 - comment - 21 Apr 2023

RTC


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

avatar obuisard obuisard - change - 21 Apr 2023
Labels Added: ?
avatar obuisard obuisard - change - 21 Apr 2023
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-04-21 19:08:04
Closed_By obuisard
avatar obuisard obuisard - close - 21 Apr 2023
avatar obuisard obuisard - merge - 21 Apr 2023
avatar obuisard
obuisard - comment - 21 Apr 2023

Thank you Hannes @Hackwar for the fix!

Add a Comment

Login with GitHub to post a comment