Tag List Report

The following document contains the listing of user tags found in the code. Below is the summary of the occurrences per tag.

Tag ClassTotal number of occurrencesTag strings used by tag class
@todo8@todo
TODO74TODO

Each tag is detailed below:

@todo

Number of occurrences found in the code: 8

org.apache.maven.DefaultArtifactFilterManagerLine
this should probably be a component with some dynamic control of filtering39
org.apache.maven.artifact.DependencyResolutionRequiredExceptionLine
it may be better for artifact.getFile() to throw it - perhaps it is a runtime exception?27
org.apache.maven.artifact.repository.MavenArtifactRepositoryLine
need to URL decode for spaces?233
org.apache.maven.artifact.repository.metadata.RepositoryMetadataLine
not happy about the store method - they use "this"30
org.apache.maven.artifact.resolver.filter.ExcludesArtifactFilterLine
I think this is equiv. to exclusion set filter in maven-core31
org.apache.maven.project.DuplicateArtifactAttachmentExceptionLine
Make this a checked exception, and modify the API of MavenProjectHelper. Currently, this modification would create compatibility problems for existing plugins.30
org.apache.maven.project.MavenProjectLine
I would like to get rid of this. jvz.337
org.apache.maven.project.artifact.ActiveProjectArtifactLine
I think this exposes a design flaw in that the immutable and mutable parts of an artifact are in one class and should be split. ie scope, file, etc depend on the context of use, whereas everything else is immutable.42

TODO

Number of occurrences found in the code: 74

