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 Class | Total number of occurrences | Tag strings used by tag class |
|---|---|---|
| Deprecated Work | 68 | @deprecated |
| FIXME Work | 1 | fixme, @fixme |
| Todo Work | 90 | todo, @todo |
Each tag is detailed below:
Deprecated Work
Number of occurrences found in the code: 68
| org.apache.maven.DefaultMaven | Line |
|---|---|
| If you use this method and your code is not in Maven Core, stop doing this. | 408 |
| org.apache.maven.artifact.handler.DefaultArtifactHandler | Line |
| This ctor is present only for Plexus XML defined component compatibility, do not use it. | 47 |
| org.apache.maven.classrealm.ClassRealmRequest | Line |
| Use {@link #getParentImports()} instead. | 70 |
| org.apache.maven.execution.DefaultMavenExecutionRequest | Line |
| use {@link #setPom(File)} | 591 |
| org.apache.maven.execution.MavenExecutionRequest | Line |
| use {@link #setTopDirectory(Path)} instead | 96 |
| use {@link #getTopDirectory()} instead | 102 |
| Since Maven 4: use {@link #getProjectActivation()}. | 161 |
| Since Maven 4: use {@link #getProjectActivation()}. | 167 |
| Since Maven 4: use {@link #getProjectActivation()}. | 176 |
| Since Maven 4: use {@link #getProjectActivation()}. | 184 |
| Since Maven 4: use {@link #getProfileActivation()}. | 326 |
| Since Maven 4: use {@link #getProfileActivation()}. | 332 |
| Since Maven 4: use {@link #getProfileActivation()}. | 338 |
| Since Maven 4: use {@link #getProfileActivation()}. | 345 |
| Since Maven 4: use {@link #getProfileActivation()}. | 351 |
| Since Maven 4: use {@link #getProfileActivation()}. | 357 |
| Since Maven 4: use {@link #getProfileActivation()}. | 363 |
| Since Maven 4: use {@link #getProfileActivation()}. | 370 |
| Since 3.9 there is no direct Maven2 interop offered at LRM level. See <a href="https://maven.apache.org/resolver/configuration.html">Resolver Configuration</a> page option {@code aether.artifactResolver.simpleLrmInterop} that provides similar semantics. This method should be never invoked, and always returns {@code false}. | 489 |
| Since 3.9 there is no direct Maven2 interop offered at LRM level. See <a href="https://maven.apache.org/resolver/configuration.html">Resolver Configuration</a> page option {@code aether.artifactResolver.simpleLrmInterop} that provides similar semantics. This method should be never invoked, and ignores parameter (value remains always {@code false}). | 499 |
| use {@link #setRootDirectory(Path)} instead | 540 |
| use {@link #getRootDirectory()} instead | 547 |
| org.apache.maven.execution.MavenSession | Line |
| use {@link #getTopDirectory()} ()} | 147 |
| This appears not to be used anywhere within Maven itself. | 294 |
| Use either {@link #getUserProperties()} or {@link #getSystemProperties()}. | 458 |
| org.apache.maven.execution.ProfileActivation | Line |
| Use {@link #getRequiredActiveProfileIds()} and {@link #getOptionalActiveProfileIds()} instead. | 40 |
| Use {@link #getRequiredInactiveProfileIds()} and {@link #getOptionalInactiveProfileIds()} instead. | 49 |
| Use {@link #activateOptionalProfile(String)} or {@link #activateRequiredProfile(String)} instead. | 59 |
| Use {@link #deactivateOptionalProfile(String)} or {@link #deactivateRequiredProfile(String)} instead. | 70 |
| org.apache.maven.execution.ProjectActivation | Line |
| Use {@link #getRequiredActiveProjectSelectors()} and {@link #getOptionalActiveProjectSelectors()} instead. | 107 |
| Use {@link #getRequiredInactiveProjectSelectors()} and {@link #getOptionalInactiveProjectSelectors()} instead. | 117 |
| Use {@link #activateOptionalProject(String)} or {@link #activateRequiredProject(String)} instead. | 128 |
| Use {@link #deactivateOptionalProject(String)} or {@link #deactivateRequiredProject(String)} instead. | 140 |
| org.apache.maven.graph.DefaultGraphBuilder | Line |
| Use {@link #DefaultGraphBuilder(BuildResumptionDataRepository, PomlessCollectionStrategy, MultiModuleCollectionStrategy, RequestPomCollectionStrategy)} instead or rely on JSR 330 | 73 |
| org.apache.maven.graph.DefaultProjectDependencyGraph | Line |
| Use {@link #DefaultProjectDependencyGraph(Collection, Collection)} instead. | 73 |
| org.apache.maven.internal.aether.DefaultRepositorySystemSessionFactory | Line |
| Renamed to {@link #MAVEN_RESOLVER_TRANSPORT_APACHE} | 149 |
| org.apache.maven.lifecycle.DefaultLifecycles | Line |
| Use {@link #DefaultLifecycles(LifecycleRegistry,Lookup)} instead | 65 |
| org.apache.maven.lifecycle.internal.DefaultMojoExecutionConfigurator | Line |
| Do not use this ctor directly, inject this component instead. | 63 |
| org.apache.maven.lifecycle.internal.stub.LifecycleExecutionPlanCalculatorStub | Line |
| instead use {@link #getNumberOfExecutions(ProjectBuildList)} | 96 |
| org.apache.maven.lifecycle.mapping.DefaultLifecycleMapping | Line |
| use lifecycles instead | 38 |
| Should not be used in Java code. | 46 |
| org.apache.maven.plugin.CacheUtils | Line |
| Use {@link Objects#equals(Object)} | 34 |
| Use {@link Objects#hashCode(Object)} | 42 |
| org.apache.maven.plugin.DebugConfigurationListener | Line |
| Use {@link #DebugConfigurationListener(Logger)} instead. | 36 |
| org.apache.maven.plugin.MavenPluginManager | Line |
| Use {@link #checkPrerequisites(PluginDescriptor)} instead. | 73 |
| org.apache.maven.plugin.PluginConfigurationException | Line |
| Use {@link #PluginConfigurationException(PluginDescriptor, String, Throwable)} | 48 |
| Use {@link #PluginConfigurationException(PluginDescriptor, String, Throwable)} | 59 |
| Use {@link #PluginConfigurationException(PluginDescriptor, String, Throwable)} | 70 |
| org.apache.maven.plugin.PluginContainerException | Line |
| Use {@link #PluginContainerException(Plugin, ClassRealm, String, Throwable)} | 64 |
| Use {@link #PluginContainerException(Plugin, ClassRealm, String, Throwable)} | 75 |
| org.apache.maven.plugin.PluginManagerException | Line |
| Left for binary compatibility. | 101 |
| Left for binary compatibility. | 111 |
| Left for binary compatibility. | 146 |
| org.apache.maven.project.DefaultProjectBuildingRequest | Line |
| This got added when implementing MNG-2199 and is no longer used. Commit 6cf9320942c34bc68205425ab696b1712ace9ba4 updated the way 'MavenProject' objects are initialized. | 196 |
| This got added when implementing MNG-2199 and is no longer used. Commit 6cf9320942c34bc68205425ab696b1712ace9ba4 updated the way 'MavenProject' objects are initialized. | 207 |
| org.apache.maven.project.DuplicateProjectException | Line |
| use {@link #DuplicateProjectException(String, File, File, String)} | 35 |
| use {@link #DuplicateProjectException(String, File, File, String)} | 43 |
| org.apache.maven.project.MavenProject | Line |
| This method is unreliable because it does not consider other dependency properties. See {@link org.apache.maven.api.JavaPathType} instead for better analysis. | 382 |
| This method is unreliable because it does not consider other dependency properties. See {@link org.apache.maven.api.JavaPathType} instead for better analysis. | 396 |
| This method is unreliable because it does not consider other dependency properties. See {@link org.apache.maven.api.JavaPathType} instead for better analysis. | 410 |
| Please use {@link MavenProjectHelper} | 823 |
| Use MavenProjectHelper.attachArtifact(..) instead. | 1655 |
| Use {@link org.apache.maven.model.io.ModelWriter}. | 1661 |
| Use {@link org.apache.maven.model.io.ModelWriter}. | 1670 |
| org.apache.maven.project.ProjectBuildingException | Line |
| use {@link #getPomFile()} | 72 |
| org.apache.maven.project.ProjectBuildingRequest | Line |
| This got added when implementing MNG-2199 and is no longer used. Commit 6cf9320942c34bc68205425ab696b1712ace9ba4 updated the way 'MavenProject' objects are initialized. | 174 |
| This got added when implementing MNG-2199 and is no longer used. Commit 6cf9320942c34bc68205425ab696b1712ace9ba4 updated the way 'MavenProject' objects are initialized. | 182 |
| org.apache.maven.toolchain.java.DefaultJavaToolChain | Line |
| clients that do not require compatibility with Maven 3.2.3 and earlier should link to {@link JavaToolchainImpl} instead. | 29 |
FIXME Work
Number of occurrences found in the code: 1
| org.apache.maven.project.MavenProject | Line |
|---|---|
| This is hacky. What if module directory doesn't match artifactid, and parent is coming from the repository?? | 1292 |
Todo Work
Number of occurrences found in the code: 90
| org.apache.maven.AbstractMavenLifecycleParticipant | Line |
|---|---|
| This is too early for build extensions, so maybe just remove it? | 58 |
| org.apache.maven.DefaultMaven | Line |
| Hack to make the cycle detection the same for the new graph builder | 153 |
| org.apache.maven.artifact.DependencyResolutionRequiredException | Line |
| it may be better for artifact.getFile() to throw it - perhaps it is a runtime exception? | 24 |
| org.apache.maven.artifact.handler.manager.DefaultArtifactHandlerManager | Line |
| watch out for module path | 77 |
| org.apache.maven.artifact.repository.MavenArtifactRepository | Line |
| completely separate local and remote artifact repositories | 35 |
| need to URL decode for spaces? | 226 |
| org.apache.maven.artifact.resolver.filter.ExcludesArtifactFilter | Line |
| I think this is equiv. to exclusion set filter in maven-core | 28 |
| org.apache.maven.artifact.resolver.filter.IncludesArtifactFilter | Line |
| what about wildcards? Just specifying groups? versions? | 45 |
| org.apache.maven.execution.BuildSummary | Line |
| Validate for < 0? | 49 |
| org.apache.maven.execution.DefaultMavenExecutionRequest | Line |
| this does not belong here. | 945 |
| org.apache.maven.internal.impl.DefaultDependencyResolver | Line |
| should be project-wide cache. | 83 |
| org.apache.maven.internal.impl.DefaultLifecycleRegistry | Line |
| this is session scoped as SPI can contribute. | 46 |
| implement | 123 |
| org.apache.maven.internal.impl.DefaultPackagingRegistry | Line |
| this is session scoped as SPI can contribute. | 54 |
| we should be able to inject a Map<String, LifecycleMapping> directly, however, SISU visibility filtering can only happen when an explicit lookup is performed. The whole problem here is caused by "project extensions" which are bound to a project's classloader, without any clear definition of a "project scope" | 78 |
| org.apache.maven.internal.impl.DefaultProjectManager | Line |
| we should not modify the underlying model here, but resources should be stored | 135 |
| in a separate field in the project, however, that could break v3 plugins | 136 |
| org.apache.maven.internal.impl.DefaultSession | Line |
| --No comment-- | 212 |
| org.apache.maven.lifecycle.DefaultLifecycleExecutor | Line |
| This whole method could probably removed by injecting lifeCyclePluginAnalyzer straight into client site. | 94 |
| But for some reason the whole plexus appcontext refuses to start when I try this. | 95 |
| org.apache.maven.lifecycle.DefaultLifecycles | Line |
| 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. | 42 |
| Remove the following code when maven-compat is gone This code is here to ensure maven-compat's EmptyLifecycleExecutor keeps on working. | 150 |
| org.apache.maven.lifecycle.MavenExecutionPlan | Line |
| lifecycles being executed | 36 |
| what runs in each phase | 37 |
| plugins that need downloading | 38 |
| project dependencies that need downloading | 39 |
| 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 modify | 42 |
| org.apache.maven.lifecycle.internal.DefaultLifecycleExecutionPlanCalculator | Line |
| take repo mans into account as one may be aggregating prefixes of many | 519 |
| 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 issue | 520 |
| org.apache.maven.lifecycle.internal.DependencyContext | Line |
| From a concurrency perspective, this class is not good. The combination of mutable/immutable state is not nice | 33 |
| org.apache.maven.lifecycle.internal.MojoDescriptorCreator | Line |
| take repo mans into account as one may be aggregating prefixes of many | 237 |
| 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 issue | 238 |
| org.apache.maven.lifecycle.internal.MojoExecutor | Line |
| ideally, the builder should take care of the ordering in a smarter way | 221 |
| and concurrency issues fixed with MNG-7157 | 222 |
| org.apache.maven.lifecycle.internal.ProjectBuildList | Line |
| Optimize; or maybe just rewrite the whole way aggregating mojos are being run. | 50 |
| org.apache.maven.lifecycle.internal.ProjectBuildListTest | Line |
| Make multiple segments on projectBuildList | 44 |
| org.apache.maven.lifecycle.internal.ProjectIndex | Line |
| Kristian wonders if this class really is necessary and if it overlaps other concepts. | 36 |
| org.apache.maven.lifecycle.internal.TaskSegment | Line |
| Consider throwing UnsupportedSomething on hashCode/equals | 63 |
| org.apache.maven.lifecycle.internal.builder.BuilderCommon | Line |
| I'm really wondering where this method belongs; smells like it should be on MavenProject, but for some reason it isn't ? This localization is kind-of a code smell. | 204 |
| org.apache.maven.lifecycle.internal.builder.multithreaded.MultiThreadedBuilder | Line |
| MNG-5766 changes likely made this redundant | 176 |
| org.apache.maven.plugin.DefaultBuildPluginManager | Line |
| the antrun plugin has its own configurator, the only plugin that does. might need to think about how that works | 48 |
| remove the coreArtifactFilterManager | 49 |
| pluginDescriptor classRealm and artifacts are set as a side effect of this call, which is not nice. | 198 |
| org.apache.maven.plugin.PluginManagerTest | Line |
| 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. | 92 |
| turn an equivalent back on when the RR plugin is released. | 101 |
| this will be the basis of the customizable lifecycle execution so need to figure this out quickly. | 126 |
| org.apache.maven.plugin.PluginParameterExpressionEvaluator | Line |
| don't catch exception | 190 |
| don't catch exception | 212 |
| don't catch exception | 232 |
| don't catch exception | 269 |
| without #, this could just be an evaluate call... | 313 |
| Copied from the DefaultInterpolator. We likely want to resurrect the PathTranslator or at least a similar component for re-usage | 349 |
| org.apache.maven.plugin.PluginParameterExpressionEvaluatorTest | Line |
| Auto-generated method stub | 506 |
| org.apache.maven.plugin.PluginParameterExpressionEvaluatorV4 | Line |
| don't catch exception | 160 |
| without #, this could just be an evaluate call... | 193 |
| Copied from the DefaultInterpolator. We likely want to resurrect the PathTranslator or at least a similar component for re-usage | 229 |
| org.apache.maven.plugin.PluginParameterExpressionEvaluatorV4Test | Line |
| Auto-generated method stub | 489 |
| org.apache.maven.plugin.internal.DefaultMavenPluginManager | Line |
| get those from the existing plexus scopes ? | 539 |
| 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? | 703 |
| store plugin version | 866 |
| figure out how to use the same PluginDescriptor when running mojos | 897 |
| org.apache.maven.project.DefaultProjectBuilder | Line |
| bv4 | 1031 |
| org.apache.maven.project.DuplicateArtifactAttachmentException | Line |
| Make this a checked exception, and modify the API of MavenProjectHelper. Currently, this modification would create compatibility problems for existing plugins. | 27 |
| org.apache.maven.project.MavenProject | Line |
| I would like to get rid of this. jvz. | 229 |
| let the scope handler deal with this | 325 |
| let the scope handler deal with this | 332 |
| let the scope handler deal with this | 337 |
| this should not be allowed to be null. | 461 |
| classpath check doesn't belong here - that's the other method | 1371 |
| let the scope handler deal with this | 1373 |
| let the scope handler deal with this | 1393 |
| classpath check doesn't belong here - that's the other method | 1415 |
| let the scope handler deal with this | 1459 |
| classpath check doesn't belong here - that's the other method | 1481 |
| let the scope handler deal with this | 1500 |
| classpath check doesn't belong here - that's the other method | 1517 |
| let the scope handler deal with this | 1519 |
| let the scope handler deal with this | 1539 |
| org.apache.maven.project.ProjectBuilderTest | Line |
| a similar test should be created to test the dependency management (basically all usages of DefaultModelBuilder.getCache() are affected by MNG-6530 | 169 |
| org.apache.maven.project.ProjectSorter | Line |
| !![jc; 28-jul-2005] check this; if we're using '-r' and there are aggregator tasks, this will result in weirdness. | 247 |
| org.apache.maven.project.artifact.PluginArtifact | Line |
| this is duplicate of MavenPluginArtifactHandlerProvider provided one | 58 |
| org.apache.maven.project.artifact.ProjectArtifact | Line |
| this is duplicate of PomArtifactHandlerProvider provided one | 63 |
| org.apache.maven.repository.TestRepositoryConnector | Line |
| Auto-generated method stub | 143 |
| org.apache.maven.settings.SettingsUtilsTest | Line |
| deep compare the lists | 153 |
| deep compare the lists | 156 |
| org.apache.maven.toolchain.DefaultToolchainManager | Line |
| this class is extended, needs refactoring | 45 |
| why is this using the context | 124 |
| org.apache.maven.toolchain.DefaultToolchainManagerPrivate | Line |
| refactor this, component extending component is bad practice. | 34 |
| org.apache.maven.toolchain.ToolchainFactory | Line |
| keep around?? */ | 37 |