Follow-up to #2568171: Upgrade to Twig 1.22 and implement our own cache class

Problem/Motivation

Made major due to the Fixes for parsing for PHP7
https://github.com/twigphp/Twig/commit/24d9266d0f1c2b4426ad8f931be30718c...

+* 1.24.0 (2016-01-25)
+
+ * adding support for the ?? operator
+ * fixed the defined test when used on a constant, a map, or a sequence
+ * undeprecated _self (should only be used to get the template name, not the template instance)
+ * fixed parsing on PHP7
+
+* 1.23.3 (2016-01-11)
+
+ * fixed typo
+
+* 1.23.2 (2015-01-11)
+
+ * added versions in deprecated messages
+ * made file cache tolerant for trailing (back)slashes on directory configuration
+ * deprecated unused Twig_Node_Expression_ExtensionReference class
+
+* 1.23.1 (2015-11-05)

Proposed resolution

Upgrade to 1.24

Remaining tasks

User interface changes

API changes

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

joelpittet created an issue. See original summary.

joelpittet’s picture

Cottser’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed
Issue tags: +8.2.0 release notes

Committed 45d0a03 and pushed to 8.2.x. Thanks!

  • alexpott committed 45d0a03 on 8.2.x
    Issue #2721351 by joelpittet: Upgrade to Twig 1.24
    
jibran’s picture

Made major due to the Fixes for parsing for PHP7

Shouldn't this qualify this issue for 8.1.x?

Cottser’s picture

I looked at the commit linked in the issue summary and it seems like an edge case to me.

alexpott’s picture

@jibran they've bumped their minor version so I think we should only bump it in a minor too

Status: Fixed » Closed (fixed)

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