Problem/Motivation

As described here there will be a core change that will make D8 compatible with PHP7 and the new Types that come with it. Now all old usages of those Classes have to be replaced with their equivalent as described in the issue.

Proposed resolution

Change all occurrences of relevant classes to their new equivalents.

Remaining tasks

None

User interface changes

None

API changes

None

CommentFileSizeAuthor
#1 stringReplaceNumericAmend_2471587_1.patch1.9 KBLKS90
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LKS90’s picture

LKS90 CreditAttribution: LKS90 at MD Systems GmbH commented
Status: Needs work » Needs review
FileSize
stringReplaceNumericAmend_2471587_1.patch1.9 KB

This patch replaces all occurrences of the String class with the SafeMarkup equivalent. It also changes one class so it extends from NumericArgument (the new equivalent to Numeric which was used before). All tests are still passing.

joachim’s picture

joachim CreditAttribution: joachim commented
Status: Needs review » Fixed

Very much appreciated -- you've probably saved me what could have been a fair bit of headdesking over why Flag would have crashed the next time I updated my copy of core! :D

Committed.

  • joachim committed 9f8ffc6 on 8.x-4.x authored by LKS90 
    Issue #2471587 by LKS90: Updated core classes that have changed for PHP7...

Status: Fixed » Closed (fixed)

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