From 82f9e43b0b5094935d0e04d1bb3e62459a5a8dde Mon Sep 17 00:00:00 2001
From: "dependabot[bot]" <49699333+dependabot[bot]@users.noreply.github.com>
Date: Wed, 7 Jun 2023 00:37:46 +0200
Subject: [PATCH] Bump maven-reporting-impl from 3.0.0 to 3.2.0 (#452)

* Bump maven-reporting-impl from 3.0.0 to 3.2.0

Bumps [maven-reporting-impl](https://github.com/apache/maven-reporting-impl) from 3.0.0 to 3.2.0.
- [Release notes](https://github.com/apache/maven-reporting-impl/releases)
- [Commits](https://github.com/apache/maven-reporting-impl/compare/maven-reporting-impl-3.0.0...maven-reporting-impl-3.2.0)

---
updated-dependencies:
- dependency-name: org.apache.maven.reporting:maven-reporting-impl
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

* Fix AbstractThirdPartyReportMojo implementation

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Slawomir Jaranowski <s.jaranowski@gmail.com>
---
 pom.xml                                       |  8 ++-
 .../license/AbstractThirdPartyReportMojo.java | 62 ++-----------------
 2 files changed, 13 insertions(+), 57 deletions(-)

diff --git a/pom.xml b/pom.xml
index 2bd53cc82..d5f95cd33 100644
--- a/pom.xml
+++ b/pom.xml
@@ -190,7 +190,13 @@
     <dependency>
       <groupId>org.apache.maven.reporting</groupId>
       <artifactId>maven-reporting-impl</artifactId>
-      <version>3.0.0</version>
+      <version>3.2.0</version>
+      <exclusions>
+        <exclusion>
+          <groupId>org.codehaus.plexus</groupId>
+          <artifactId>plexus-container-default</artifactId>
+        </exclusion>
+      </exclusions>
     </dependency>
 
     <!-- Doxia -->
diff --git a/src/main/java/org/codehaus/mojo/license/AbstractThirdPartyReportMojo.java b/src/main/java/org/codehaus/mojo/license/AbstractThirdPartyReportMojo.java
index 61c529c90..103b2070b 100644
--- a/src/main/java/org/codehaus/mojo/license/AbstractThirdPartyReportMojo.java
+++ b/src/main/java/org/codehaus/mojo/license/AbstractThirdPartyReportMojo.java
@@ -25,7 +25,6 @@
 import org.apache.commons.collections.CollectionUtils;
 import org.apache.maven.artifact.resolver.ArtifactNotFoundException;
 import org.apache.maven.artifact.resolver.ArtifactResolutionException;
-import org.apache.maven.doxia.siterenderer.Renderer;
 import org.apache.maven.plugin.MojoExecutionException;
 import org.apache.maven.plugin.MojoFailureException;
 import org.apache.maven.plugins.annotations.Component;
@@ -271,16 +270,6 @@ public abstract class AbstractThirdPartyReportMojo extends AbstractMavenReport
      @Parameter( property = "license.licenseMergesUrl" )
      protected String licenseMergesUrl;
 
-    /**
-     * The output directory for the report. Note that this parameter is only evaluated if the goal is run directly from
-     * the command line. If the goal is run indirectly as part of a site generation, the output directory configured in
-     * the Maven Site Plugin is used instead.
-     *
-     * @since 1.1
-     */
-    @Parameter( defaultValue = "${project.reporting.outputDirectory}", required = true )
-    private File outputDirectory;
-
     /**
      * Flag to activate verbose mode.
      * <p>
@@ -303,26 +292,10 @@ public abstract class AbstractThirdPartyReportMojo extends AbstractMavenReport
     @Parameter( property = "license.encoding", defaultValue = "${project.build.sourceEncoding}" )
     private String encoding;
 
-    /**
-     * The Maven Project.
-     *
-     * @since 1.1
-     */
-    @Parameter( defaultValue = "${project}", readonly = true )
-    private MavenProject project;
-
     // ----------------------------------------------------------------------
     // Plexus Components
     // ----------------------------------------------------------------------
 
-    /**
-     * Doxia Site Renderer component.
-     *
-     * @since 1.1
-     */
-    @Component
-    private Renderer siteRenderer;
-
     /**
      * Internationalization component.
      *
@@ -461,35 +434,7 @@ protected void executeReport( Locale locale )
     /**
      * {@inheritDoc}
      */
-    protected MavenProject getProject()
-    {
-        return project;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    protected String getOutputDirectory()
-    {
-        if ( !outputDirectory.isAbsolute() )
-        {
-            outputDirectory = new File( project.getBasedir(), outputDirectory.getPath() );
-        }
-
-        return outputDirectory.getAbsolutePath();
-    }
-
-    /**
-     * {@inheritDoc}
-     */
-    protected Renderer getSiteRenderer()
-    {
-        return siteRenderer;
-    }
-
-    /**
-     * {@inheritDoc}
-     */
+    @Override
     public String getDescription( Locale locale )
     {
         return i18n.getString( getOutputName(), locale, "report.description" );
@@ -498,6 +443,7 @@ public String getDescription( Locale locale )
     /**
      * {@inheritDoc}
      */
+    @Override
     public String getName( Locale locale )
     {
         return i18n.getString( getOutputName(), locale, "report.title" );
@@ -510,6 +456,7 @@ public String getName( Locale locale )
     /**
      * {@inheritDoc}
      */
+    @Override
     public boolean isIncludeTransitiveDependencies()
     {
         return includeTransitiveDependencies;
@@ -518,12 +465,14 @@ public boolean isIncludeTransitiveDependencies()
     /**
      * {@inheritDoc}
      */
+    @Override
     public boolean isExcludeTransitiveDependencies()
     {
         return excludeTransitiveDependencies;
     }
 
     /** {@inheritDoc} */
+    @Override
     public ArtifactFilters getArtifactFilters()
     {
         if ( artifactFilters == null )
@@ -538,6 +487,7 @@ public ArtifactFilters getArtifactFilters()
     /**
      * {@inheritDoc}
      */
+    @Override
     public boolean isVerbose()
     {
         return verbose;