Have pieforms to always throw an exception if called after smarty() call
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mahara |
Confirmed
|
Wishlist
|
Unassigned |
Bug Description
Sometimes when trying to add a form to a page we get an error similar to this:
Pieform element xxxxxxx wants to add headdata to the page, but it's too late. Build your form before generating the page.
It is meaning we need to generate the form before getting to the call
$smarty = smarty();
It is due to some pieform fields needing to add content to the <head> of a page via the pieform_
However, some other pieform fields do not have this problem, and so can cause confusion for developers
It would be better to force all pieform content to be generated before the smarty() call for consistency
Changed in mahara: | |
importance: | Medium → Wishlist |
milestone: | 19.04.0 → none |
tags: | added: refactoring |