Problem/Motivation

The former is deprecated since 1.18.1 which we recently updated to.

Proposed resolution

Use the other class, which seems to be a replacement and is capable of the same

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 2563821-4.patch911 bytesdawehner
#2 2563821-2.patch882 bytesdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner created an issue. See original summary.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented
Title: Twig_Loader_String is deprecated use Twig_Loader_Array instead » Implement Twig_Loader_String for ourselves
Status: Active » Needs review
FileSize
2563821-2.patch882 bytes

There we go.

Status: Needs review » Needs work

The last submitted patch, 2: 2563821-2.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner as a volunteer commented
Status: Needs work » Needs review
FileSize
2563821-4.patch911 bytes

There we go.

jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran as a volunteer commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +Quickfix

Seems like a quick fix.

star-szr’s picture

star-szr
he/him
Primary language English
CreditAttribution: star-szr as a volunteer commented

+1.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick at Acquia commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.0.x. Thanks!

  • webchick committed 5adf95a on 8.0.x 
    Issue #2563821 by dawehner: Implement Twig_Loader_String for ourselves

Status: Fixed » Closed (fixed)

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