4 Commits

Author	SHA1	Message	Date
Miguel Nogueira	4eb115d165	Revert "Apply fixes from StyleCI (pull request #6)" ... This reverts pull request #6. > This pull request applies code style fixes from an analysis carried out by [StyleCI](https://bitbucket.styleci.io). > > For more information, click [here](https://bitbucket.styleci.io/analyses/a2Jl7D).	2020-10-21 00:29:50 +00:00
Miguel Nogueira	773ec570d9	Apply fixes from StyleCI	2020-10-21 00:01:41 +00:00
Miguel Nogueira	5f1f92a9ce	Code review ... This commit fixes some superficial instances of Broken Access Control (https://owasp.org/www-project-top-ten/OWASP_Top_Ten_2017/Top_10-2017_A5-Broken_Access_Control). There may be some more instances of this, as authorization was only done after most of the controllers were done (big mistake). Some refactoring was also performed, where Route Model Binding with DI (dependency injection) was used whenever possible, to increase testability of the codebase. Some reused code was also moved to Helper classes as to enforce DRY; There may be some lines of code that are still copy-pasted from other parts of the codebase for reuse. Non-breaking refactoring changes were made, but the app as a whole still needs full manual testing, and customised responses to HTTP 500 responses. Some errors are also not handled gracefully and this wasn't checked in this commit.	2020-07-16 21:21:28 +01:00
Miguel Nogueira	bca6020ab0	Add ability to edit forms and add new fields ... This commit adds the ability to edit and modify existing forms. On the technical side, it also adds a new reusable validation Facade which helps reduce duplicated code.	2020-07-15 06:48:49 +01:00