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 | 32 | @deprecated |
| FIXME Work | 0 | fixme, @fixme |
| Todo Work | 63 | todo, @todo |
Each tag is detailed below:
Deprecated Work
Number of occurrences found in the code: 32
| org.apache.maven.ArtifactFilterManager | Line |
|---|---|
| use {@code META-INF/maven/extension.xml} to define artifacts exported by Maven core and plugin extensions. | 34 |
| use {@code META-INF/maven/extension.xml} to define artifacts exported by Maven core and plugin extensions. | 50 |
| org.apache.maven.ArtifactFilterManagerDelegate | Line |
| use {@code META-INF/maven/extension.xml} to define artifacts exported by Maven core extensions. | 24 |
| org.apache.maven.DefaultProjectDependenciesResolver | Line |
| As of 3.2.2, and there is no direct replacement. This is an internal class which was not marked as such, but should have been. | 47 |
| org.apache.maven.ProjectDependenciesResolver | Line |
| As of 3.2.2, and there is no direct replacement. This is an internal class which was not marked as such, but should have been. | 31 |
| org.apache.maven.artifact.deployer.ArtifactDeployer | Line |
| to be removed before 2.0 after the install/deploy plugins use the alternate method | 43 |
| org.apache.maven.artifact.deployer.DefaultArtifactDeployer | Line |
| we want to use the artifact method only, and ensure artifact.file is set correctly. | 66 |
| org.apache.maven.artifact.installer.ArtifactInstaller | Line |
| to be removed before 2.0 after the install/deploy plugins use the alternate method | 41 |
| org.apache.maven.artifact.installer.DefaultArtifactInstaller | Line |
| we want to use the artifact method only, and ensure artifact.file is set correctly. | 60 |
| org.apache.maven.artifact.metadata.ArtifactMetadataRetrievalException | Line |
| use {@link #ArtifactMetadataRetrievalException(String, Throwable, Artifact)} | 32 |
| use {@link #ArtifactMetadataRetrievalException(String, Throwable, Artifact)} | 41 |
| use {@link #ArtifactMetadataRetrievalException(String, Throwable, Artifact)} | 51 |
| org.apache.maven.execution.RuntimeInformation | Line |
| Use {@link org.apache.maven.rtinfo.RuntimeInformation} instead. | 26 |
| org.apache.maven.profiles.DefaultProfileManager | Line |
| without passing in the system properties, the SystemPropertiesProfileActivator will not work correctly in embedded environments. | 63 |
| org.apache.maven.project.InvalidProjectModelException | Line |
| use {@link File} constructor for pomLocation | 41 |
| use {@link File} constructor for pomLocation | 60 |
| org.apache.maven.project.MavenProjectBuilder | Line |
| use {@link ProjectBuilder} instead | 32 |
| org.apache.maven.project.ModelUtils | Line |
| --No comment-- | 26 |
| org.apache.maven.project.ProjectBuilderConfiguration | Line |
| use {@link ProjectBuildingRequest} instead | 28 |
| org.apache.maven.project.interpolation.AbstractStringBasedModelInterpolator | Line |
| Use {@link ModelInterpolator#interpolate(Model, File, ProjectBuilderConfiguration, boolean)} instead. | 118 |
| org.apache.maven.project.interpolation.ModelInterpolator | Line |
| Use {@link ModelInterpolator#interpolate(Model, File, ProjectBuilderConfiguration, boolean)} instead. | 38 |
| Use {@link ModelInterpolator#interpolate(Model, File, ProjectBuilderConfiguration, boolean)} instead. | 44 |
| org.apache.maven.repository.DelegatingLocalArtifactRepository | Line |
| instead use {@link #getIdeWorkspace()} | 59 |
| org.apache.maven.repository.MetadataResolutionRequest | Line |
| instead use {@link #getRemoteRepositories()} | 101 |
| instead use {@link #setRemoteRepositories(List)} | 113 |
| org.apache.maven.repository.RepositorySystem | Line |
| Use {@link MavenRepositorySystem} if needed, or maven-resolver directly, until maven 4.x api is out | 41 |
| org.apache.maven.repository.legacy.metadata.ArtifactMetadataRetrievalException | Line |
| use {@link #ArtifactMetadataRetrievalException(String, Throwable, Artifact)} | 33 |
| use {@link #ArtifactMetadataRetrievalException(String, Throwable, Artifact)} | 42 |
| use {@link #ArtifactMetadataRetrievalException(String, Throwable, Artifact)} | 52 |
| org.apache.maven.repository.legacy.resolver.conflict.DefaultConflictResolver | Line |
| As of 3.0, use a specific implementation instead, e.g. {@link NearestConflictResolver} | 28 |
| org.apache.maven.toolchain.DefaultToolchainsBuilder | Line |
| instead use {@link org.apache.maven.toolchain.building.DefaultToolchainsBuilder} | 34 |
| org.apache.maven.toolchain.ToolchainsBuilder | Line |
| use {@link org.apache.maven.toolchain.building.ToolchainsBuilder} instead | 29 |
Todo Work
Number of occurrences found in the code: 63
| org.apache.maven.artifact.installer.DefaultArtifactInstaller | Line |
|---|---|
| Should this be changed for MNG-6754 too? | 113 |
| org.apache.maven.artifact.manager.DefaultWagonManager | Line |
| this should be illegal in settings.xml | 157 |
| org.apache.maven.artifact.repository.metadata.AbstractRepositoryMetadata | Line |
| not sure that it should assume this, maybe the calls to addMetadata should pre-merge, then artifact replaces? | 148 |
| not sure that it should assume this, maybe the calls to addMetadata should pre-merge, then artifact replaces? | 155 |
| org.apache.maven.artifact.repository.metadata.DefaultRepositoryMetadataManager | Line |
| should this be inside the above check? touch file so that this is not checked again until interval has passed | 144 |
| currently this is first wins, but really we should take the latest by comparing either the snapshot timestamp, or some other timestamp later encoded into the metadata. | 171 |
| this needs to be repeated here so the merging doesn't interfere with the written metadata - we'd be much better having a pristine input, and an ongoing metadata for merging instead | 173 |
| this could be a lot nicer... should really be in the snapshot transformation? | 200 |
| share with DefaultPluginMappingManager. | 269 |
| org.apache.maven.artifact.repository.metadata.RepositoryMetadata | Line |
| not happy about the store method - they use "this" | 27 |
| org.apache.maven.artifact.repository.metadata.SnapshotArtifactRepositoryMetadata | Line |
| split instantiation (versioning, plugin mappings) from definition | 27 |
| org.apache.maven.artifact.resolver.ArtifactResolutionResult | Line |
| carlos: all these possible has*Exceptions and get*Exceptions methods make the clients too complex requiring a long list of checks, need to create a parent/interface/encapsulation for the types of exceptions | 42 |
| this needs to accept a {@link OverConstrainedVersionException} as returned by {@link #getVersionRangeViolation(int)} but it's not used like that in DefaultLegacyArtifactCollector | 163 |
| org.apache.maven.artifact.resolver.DebugResolutionListener | Line |
| better way than static? this might hide messages in a reactor | 76 |
| org.apache.maven.artifact.resolver.DefaultArtifactResolver | Line |
| hack because metadata isn't generated in m2e correctly and i want to run the maven i have in the workspace | 378 |
| org.apache.maven.artifact.resolver.ResolutionListener | Line |
| Use the following two instead of manageArtifact | 80 |
| Remove ResolutionListenerDM interface | 81 |
| 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? | 178 |
| org.apache.maven.project.MavenProjectBuilder | Line |
| maven-site-plugin -- not used by the plugin directly, but used by Doxia Integration Tool & MPIR see DOXIASITETOOLS-167 & MPIR-349 | 39 |
| remote-resources-plugin | 44 |
| remote-resources-plugin | 49 |
| this is only to provide a project for plugins that don't need a project to execute but need some of the values from a MavenProject. Ideally this should be something internal and nothing outside Maven would ever need this so it should not be exposed in a public API | 57 |
| org.apache.maven.project.artifact.ActiveProjectArtifact | Line |
| 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. | 39 |
| org.apache.maven.project.artifact.MavenMetadataSource | Line |
| hack: don't rebuild model if it was already loaded during reactor resolution | 186 |
| shouldn't the versions be merged across relocations? | 617 |
| org.apache.maven.project.interpolation.AbstractStringBasedModelInterpolator | Line |
| Consolidate this logic with the PluginParameterExpressionEvaluator, minus deprecations/bans. | 60 |
| org.apache.maven.project.interpolation.RegexBasedModelInterpolator | Line |
| Consolidate this logic with the PluginParameterExpressionEvaluator, minus deprecations/bans. | 31 |
| org.apache.maven.repository.MetadataResolutionResult | Line |
| this needs to accept a {@link OverConstrainedVersionException} as returned by {@link #getVersionRangeViolation(int)} but it's not used like that in {@link org.apache.maven.repository.legacy.resolver.DefaultLegacyArtifactCollector} | 147 |
| org.apache.maven.repository.TestRepositorySystem | Line |
| Auto-generated method stub | 192 |
| Auto-generated method stub | 291 |
| org.apache.maven.repository.legacy.DefaultUpdateCheckManager | Line |
| throw something instead? | 85 |
| throw something instead? | 127 |
| org.apache.maven.repository.legacy.DefaultWagonManager | Line |
| remove the update check manager | 64 |
| separate into retriever and publisher | 65 |
| remove hardcoding of checksum logic | 66 |
| configure on repository | 345 |
| configure these on the repository | 515 |
| shouldn't need a file intermediary - improve wagon to take a stream | 541 |
| org.apache.maven.repository.legacy.repository.DefaultArtifactRepositoryFactory | Line |
| use settings? | 40 |
| org.apache.maven.repository.legacy.resolver.DefaultArtifactCollectorTest | Line |
| use newest conflict resolver | 156 |
| use newest conflict resolver | 178 |
| org.apache.maven.repository.legacy.resolver.DefaultLegacyArtifactCollector | Line |
| we probably want to warn the user that he is building an artifact with different values than in dependencyManagement | 216 |
| Does this check need to happen here? Had to add the same call below when we iterate on child nodes -- will that suffice? | 244 |
| shouldn't need to double up on this work, only done for simplicity of handling recommended version but the restriction is identical | 261 |
| ick. this forces the OCE that should have come from the previous call. It is still correct | 265 |
| add better exception that can detail the two conflicting artifacts | 341 |
| add better exception | 352 |
| should this be part of mediation? previous one is more dominant | 357 |
| maybe its better to just pass the range through to retrieval and use a transformation? | 453 |
| might be better to have source.retrieve() throw a specific exception for this situation and catch here rather than have it return null | 510 |
| org.apache.maven.repository.legacy.resolver.conflict.DefaultConflictResolverFactory | Line |
| you don't need the container in here with the active maps (jvz). | 35 |
| org.apache.maven.repository.legacy.resolver.conflict.NewestConflictResolver | Line |
| log message or throw exception? | 53 |
| org.apache.maven.repository.legacy.resolver.conflict.OldestConflictResolver | Line |
| log message or throw exception? | 53 |
| org.apache.maven.repository.legacy.resolver.transform.AbstractVersionTransformation | Line |
| try and refactor to remove abstract methods - not particular happy about current design | 44 |
| also do this logging for other metadata? | 97 |
| figure out way to avoid duplicated message | 98 |
| org.apache.maven.repository.legacy.resolver.transform.ReleaseArtifactTransformation | Line |
| Should this be changed for MNG-6754 too? | 76 |
| org.apache.maven.repository.legacy.resolver.transform.SnapshotTransformation | Line |
| Should this be changed for MNG-6754 too? | 82 |
| org.apache.maven.repository.metadata.MetadataResolutionException | Line |
| Auto-generated constructor stub | 28 |
| Auto-generated constructor stub | 33 |
| Auto-generated constructor stub | 38 |
| Auto-generated constructor stub | 43 |
| org.apache.maven.settings.DefaultMavenSettingsBuilder | Line |
| 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 Plexus | 115 |