| Package | Notes of packages which should not be there | Classes from illegal package | Decision |
|---|
| 1 | org.onap.cps.ncmp.rest
| | Code Block |
|---|
org.onap.cps.spi, |
| | Code Block |
|---|
ConditionProperties
ModuleDefinition
ModuleReference
DataNode
DataNodeNotFoundException
AlreadyDefinedException
CpsException
DataValidationException |
| CPS Utils can be used everywhere in the project
Apart from ConditionProperties the use of the classes is OK.
Just allow spi.model and spi.exceptions
Models and exceptions should be moved under the ‘api’ package. And a separate package should be made for options and CascadeDeleteAllowed called ‘parameters’. |
| 2 | | Code Block |
|---|
org.onap.cps.ncmp.impl |
| | Code Block |
|---|
NetworkCmProxyFacade
LockReasonCategory.LOCKED_MISBEHAVING |
| The Facade class should be split into an interface in the api package and an actually implementation in the impl. package. LockReason category should be moved under the api package in NCMP Service.
The resulting error should be examined by someone |
| 3 | org.onap.cps.ncmp.api
| | Code Block |
|---|
org.onap.cps.ncmp.impl |
| | Code Block |
|---|
CmHandleQueryService
CmHandleRegistrationService
InventoryPersistence
ParameterizedCmHandleQueryService
CmHandleQueryConditions
InventoryQueryConditions
YangModelCmHandle
TrustLevelManager
AlternateIdMatcher
YangDataConverter |
| The Facade class should be split into an interface in the api package and an actually implementation in the impl. package. |
| 4 | | Code Block |
|---|
org.onap.cps.ncmp.config |
| | Code Block |
|---|
CpsApplicationContext |
| Needs further investigation, but probably fine.
it is used in CmResourceAddress class. |
| 5 | | Code Block |
|---|
org.onap.cps.spi |
| | Code Block |
|---|
ModuleDefinition
ModuleReference |
| This should be fine. look at the comments above! |
| 6 | org.onap.cps.ncmp.impl
| | Code Block |
|---|
org.onap.cps.cpspath |
| | Code Block |
|---|
CpsPathUtil
PathParsingException |
| Cps Path module should be declared as a common module for all. |
| 7 | | Code Block |
|---|
org.onap.cps.events |
| | Code Block |
|---|
EventsPublisher |
| This should be kept in place to not duplicate code in the projects. An exception should be made here. |
| 8 | | Code Block |
|---|
org.onap.cps.impl |
| | org/onap/cps/impl/utils/CpsValidator.java interface should be moved to org/onap/cps/utils
|
| 9 | | Code Block |
|---|
org.onap.cps.spi |
| | Code Block |
|---|
DataNode
FetchDescendantsOption
DataValidationException
ModuleDefinition
ModuleReference
DataNode
DataNodeNotFoundException
AlreadyDefinedException
CpsException |
| FetchDescendantsOption should be moved into ‘spi.api’
along with PaginationOption and CascadeDeleteAllowed.
The rest of the classes is OK here
|