FelipeM525 opened a new pull request, #9394:
URL: https://github.com/apache/cloudstack/pull/9394

   ### Description
   
   The class `VolumeVO` lacks access modifiers in its fields. This PR aims to 
improve adherence to object-oriented programming by adding private access 
modifiers to all fields in the class mentioned above, along with their 
respective getters and setters. I've also added a getter to an occurrence in 
`VolumeApiServiceImpl` where a field belonging to `VolumeVO` was being accessed 
directly.
   
   ### Types of changes
   
   - [ ] Breaking change (fix or feature that would cause existing 
functionality to change)
   - [ ] New feature (non-breaking change which adds functionality)
   - [ ] Bug fix (non-breaking change which fixes an issue)
   - [ ] Enhancement (improves an existing feature and functionality)
   - [x] Cleanup (Code refactoring and cleanup, that may add test cases)
   - [ ] build/CI
   
   #### Feature/Enhancement Scale
   
   - [ ] Major
   - [x] Minor
   
   ### How Has This Been Tested?
   I ran all the tests related to `VolumeVO` and `VolumeApiServiceImpl`, 
checked all usages of both classes, and made sure nothing was broken.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to