mirror of https://github.com/apache/cloudstack.git
08f2ec5044
Removed unused code from com.cloud.api.ApiServer**Removed \_ from variables names**: private variables with \_ at the beginning is common in C++ but not in Java. **Removed unused code from ApiServer:** - com.cloud.api.ApiServer.getPluggableServices(): unused method; - com.cloud.api.ApiServer.getApiAccessCheckers(): unused method; **Methods and variables access level reviewed:** - com.cloud.api.ApiServer.handleAsyncJobPublishEvent(String, String ,Object): this method was private but the annotation @MessageHandler requests public methods, as can be seen in org.apache.cloudstack.framework.messagebus.MessageDispatcher.buildHandlerMethodCache(Class\<?\>), which searches methods with the @MessageHandler annotation and changes it to be accessible (setAccessible(true)). Thus, there is no reason for handleAsyncJobPublishEvent be a private method and lead some other dev to wrong conclusions about the use of the method; - Global variables and methods called just by this class (ApiServer) were changed to private. **Changed variables and methods from static to non-static (if possible):** as some variables/methods are used just by one object of this class, instantiated by Spring, they were changed to non-static. With that, calls from com.cloud.api.ApiServlet.ApiServlet() that used static methods from ApiServer, were changed from ApiServer.\<staticMethodName\> to \_apiServer.\<methodName\> that refers to the org.apache.cloudstack.api.ApiServerService interface. Thus, methods com.cloud.api.ApiServer.getJSONContentType() and com.cloud.api.ApiServer.isSecureSessionCookieEnabled() had to be added in the interface (org.apache.cloudstack.api.ApiServerService, interface implemented by class ApiServer). * pr/1263: The goal of this PR is to review com.cloud.api.ApiServer class, with the following actions: Signed-off-by: Will Stevens <williamstevens@gmail.com> |
||
|---|---|---|
| .. | ||
| account | ||
| acl | ||
| agent/manager | ||
| alert | ||
| api | ||
| async | ||
| capacity | ||
| configuration | ||
| consoleproxy | ||
| dc | ||
| deploy | ||
| event | ||
| ha | ||
| hypervisor | ||
| metadata | ||
| network | ||
| projects | ||
| resource | ||
| resourcelimit | ||
| server | ||
| servlet | ||
| storage | ||
| tags | ||
| template | ||
| test | ||
| usage | ||
| user | ||
| uuididentity | ||
| vm | ||