search

oasp / oasp4j

The Open Application Standard Platform for Java
Apache License 2.0
60 stars 303 forks source link

Added prefix 'Abstract' in the name for all the relevant abstract classes. #575

Closed kiran-vadla closed 7 years ago

kiran-vadla commented 7 years ago

Fix for Issue #561 is provided as part of this PR . As a fix . added prefix 'Abstract' in the name for all the relevant abstract classes.

As part of the fix , did not add the prefix to the classes 'PermisssionConstants.java' and 'NamedQueries.java' . Reasons were two-fold : First being , that these are classes where constants are defined . Ideally it should not have been defined 'abstract' . Second reason being , adding the prefix 'Abstract' in the name will have effect on the CobiGen code since the names of these classes are referenced from Cobigen Code and also JUNITs were failing. Apart from these two java files , there were few more framework classes that were referenced from Cobigen Code . Hence this change is not applied to these framework classes as well so as to prevent Cobigen from breaking.

There were couple of abstract classes namely 'BaseTest' and 'BaseWebSecurityConfig' . As part of the fix , these classes are renamed to AbstractTest and AbstractWebSecurityConfig respectively.

Thanks, @kiran-vadla

oasp-ci commented 7 years ago

Can one of the admins verify this patch?

kiran-vadla commented 7 years ago

Since Build is failing, closing this PR . Will be submitting a new PR .