Problem/Motivation

Twig 1.25.0 is out. Help contrib - more methods marked deprecated or internal

  • deprecated the ability to store non Node instances in Node::$nodes
  • deprecated Twig_Environment::getLexer(), Twig_Environment::getParser(), Twig_Environment::getCompiler()
  • deprecated Twig_Compiler::getFilename()
  • marked the following methods as being internals on Twig_Environment:
    getFunctions(), getFilters(), getTests(), getFunction(), getFilter(), getTest(),
    getTokenParsers(), getTags(), getNodeVisitors(), getUnaryOperators(), getBinaryOperators(),
    getFunctions(), getFilters(), getGlobals(), initGlobals(), initExtensions(), and initExtension()

Changelog

CommentFileSizeAuthor
update_twig_to_1_25.patch1.55 KBslasher13
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

slasher13 created an issue. See original summary.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Needs review » Reviewed & tested by the community

Thanks, it's nice not having vendor in the repo:)

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Issue tags: +Twig

  • catch committed 83ddf6e on 8.3.x 
    Issue #2804365 by slasher13: Update Twig to 1.25
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.3.x, thanks!

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Version: 8.3.x-dev » 8.2.x-dev
Status: Fixed » Patch (to be ported)

We should discuss whether to cherry pick this back to the release candidate.

catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Version: 8.2.x-dev » 8.3.x-dev
Status: Patch (to be ported) » Fixed

That didn't happen. Leaving fixed against 8.3.x

Status: Fixed » Closed (fixed)

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

xjm’s picture

xjm
she/her
Primary language English
CreditAttribution: xjm at Acquia commented
Issue tags: +8.3.0 release notes