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

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.