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 manager classes that handle specific entity (and link) forms individually manage these individually, instead of using growing if/else constructs for all forms, before adding functionality in this area.

Back