org.apache.maven.AbstractMavenLifecycleParticipantLine
This is too early for build extensions, so maybe just remove it?53
org.apache.maven.DefaultArtifactFilterManagerLine
configure this from bootstrap or scan lib104
org.apache.maven.DefaultMavenLine
Need a general way to inject standard properties183
optimize for the single project or no project228
org.apache.maven.artifact.repository.MavenArtifactRepositoryLine
completely separate local and remote artifact repositories31
org.apache.maven.artifact.repository.metadata.AbstractRepositoryMetadataLine
not sure that it should assume this, maybe the calls to addMetadata should pre-merge, then artifact replaces?195
not sure that it should assume this, maybe the calls to addMetadata should pre-merge, then artifact replaces?202
org.apache.maven.artifact.resolver.ResolutionListenerLine
Use the following two instead of manageArtifact86
Remove ResolutionListenerDM interface87
org.apache.maven.artifact.resolver.ResolutionNodeLine
if it was null, we really need to go find them now... or is this taken care of by the ordering?201
org.apache.maven.artifact.resolver.filter.IncludesArtifactFilterLine
what about wildcards? Just specifying groups? versions?53
org.apache.maven.execution.DefaultMavenExecutionRequestLine
this does not belong here.956
org.apache.maven.lifecycle.DefaultLifecycleExecutorLine
This whole method could probably removed by injecting lifeCyclePluginAnalyzer straight into client site.100
But for some reason the whole plexus appcontext refuses to start when I try this.101
org.apache.maven.lifecycle.DefaultLifecyclesLine
The configuration for the lifecycle needs to be externalized so that I can use the annotations properly for the wiring and reference and external source for the lifecycle configuration.41
org.apache.maven.lifecycle.MavenExecutionPlanLine
lifecycles being executed36
what runs in each phase37
plugins that need downloading38
project dependencies that need downloading39
unfortunately the plugins need to be downloaded in order to get the plugin.xml file. need to externalize this from the plugin archive.40
this will be the class that people get in IDEs to modify42
org.apache.maven.lifecycle.internal.DefaultLifecycleExecutionPlanCalculatorLine
take repo mans into account as one may be aggregating prefixes of many655
collect at the root of the repository, read the one at the root, and fetch remote if something is missing or the user forces the issue656
org.apache.maven.lifecycle.internal.DependencyContextLine
From a concurrency perspective, this class is not good. The combination of mutable/immutable state is not nice38
org.apache.maven.lifecycle.internal.GoalTaskLine
From a concurrecy perspective, this class is not good. The combination of mutable/immutable state is not nice25
org.apache.maven.lifecycle.internal.MojoDescriptorCreatorLine
take repo mans into account as one may be aggregating prefixes of many239
collect at the root of the repository, read the one at the root, and fetch remote if something is missing or the user forces the issue240
org.apache.maven.lifecycle.internal.ProjectBuildListLine
Optimize; or maybe just rewrite the whole way aggregating mojos are being run.54
org.apache.maven.lifecycle.internal.TaskSegmentLine
Consider throwing UnsupprtedSomething on hashCode/equals72
org.apache.maven.plugin.DefaultBuildPluginManagerLine
the antrun plugin has its own configurator, the only plugin that does. might need to think about how that works33
remove the coreArtifactFilterManager34
pluginDescriptor classRealm and artifacts are set as a side effect of this call, which is not nice.162
org.apache.maven.plugin.PluginManagerTestLine
These two tests display a lack of symmetry with respect to the input which is a free form string and the mojo descriptor which comes back. All the free form parsing needs to be done somewhere else, this is really the function of the CLI, and then the pre-processing of that output still needs to be fed into a hinting process which helps flesh out the full specification of the plugin. The plugin manager should only deal in concrete terms -- all version finding mumbo jumbo is a customization to base functionality the plugin manager provides.105
turn an equivalent back on when the RR plugin is released.115
this will be the basis of the customizable lifecycle execution so need to figure this out quickly.140
org.apache.maven.plugin.PluginParameterExpressionEvaluatorLine
used by the Enforcer plugin51
don't catch exception187
don't catch exception227
don't catch exception259
don't catch exception316
without #, this could just be an evaluate call...371
Copied from the DefaultInterpolator. We likely want to resurrect the PathTranslator or at least a similar component for re-usage415
org.apache.maven.plugin.PluginParameterExpressionEvaluatorTestLine
Auto-generated method stub440
org.apache.maven.plugin.internal.DefaultMavenPluginManagerLine
could the configuration be passed to lookup and the configurator known to plexus via the descriptor so that this method could entirely be handled by a plexus lookup?555
org.apache.maven.project.MavenProjectLine
Find a way to use <relativePath/> here...it's tricky, because the moduleProject usually doesn't have a file associated with it yet.265
This is suboptimal. Without a cache in the project builder, rebuilding the parent chain currently causes O(n^2) parser invocations for an inheritance hierarchy of depth n.348
let the scope handler deal with this513
classpath check doesn't belong here - that's the other method531
let the scope handler deal with this534
let the scope handler deal with this558
this checking for file == null happens because the resolver has been confused about the root artifact or not. things like the stupid dummy artifact coming from surefire.576
classpath check doesn't belong here - that's the other method605
let the scope handler deal with this653
classpath check doesn't belong here - that's the other method670
let the scope handler deal with this673
let the scope handler deal with this697
let the scope handler deal with this726
classpath check doesn't belong here - that's the other method743
let the scope handler deal with this746
let the scope handler deal with this770
this should not be allowed to be null.841
org.apache.maven.project.ProjectSorterLine
!![jc; 28-jul-2005] check this; if we're using '-r' and there are aggregator tasks, this will result in weirdness.235
org.apache.maven.project.artifact.MavenMetadataSourceLine
This prevents a cycle in the composition which shows us another problem we need to deal with.96
Not even sure this is really required as the project will be cached in the builder, we'll see this is currently the biggest hotspot180
shouldn't the versions be merged across relocations?663
org.apache.maven.project.artifact.MavenMetadataSourceTestLine
restore these if it makes sense93
org.apache.maven.repository.TestRepositoryConnectorLine
Auto-generated method stub111
org.apache.maven.repository.TestRepositorySystemLine
Auto-generated method stub172
Auto-generated method stub292
org.apache.maven.settings.DefaultMavenSettingsBuilderLine
This replacing shouldn't be necessary as user.home should be in the context of the container and thus the value would be interpolated by Plexus129
org.apache.maven.toolchain.DefaultToolchainManagerLine
why is this using the context90
org.apache.maven.toolchain.RequirementMatcherFactoryLine
error reporting90
org.apache.maven.toolchain.ToolchainFactoryLine
keep around?? */39
org.apache.maven.toolchain.java.DefaultJavaToolchainFactoryLine
possibly move at least parts to a utility method or abstract implementation.78