Replace unmaintained Dwoo templating engine

Bug #1890217 reported by Rebecca Blundell
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mahara
Confirmed
High
Doris Tam

Bug Description

The Dwoo templating engine, which we use to process all our Smarty templates is no longer maintained. The website has been taken down and the Github project is archived: https://github.com/dwoo-project/dwoo

This may start to cause problems for us as PHP moves on and Dwoo does not keep up with the new code. Currently it has been unmaintained since 2018. It looks like Dwoo was developed to make Smarty faster, but perhaps it is no longer faster. Smarty is at least patched for the latest PHP versions.

We need to investigate what to do next in order to not be relying on an unmaintained library for all our templating. We can probably patch it ourselves for a limited time, but will need to move on from Dwoo.

Tags: refactoring
Revision history for this message
Kristina Hoeppner (kris-hoeppner) wrote :

This was discussed in the 81st developer meeting. Minutes are at https://wiki.mahara.org/wiki/Developer_Area/Developer_Meetings/81

Earliest that this can be looked into: After the 20.10 release.

Changed in mahara:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Cecilia Vela Gurovic (ceciliavg)
tags: added: refactoring
Revision history for this message
Doris Tam (doristam) wrote :

Quoted from the 81st developer meeting "It was mentioned that changing to Smarty templates would be the easiest as Dwoo was a fork of Smarty back in the past and so much of the template content would not change."

Switching to Smarty sounds like the next step

Changed in mahara:
assignee: Cecilia Vela Gurovic (ceciliavg) → nobody
Revision history for this message
Doris Tam (doristam) wrote :

Fun fact, the template language for PHP called Twig is already being used in Mahara for behat-html-formatter

> ./composer.phar why twig/twig
emuse/behat-html-formatter v0.1.0 requires twig/twig (~1.0)

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "main" branch: https://reviews.mahara.org/12286

Doris Tam (doristam)
Changed in mahara:
assignee: nobody → Doris Tam (doristam)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.