| Description | 
- Folder name should be name + type from config, not name of zip file
 
- Unique constraint name + type should be in db too (not only AR rules)
 
- Proper OOP with survey themes, question themes, in controller (ThemeOptionsInterface?) instead of if-statements
 
- Discuss proper indentation of PHP files
 
- When using json_encode, always check json_last_error for errors
 
- <type> in question theme config conflicts with <type> in limestore config: https://manual.limesurvey.org/LimeStore_extension_development
 
- It would be good if reading XML from file was only used in ONE place in the code, to reduce the risk of forgetting libxml_disable_entity_loader
 
- When uploading theme, don't branch on $_POST, but on config.xml <type> tag
 
- Move all core question themes to one folder (now split between views/ and themes/)
 
  | 
|---|