Skip to content
This repository was archived by the owner on Jun 10, 2019. It is now read-only.

Does 1.9.3.10 include this (or Magento's) PHP 7.2 fixes? #151

Open
durzel opened this issue Sep 20, 2018 · 4 comments
Open

Does 1.9.3.10 include this (or Magento's) PHP 7.2 fixes? #151

durzel opened this issue Sep 20, 2018 · 4 comments

Comments

@durzel
Copy link
Contributor

durzel commented Sep 20, 2018

Apologies the indirect line of enquiry...

The README says that Magento have released a patch for PHP 7.2 support, which depreciates this extension. It is not clear however whether Magento 1.9.3.10 - released on on 18th September - also includes these fixes? I can't see an obvious reason why they wouldn't be given that this extension plays perfectly nicely with core.

I haven't had the opportunity to investigate myself yet, am just conscious about upgrading to 1.9.3.10 and assuming that the fixes are rolled in, or likewise assuming they are not.

@kanevbg
Copy link

kanevbg commented Sep 20, 2018

I have done the checks - the support for PHP7 is not included in 1.9.3.10

@matinfo
Copy link

matinfo commented Sep 20, 2018

Yes, is two different thing :-)

The newly release 1.9.3.10 include all the last security fixe. The PHP7.2 patch make magento working with PHP7.2. Not every platform using 1.9.3.10 use PHP7.2!

@durzel
Copy link
Contributor Author

durzel commented Sep 20, 2018

Thanks. It does seem odd, particularly as Magento haven't listed 1.9.3.10 on the list of versions that require SUPEE patches (1.9.3.9 is listed and says it needs no patches). The implication - to me - is that the PHP 7 support was rolled into 1.9.3.10.

Thanks for clarifying.

@udovicic
Copy link
Member

Hi,
Patch was created before the 1.9.3.10 was created, but was not included in that release. It will be part of core starting from 1.9.4.0 and on.
As for the naming of the patch, you should use the same one as for 1.9.3.9. Unfortunately, that was not renamed or noted before publishing it to Magento download pages, but has been confirmed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants