openkm / document-management-system

OpenKM is a Open Source Document Management System
https://www.openkm.com/
GNU General Public License v2.0
671 stars 297 forks source link

Refactoring #319

Closed Dharminsinh closed 2 years ago

Dharminsinh commented 2 years ago
  1. Rename method/variable:-  Location:- document-management-system\src\main\java\com\openkm\cmis\CmisRepository.java
  2. Rename method/variable:-  Location:- document-management-system\src\main\java\com\openkm\cmis\CmisRepository.java
  3. Extract method:-  Location:- document-management-system\src\main\java\com\openkm\automation\AutomationManager.java
  4. Pull-up variable/method:-  Location:- document-management-system\src\main\java\com\openkm\frontend\client\extension\widget\userinfo\UserInfoExtension.java
  5. Pull-up variable/method:-  Location:- document-management-system\src\main\java\com\openkm\frontend\client\extension\widget\userinfo\UserInfoExtension.java
  6. Extract class:-  Location:- document-management-system\src\main\java\com\openkm\cmis\DefinitionImpl.java
  7. Push-down variable/method:-  Location:- document-management-system\src\main\java\com\openkm\dao\PluginDAO.java
darkman97i commented 2 years ago

@Dharminsinh We will not accept changes in this manner, here is a mixing of several things what should taken in consideration in a separated manner ( each pull request should corresponds to a single topic, not several mixed into one ):

You are welcome and we appreciate your efforts, but please separe the pull request by topic