Project

General

Profile

Feature #1734

Updated by Alexander Watzinger almost 2 years ago

Because of the generic form functions, which is generally a good approach to manage the complex model relations, we ended up with very long if/else constructs that can get quite confusing. 
 This concerns especially the functions at *openatlas/forms*: 
 * process_form_data() 
 * process_origin_data() 
 * populate_update_form() 
 * additional_fields() 

 After testing a new approach with similar software we decided to remedy this with creating form classes that manage these individually, instead of using a growing if/else constructs for all forms, before adding functionality in this area.

Back