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 stringReplace_2471577_1.patch1.92 KBLKS90
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

LKS90’s picture

LKS90 CreditAttribution: LKS90 at MD Systems GmbH commented
FileSize
stringReplace_2471577_1.patch1.92 KB

This patch removes all occurrences. It also adds a linebreak according to Coding standards. Why it says that I removed and added the same line, I don't know.

LKS90’s picture

LKS90 CreditAttribution: LKS90 at MD Systems GmbH commented
Status: Needs work » Needs review
Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Needs review » Reviewed & tested by the community

Looks good.

Garrett Albright’s picture

Garrett Albright CreditAttribution: Garrett Albright commented
Status: Reviewed & tested by the community » Closed (fixed)

Committed. Thanks for being on top of this.

Status: Closed (fixed) » Needs work

The last submitted patch, 1: stringReplace_2471577_1.patch, failed testing.

miro_dietiker’s picture

miro_dietiker
Location Switzerland
CreditAttribution: miro_dietiker at MD Systems GmbH commented
Status: Needs work » Closed (fixed)