The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.
Tag | Total number of occurrences |
---|---|
@todo | 13 |
TODO | 22 |
Each tag is detailed below:
Number of occurrences found in the code: 13
org.apache.maven.artifact.Artifact | Line |
---|---|
do we really need an interface here? | 38 |
get rid of the multiple states we can have (project, parent, etc artifacts, file == null, snapshot, etc) - construct subclasses and use accordingly? | 39 |
would like to get rid of this - or at least only have one. Base version should be immutable. | 96 |
remove, a quick hack for the lifecycle executor | 157 |
org.apache.maven.artifact.DefaultArtifact | Line |
this should possibly be replaced by type handler | 42 |
should be final | 54 |
org.apache.maven.artifact.DependencyResolutionRequiredException | Line |
it may be better for artifact.getFile() to throw it - perhaps it is a runtime exception? | 27 |
org.apache.maven.artifact.metadata.ArtifactMetadata | Line |
merge with artifactmetadatasource | 30 |
retrieval exception not appropriate for store | 31 |
this should only be needed on the repository metadata | 72 |
this should only be needed on the repository metadata | 81 |
org.apache.maven.artifact.resolver.ArtifactResolver | Line |
possibly fix the signatures, it's unfortunate that in some methods the local repo is listed first and second in others. | 38 |
org.apache.maven.artifact.resolver.filter.ExcludesArtifactFilter | Line |
I think this is equiv. to exclusion set filter in maven-core | 31 |
Number of occurrences found in the code: 22
org.apache.maven.artifact.Artifact | Line |
---|---|
into scope handler | 50 |
org.apache.maven.artifact.factory.ArtifactFactory | Line |
deprecate and chase down (probably used for copying only) | 29 |
org.apache.maven.artifact.factory.DefaultArtifactFactory | Line |
remove, it doesn't know the ones from the plugins | 31 |
can refactor - inherited scope calculation belongs in the collector, use scope handler | 123 |
org.apache.maven.artifact.resolver.DefaultArtifactCollector | Line |
we probably want to warn the user that he is building an artifact with different values than in dependencyManagement | 130 |
Does this check need to happen here? Had to add the same call below when we iterate on child nodes -- will that suffice? | 152 |
shouldn't need to double up on this work, only done for simplicity of handling recommended version but the restriction is identical | 174 |
ick. this forces the OCE that should have come from the previous call. It is still correct | 177 |
use as conflict resolver(s), chain | 239 |
should this be part of mediation? previous one is more dominant | 241 |
maybe its better to just pass the range through to retrieval and use a transformation? | 340 |
might be better to have source.retrieve() throw a specific exception for this situation and catch here rather than have it return null | 411 |
org.apache.maven.artifact.resolver.DefaultArtifactCollectorTest | Line |
use newest conflict resolver | 155 |
use newest conflict resolver | 175 |
org.apache.maven.artifact.resolver.ResolutionListener | Line |
Use the following two instead of manageArtifact | 85 |
Remove ResolutionListenerDM interface | 86 |
org.apache.maven.artifact.resolver.ResolutionNode | Line |
if it was null, we really need to go find them now... or is this taken care of by the ordering? | 200 |
org.apache.maven.artifact.resolver.filter.IncludesArtifactFilter | Line |
what about wildcards? Just specifying groups? versions? | 50 |
org.apache.maven.artifact.versioning.VersionRange | Line |
should throw this immediately, but need artifact | 297 |
could be more efficient by sorting the list and then moving along the restrictions in order? | 520 |
org.apache.maven.artifact.versioning.VersionRangeTest | Line |
current policy is to retain the original version - is this correct, do we need strategies or is that handled elsewhere? assertEquals( CHECK_VERSION_RECOMMENDATION, "1.1", mergedRange.getRecommendedVersion().toString() ); | 173 |
test reversed restrictions on all below | 194 |