fixed EventHandlerRegistry to hide all ConcurrentMap assignments to the EventHandlerRegistry constructor
added singleton to the EventHandlerRegistry
renamed all getRecords,updateRecords,deleteRecords,CreateTables member objects to comply with the project's standards for constants
fixed the SQL Injection vulnerability by changing the sql queries from adding the objects non-parametrized to parametrized
splitted repetitive sub-strings into a one object string
fixed a direct return value from a variable, by removing the variable object
changed the occurences from using .peek( (used for debugging) to map( (non-debugging)
removed Helpers.java
added custom SqlExecutionExcepions.java file, instead of using RuntimeException
fixed Login bug type, where if the second user tries to login by providing any incorrect user credentials, he would be successfully logged in with message of Incorrect email/password
Checklist
[x] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added manual/automated tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
Description
EventHandlerRegistry
tohide
all ConcurrentMap assignments to the EventHandlerRegistry constructorgetRecords,updateRecords,deleteRecords,CreateTables
member objects to comply with the project's standards for constants.peek(
(used for debugging) tomap(
(non-debugging)Helpers.java
SqlExecutionExcepions.java
file, instead of usingRuntimeException
Login
bug type, where if the second user tries to login by providing any incorrect user credentials, he would be successfully logged in with message ofIncorrect email/password
Checklist
Remark
For automation please see closing-issues-using-keywords