[
https://issues.apache.org/jira/browse/CXF-8690?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17520875#comment-17520875
]
Andriy Redko edited comment on CXF-8690 at 4/12/22 1:28 AM:
------------------------------------------------------------
[~Fanyi] thank you for good suggestions, we could target some refactorings in
scope of 4.0.0, where the breaking changes are possible. Since most of the
classes are public, we should restrain from moving them around (people may use
it).
was (Author: reta):
[~Fanyi] thank you for good suggestions, we could target some refactorings in
scope of 4.0.0, were the breaking changes are possible. Since most of the
classes are public, we should restrain from moving them around (people may use
it).
> Refactor org.apache.cxf.common.util package
> --------------------------------------------
>
> Key: CXF-8690
> URL: https://issues.apache.org/jira/browse/CXF-8690
> Project: CXF
> Issue Type: Bug
> Affects Versions: 3.5.2
> Reporter: Fanyi
> Priority: Major
> Fix For: 3.5.2
>
> Attachments: CXF-CachedClass-java.png,
> CXF-XmlSchemaPrimitiveUtils-java.png
>
>
> *Issue Description*
> Package *__ _org.apache.cxf.common.util_* __ is a promiscuous package, and
> groups together miscellaneous functionalities that might be useful to
> different subsystems. This means the cohesion between classes in the package
> is low. However, since the project has grown larger, this package structure
> is becoming increasingly hard to navigate. We found that Class
> _*CachedClass*_ and *_XmlSchemaPrimitiveUtils_* is not used by classes in
> their package _{*}org.apache.cxf.common.util{*}._ Moreover, there are
> dependencies between the above two classes and other packages.
> Location: The source file can be found at path File
> /core/src/main/java/org/apache/cxf/common/util/CachedClass.java
> /core/src/main/java/org/apache/cxf/common/util/XmlSchemaPrimitiveUtils.java
>
> *Refactoring suggestions*
>
> To avoid making *_org.apache.cxf.common.util_* too big and unmanageable, we
> suggest to move class *_CachedClass_* into package
> \{_}*org.apache.cxf.common.jaxb*{_},and move class
> *_XmlSchemaPrimitiveUtils_* into package
> *{_}org.apache.cxf.jaxrs.model.wadl{_}.*
>
>
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
