Skip to content

Fix an PHP 8 issue with the 1.3 branch#702

Merged
mblaney merged 1 commit intosimplepie:one-dot-threefrom
zero-24:patch-1
Dec 19, 2021
Merged

Fix an PHP 8 issue with the 1.3 branch#702
mblaney merged 1 commit intosimplepie:one-dot-threefrom
zero-24:patch-1

Conversation

@zero-24
Copy link

@zero-24 zero-24 commented Nov 26, 2021

Hi,

within Joomla 3 we still have to ship the 1.3 version of simplepie as we are required to still support PHP 5.3 but this version gets the issue patched here when used on PHP 8: joomla/joomla-cms#36120

This change here should fix it. So the question is do you still accept changes to this branch and would do a release of the 1.3 branch?

Thanks

@jrfnl
Copy link
Contributor

jrfnl commented Dec 18, 2021

Related to #617, which was included in the 1.5.3 release.

@mblaney mblaney merged commit e497949 into simplepie:one-dot-three Dec 19, 2021
@mblaney
Copy link
Member

mblaney commented Dec 19, 2021

no worries @zero-24 just released 1.3.2

@zero-24
Copy link
Author

zero-24 commented Dec 20, 2021

Thanks @mblaney I have just run composer update but I have got the version 1.3.2-dev in the comments is that expected? (joomla/joomla-cms#36358)

@zero-24 zero-24 deleted the patch-1 branch December 20, 2021 14:56
@jrfnl
Copy link
Contributor

jrfnl commented Dec 20, 2021

@zero-24 Similar to #708.

@mblaney
Copy link
Member

mblaney commented Dec 24, 2021

yeah my bad fixed the version and released 1.3.3

@zero-24
Copy link
Author

zero-24 commented Jan 5, 2022

No problem thanks for your help @mblaney I have just updated my PR to 1.3.3 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants