Skip to content

Remove non-twig rendering and add back caching

Nick Sellen requested to merge remove-non-twig-code-path into master

What does this MR do?

Previously I made it so passing NO_TWIG as a parameter would use the old way of rendering the template. I changed how the head data is passed into the template and broke the old way. This MR removes the old way completely.

I also added back in the caching as it was removed. I had not seen that the caching lookup was done in lib/inc.php.

How confident are you it won't break things if deployed?

Reasonably so (aside from issues with the templating itself).

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s)) this is building on / reverting other changes really
Edited by Chris Oelmueller

Merge request reports