bug PR-4.3-dev ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
22 Feb 2023

Pull Request for Issue #22096 .

Summary of Changes

replaces urldecode with rawurldecode

rawurldecode() prints the plus sign per RFC3986 requirements. (urldecode(0 follows the RFC1866 spec and allows pluses to be decoded as spaces in the query part of the URL only)

Testing Instructions

No idea

@Hackwar the reasoning makes sense to me but I don't know if there are consequences

Actual result BEFORE applying this Pull Request

see #22096

Expected result AFTER applying this Pull Request

see #22096

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 - 22 Feb 2023
Category Libraries
avatar brianteeman brianteeman - open - 22 Feb 2023
avatar brianteeman brianteeman - change - 22 Feb 2023
Status New Pending
avatar brianteeman brianteeman - change - 22 Feb 2023
Labels Added: ?
avatar brianteeman brianteeman - change - 3 Apr 2023
Title
[4.2] allow + rawurldecode
[4.3] allow + rawurldecode
avatar brianteeman brianteeman - edited - 3 Apr 2023
avatar brianteeman
brianteeman - comment - 3 Apr 2023

rebased to 4.3

avatar brianteeman brianteeman - change - 3 Apr 2023
Labels Added: PR-4.3-dev ?
avatar brianteeman brianteeman - change - 13 May 2023
Labels Added: bug
Removed: ?
avatar brianteeman brianteeman - change - 8 Sep 2023
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2023-09-08 11:49:32
Closed_By brianteeman
avatar brianteeman brianteeman - close - 8 Sep 2023

Add a Comment

Login with GitHub to post a comment