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 | 17 | @deprecated |
| FIXME Work | 2 | fixme, @fixme |
| Todo Work | 54 | todo, @todo |
Each tag is detailed below:
Deprecated Work
Number of occurrences found in the code: 17
| org.apache.maven.artifact.deployer.ArtifactDeployer | Line |
|---|---|
| to be removed before 2.0 after the install/deploy plugins use the alternate method | 42 |
| org.apache.maven.artifact.deployer.DefaultArtifactDeployer | Line |
| we want to use the artifact method only, and ensure artifact.file is set correctly. | 64 |
| 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. | 57 |
| 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. | 64 |
| 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-- | 35 |
| 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. | 39 |
| Use {@link ModelInterpolator#interpolate(Model, File, ProjectBuilderConfiguration, boolean)} instead. | 45 |
| org.apache.maven.repository.MetadataResolutionRequest | Line |
| instead use {@link #getRemoteRepositories()} | 101 |
| instead use {@link #setRemoteRepositories(List)} | 113 |
| org.apache.maven.repository.legacy.resolver.conflict.DefaultConflictResolver | Line |
| As of 3.0, use a specific implementation instead, e.g. {@link NearestConflictResolver} | 28 |
FIXME Work
Number of occurrences found in the code: 2
| org.apache.maven.project.inheritance.DefaultModelInheritanceAssembler | Line |
|---|---|
| Filter out the inherited == false stuff! | 114 |
| somehow report to the user that there are too many '..' elements. For now, ignore the extra '..'. | 597 |
Todo Work
Number of occurrences found in the code: 54
| org.apache.maven.artifact.installer.DefaultArtifactInstaller | Line |
|---|---|
| Should this be changed for MNG-6754 too? | 110 |
| org.apache.maven.artifact.manager.DefaultWagonManager | Line |
| this should be illegal in settings.xml | 153 |
| 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 | 142 |
| 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. | 169 |
| 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 | 171 |
| this could be a lot nicer... should really be in the snapshot transformation? | 198 |
| share with DefaultPluginMappingManager. | 267 |
| org.apache.maven.artifact.repository.metadata.SnapshotArtifactRepositoryMetadata | Line |
| split instantiation (versioning, plugin mappings) from definition | 28 |
| 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 | 384 |
| 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.inheritance.DefaultModelInheritanceAssembler | Line |
| Remove this! | 54 |
| Remove this! | 264 |
| Remove this! | 444 |
| This should eventually be migrated to DefaultPathTranslator. | 552 |
| Move this to plexus-utils' PathTool. | 582 |
| 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. | 32 |
| 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} | 149 |
| org.apache.maven.repository.legacy.DefaultUpdateCheckManager | Line |
| throw something instead? | 81 |
| throw something instead? | 123 |
| org.apache.maven.repository.legacy.DefaultWagonManager | Line |
| remove the update check manager | 58 |
| separate into retriever and publisher | 59 |
| remove hardcoding of checksum logic | 60 |
| configure on repository | 337 |
| configure these on the repository | 503 |
| shouldn't need a file intermediatary - improve wagon to take a stream | 529 |
| org.apache.maven.repository.legacy.repository.DefaultArtifactRepositoryFactory | Line |
| use settings? | 37 |
| org.apache.maven.repository.legacy.resolver.DefaultArtifactCollectorTest | Line |
| use newest conflict resolver | 155 |
| use newest conflict resolver | 176 |
| 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 | 213 |
| Does this check need to happen here? Had to add the same call below when we iterate on child nodes -- will that suffice? | 241 |
| shouldn't need to double up on this work, only done for simplicity of handling recommended version but the restriction is identical | 258 |
| ick. this forces the OCE that should have come from the previous call. It is still correct | 262 |
| add better exception that can detail the two conflicting artifacts | 338 |
| add better exception | 349 |
| should this be part of mediation? previous one is more dominant | 354 |
| maybe its better to just pass the range through to retrieval and use a transformation? | 450 |
| might be better to have source.retrieve() throw a specific exception for this situation and catch here rather than have it return null | 507 |
| org.apache.maven.repository.legacy.resolver.conflict.DefaultConflictResolverFactory | Line |
| you don't need the container in here with the active maps (jvz). | 34 |
| org.apache.maven.repository.legacy.resolver.conflict.NewestConflictResolver | Line |
| log message or throw exception? | 50 |
| org.apache.maven.repository.legacy.resolver.conflict.OldestConflictResolver | Line |
| log message or throw exception? | 50 |
| 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? | 96 |
| figure out way to avoid duplicated message | 97 |
| org.apache.maven.repository.legacy.resolver.transform.ReleaseArtifactTransformation | Line |
| Should this be changed for MNG-6754 too? | 73 |
| org.apache.maven.repository.legacy.resolver.transform.SnapshotTransformation | Line |
| Should this be changed for MNG-6754 too? | 81 |
| org.apache.maven.repository.metadata.MetadataResolutionException | Line |
| Auto-generated constructor stub | 27 |
| Auto-generated constructor stub | 32 |
| Auto-generated constructor stub | 37 |
| Auto-generated constructor stub | 42 |


