? ?
avatar Fedik
Fedik
22 Jan 2022

Pull Request for Issue #36788 and #35895 .

Summary of Changes

Correctly check path. Same as #36789

Testing Instructions

Follow #36788 and #35895

Actual result BEFORE applying this Pull Request

A warning about open_basedir stuff

Expected result AFTER applying this Pull Request

No warning

Documentation Changes Required

None

avatar Fedik Fedik - open - 22 Jan 2022
avatar ChristineWk
ChristineWk - comment - 22 Jan 2022

Thank you @Fedik
For J 3 = OK. (36789)
For J 4 = (Patchtester) got Message:
404RuntimeException
Invalid Issue

avatar ChristineWk
ChristineWk - comment - 22 Jan 2022

Tested successfully.


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

avatar Fedik Fedik - change - 23 Jan 2022
Status New Pending
avatar Fedik Fedik - change - 23 Jan 2022
The description was changed
avatar Fedik Fedik - edited - 23 Jan 2022
avatar joomla-cms-bot joomla-cms-bot - change - 23 Jan 2022
Category Administration com_menus
avatar richard67
richard67 - comment - 23 Jan 2022

I see that @ChristineWk could not mark her test result because the "Test this" button is missing in the issue tracker. A problem which we sometimes have for (yet) unknown reasons. Maybe it will appear later, we will see.

avatar richard67 richard67 - change - 23 Jan 2022
Labels Added: ?
avatar richard67 richard67 - change - 23 Jan 2022
Build 4.0-dev 4.1-dev
avatar richard67
richard67 - comment - 23 Jan 2022

I have also tested with success. Still no "Test this" button for this PR in the issue tracker, but it has 2 human tests so I set it RTC (if that will work at least).

avatar richard67 richard67 - change - 23 Jan 2022
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 23 Jan 2022

RTC


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

avatar bembelimen bembelimen - change - 25 Jan 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-01-25 03:36:04
Closed_By bembelimen
Labels Added: ?
avatar bembelimen bembelimen - close - 25 Jan 2022
avatar bembelimen bembelimen - merge - 25 Jan 2022
avatar bembelimen
bembelimen - comment - 25 Jan 2022

Thx

Add a Comment

Login with GitHub to post a comment