? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
8 Feb 2022

Pull Request for Issue reported by Jenn Gress

This is an release blocker as without a patch for this issue no update is possible anymore.

Summary of Changes

The http head call returns a different structure than in J3
Within J3 it returns an string:
image

On J4 it returns an array of strings while that header can only have one value?
image

Testing Instructions

  • Install 4.1.0-rc3
  • set the update server to testing
  • try to to reinistall 4.1.0-rc using the updater component

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

Update works

Documentation Changes Required

none

cc @bembelimen

avatar zero-24 zero-24 - open - 8 Feb 2022
avatar zero-24 zero-24 - change - 8 Feb 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Feb 2022
Category Administration com_joomlaupdate
avatar zero-24
zero-24 - comment - 8 Feb 2022

Ok the issue is this change within the framework: joomla-framework/http@07647b4

It changed the collection which from my understanding is an 1:1 mapping of header to value to and 1:x mapping where the x is an array.

avatar brianteeman
brianteeman - comment - 8 Feb 2022

The commit message is "Set each header as an array of values instead of a string" it looks like it was intentional

avatar zero-24
zero-24 - comment - 8 Feb 2022

The commit message is "Set each header as an array of values instead of a string" it looks like it was intentional

Sure it was meant to give some more context about what was changed that resulted into this behavior over the J3 behavior and why I was supprised about this change. ;)

avatar bayareajenn bayareajenn - test_item - 8 Feb 2022 - Tested successfully
avatar bayareajenn
bayareajenn - comment - 8 Feb 2022

I have tested this item successfully on a30138f

Tested patch in 4.1.0rc3 and PHP8.1 and it works.


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

avatar bembelimen bembelimen - test_item - 8 Feb 2022 - Tested successfully
avatar bembelimen
bembelimen - comment - 8 Feb 2022

I have tested this item successfully on a30138f


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

avatar bembelimen bembelimen - change - 8 Feb 2022
Status Pending Ready to Commit
avatar bembelimen
bembelimen - comment - 8 Feb 2022

RTC


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

avatar bembelimen bembelimen - change - 8 Feb 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-02-08 22:46:50
Closed_By bembelimen
Labels Added: ?
avatar bembelimen bembelimen - close - 8 Feb 2022
avatar bembelimen bembelimen - merge - 8 Feb 2022
avatar bembelimen
bembelimen - comment - 8 Feb 2022

Thx

Add a Comment

Login with GitHub to post a comment