- Issue created by @catch
- Merge request !8635Deprecate the page variable and remove documentation for it. → (Open) created by catch
- Status changed to Needs work
5 months ago 4:35pm 2 July 2024 - First commit to issue fork.
- 🇬🇧United Kingdom longwave UK
Please see https://www.drupal.org/node/3334622 → and 📌 Provide a mechanism for deprecation of variables used in twig templates Fixed for how to deprecate Twig variables.
I realised this wasn't documented in the deprecation policy so also opened 📌 Document how to deprecate Twig variables Active to fix that.
- First commit to issue fork.
- 🇮🇳India ankitv18
ankitv18 → changed the visibility of the branch 3458589-deprecate-variablespage-for to hidden.
- 🇮🇳India Sahana _N
Hi
It looks much better. The patch is applied cleanly and I even searched in the Drupal code base for $variables[‘page]. I did not find it. So for me, it looks good.
RTBC ++
- Status changed to Needs review
2 months ago 4:54am 13 September 2024 - 🇳🇿New Zealand quietone
There are failing tests due to get deprecation messages from this deprecation.
The variable is still in use and it looks like it needs to be set to TRUE or FALSE. Can this really be deprecated or do I just not know something?
- Status changed to Needs work
2 months ago 3:48pm 13 September 2024