See:

#2796169: Using a better supported encryption library than mcrypt (Required for PHP 7.2 support)
#2946164: Remove usage of deprecrated each() function for PHP 7.2+ future proofing
#2907166: Warning: sourceDataForm() expected to be a reference

CommentFileSizeAuthor
#11 panopoly_core-migrate-php7-compat-2948744-11.patch565 bytescboyden
#9 panopoly_core-migrate-php7-compat-2948744-9.patch807 bytesbyronveale
#8 panopoly_core-migrate-php7-compat-2948744-8.patch1 KBbkosborne
#7 panopoly_core-migrate-php7-compat-2948744-7.patch807 bytesbkosborne
#6 panopoly_core-migrate-php7-compat-2948744-6.patch804 bytesbkosborne
#5 panopoly_core-migrate-php7-compat-2948744-5.patch905 bytesbkosborne
#2 panopoly_core-migrate-php7-compat-2948744.patch560 bytesbkosborne

Comments

bkosborne created an issue. See original summary.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Status: Active » Needs review
StatusFileSize
new panopoly_core-migrate-php7-compat-2948744.patch560 bytes
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Title: Patch migrate to make it compitble with PHP 7 » Upgrade & patch migrate module to make it compatible with PHP 7
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Issue summary: View changes
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
StatusFileSize
new panopoly_core-migrate-php7-compat-2948744-5.patch905 bytes
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
Issue summary: View changes
StatusFileSize
new panopoly_core-migrate-php7-compat-2948744-6.patch804 bytes

Added another issue that I found.

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
StatusFileSize
new panopoly_core-migrate-php7-compat-2948744-7.patch807 bytes

Ugh - typo

bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented
StatusFileSize
new panopoly_core-migrate-php7-compat-2948744-8.patch1 KB

And another...

byronveale’s picture

byronveale commented
StatusFileSize
new panopoly_core-migrate-php7-compat-2948744-9.patch807 bytes

Re-rolled patch from comment #8 against current 7.x-1.x-dev (1.54 tag).

cboyden’s picture

cboyden commented
Related issues: +#3007610: Update to Migrate 7.x-2.11

There's been a Migrate release and another Panopoly issue to update, see #3007610: Update to Migrate 7.x-2.11.

cboyden’s picture

cboyden commented
StatusFileSize
new panopoly_core-migrate-php7-compat-2948744-11.patch565 bytes

Updated patch to build the 2.11 version of Migrate.

dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented

Here's a test build on Travis:

https://travis-ci.org/panopoly/panopoly/builds/445347038

  • dsnopek committed c2a6957 on 7.x-1.x 
    Update Panopoly Core for Issue #2948744 by bkosborne, cboyden,...
dsnopek’s picture

dsnopek
he/him
Primary language English
Location USA
commented
Status: Needs review » Fixed

Tests pass. I also did manual testing locally with the demo data - everything seemed fine! Committed :-)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.