Skip to content

Commit b762406

Browse files
michael-omichael-o
committed
Prepare for Doxia 2.0.0
1 parent 6c265af commit b762406

File tree

6 files changed

+30
-148
lines changed

6 files changed

+30
-148
lines changed

‎pom.xml

Lines changed: 13 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ under the License.
2828
</parent>
2929

3030
<artifactId>maven-javadoc-plugin</artifactId>
31-
<version>3.6.4-SNAPSHOT</version>
31+
<version>4.0.0-M1-SNAPSHOT</version>
3232
<packaging>maven-plugin</packaging>
3333

3434
<name>Apache Maven Javadoc Plugin</name>
@@ -112,8 +112,7 @@ under the License.
112112
<properties>
113113
<javaVersion>8</javaVersion>
114114
<mavenVersion>3.2.5</mavenVersion>
115-
<doxiaVersion>1.11.1</doxiaVersion>
116-
<doxia-sitetoolsVersion>1.11.1</doxia-sitetoolsVersion>
115+
<doxiaSitetoolsVersion>2.0.0-M16</doxiaSitetoolsVersion>
117116
<wagonVersion>2.4</wagonVersion>
118117
<aetherVersion>1.0.0.v20140518</aetherVersion>
119118
<!-- https://cwiki.apache.org/confluence/x/VIHOCg#MavenEcosystemCleanup-ResolverandMaven -->
@@ -126,8 +125,8 @@ under the License.
126125
<enforcerPluginVersion>3.4.1</enforcerPluginVersion>
127126
<pluginPluginVersion>3.10.1</pluginPluginVersion>
128127
<jarPluginVersion>3.3.0</jarPluginVersion>
129-
<sitePluginVersion>3.12.1</sitePluginVersion>
130-
<projectInfoReportsPluginVersion>3.5.0</projectInfoReportsPluginVersion>
128+
<sitePluginVersion>4.0.0-M13</sitePluginVersion>
129+
<projectInfoReportsPluginVersion>4.0.0-M1-SNAPSHOT</projectInfoReportsPluginVersion>
131130
<project.build.outputTimestamp>2023-11-30T21:35:29Z</project.build.outputTimestamp>
132131
<slf4jVersion>1.7.36</slf4jVersion>
133132
</properties>
@@ -215,7 +214,7 @@ under the License.
215214
<dependency>
216215
<groupId>org.apache.maven.reporting</groupId>
217216
<artifactId>maven-reporting-api</artifactId>
218-
<version>3.1.1</version>
217+
<version>4.0.0-M9</version>
219218
</dependency>
220219
<dependency>
221220
<groupId>org.apache.maven</groupId>
@@ -242,21 +241,20 @@ under the License.
242241
<dependency>
243242
<groupId>org.apache.maven.doxia</groupId>
244243
<artifactId>doxia-sink-api</artifactId>
245-
<version>${doxiaVersion}</version>
244+
<version>2.0.0-M8</version>
246245
</dependency>
247246

248247
<!-- Doxia-sitetools -->
249248
<dependency>
250249
<groupId>org.apache.maven.doxia</groupId>
251250
<artifactId>doxia-site-renderer</artifactId>
252-
<version>${doxia-sitetoolsVersion}</version>
253-
<exclusions>
254-
<exclusion>
255-
<!-- MJAVADOC-726 -->
256-
<groupId>org.apache.velocity</groupId>
257-
<artifactId>velocity-tools</artifactId>
258-
</exclusion>
259-
</exclusions>
251+
<version>${doxiaSitetoolsVersion}</version>
252+
</dependency>
253+
<!-- Doxia-sitetools -->
254+
<dependency>
255+
<groupId>org.apache.maven.doxia</groupId>
256+
<artifactId>doxia-integration-tools</artifactId>
257+
<version>${doxiaSitetoolsVersion}</version>
260258
</dependency>
261259

262260
<!-- wagon -->

‎src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java

Lines changed: 9 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@
6565
import org.apache.maven.artifact.handler.manager.ArtifactHandlerManager;
6666
import org.apache.maven.artifact.versioning.ArtifactVersion;
6767
import org.apache.maven.artifact.versioning.DefaultArtifactVersion;
68+
import org.apache.maven.doxia.tools.SiteTool;
6869
import org.apache.maven.execution.MavenSession;
6970
import org.apache.maven.model.Dependency;
7071
import org.apache.maven.model.Plugin;
@@ -269,6 +270,12 @@ public abstract class AbstractJavadocMojo extends AbstractMojo {
269270
// Mojo components
270271
// ----------------------------------------------------------------------
271272

273+
/**
274+
* SiteTool.
275+
*/
276+
@Component
277+
protected SiteTool siteTool;
278+
272279
/**
273280
* Archiver manager
274281
*
@@ -4250,64 +4257,8 @@ private void validateJavadocOptions() throws MavenReportException {
42504257
}
42514258

42524259
// locale
4253-
if (this.locale != null && !this.locale.isEmpty()) {
4254-
StringTokenizer tokenizer = new StringTokenizer(this.locale, "_");
4255-
final int maxTokens = 3;
4256-
if (tokenizer.countTokens() > maxTokens) {
4257-
throw new MavenReportException(
4258-
"Unsupported option <locale/> '" + this.locale + "', should be language_country_variant.");
4259-
}
4260-
4261-
Locale localeObject = null;
4262-
if (tokenizer.hasMoreTokens()) {
4263-
String language = tokenizer.nextToken().toLowerCase(Locale.ENGLISH);
4264-
if (!Arrays.asList(Locale.getISOLanguages()).contains(language)) {
4265-
throw new MavenReportException(
4266-
"Unsupported language '" + language + "' in option <locale/> '" + this.locale + "'");
4267-
}
4268-
localeObject = new Locale(language);
4269-
4270-
if (tokenizer.hasMoreTokens()) {
4271-
String country = tokenizer.nextToken().toUpperCase(Locale.ENGLISH);
4272-
if (!Arrays.asList(Locale.getISOCountries()).contains(country)) {
4273-
throw new MavenReportException(
4274-
"Unsupported country '" + country + "' in option <locale/> '" + this.locale + "'");
4275-
}
4276-
localeObject = new Locale(language, country);
4277-
4278-
if (tokenizer.hasMoreTokens()) {
4279-
String variant = tokenizer.nextToken();
4280-
localeObject = new Locale(language, country, variant);
4281-
}
4282-
}
4283-
}
4284-
4285-
if (localeObject == null) {
4286-
throw new MavenReportException(
4287-
"Unsupported option <locale/> '" + this.locale + "', should be language_country_variant.");
4288-
}
4289-
4290-
this.locale = localeObject.toString();
4291-
final List<Locale> availableLocalesList = Arrays.asList(Locale.getAvailableLocales());
4292-
if (StringUtils.isNotEmpty(localeObject.getVariant()) && !availableLocalesList.contains(localeObject)) {
4293-
StringBuilder sb = new StringBuilder();
4294-
sb.append("Unsupported option <locale/> with variant '").append(this.locale);
4295-
sb.append("'");
4296-
4297-
localeObject = new Locale(localeObject.getLanguage(), localeObject.getCountry());
4298-
this.locale = localeObject.toString();
4299-
4300-
sb.append(", trying to use <locale/> without variant, i.e. '")
4301-
.append(this.locale)
4302-
.append("'");
4303-
if (getLog().isWarnEnabled()) {
4304-
getLog().warn(sb.toString());
4305-
}
4306-
}
4307-
4308-
if (!availableLocalesList.contains(localeObject)) {
4309-
throw new MavenReportException("Unsupported option <locale/> '" + this.locale + "'");
4310-
}
4260+
if (StringUtils.isNotEmpty(this.locale)) {
4261+
this.locale = siteTool.getSiteLocales(locale).get(0).toString();
43114262
}
43124263
}
43134264

‎src/main/java/org/apache/maven/plugins/javadoc/JavadocJar.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,12 +21,12 @@
2121
import java.io.File;
2222
import java.io.IOException;
2323
import java.util.List;
24-
import java.util.Locale;
2524

2625
import org.apache.maven.archiver.MavenArchiveConfiguration;
2726
import org.apache.maven.archiver.MavenArchiver;
2827
import org.apache.maven.artifact.DependencyResolutionRequiredException;
2928
import org.apache.maven.artifact.handler.ArtifactHandler;
29+
import org.apache.maven.doxia.tools.SiteTool;
3030
import org.apache.maven.model.Resource;
3131
import org.apache.maven.plugin.MojoExecutionException;
3232
import org.apache.maven.plugins.annotations.Component;
@@ -178,7 +178,7 @@ public void doExecute() throws MojoExecutionException {
178178
}
179179

180180
try {
181-
executeReport(Locale.getDefault());
181+
executeReport(SiteTool.DEFAULT_LOCALE);
182182
} catch (MavenReportException e) {
183183
failOnError("MavenReportException: Error while generating Javadoc", e);
184184
} catch (RuntimeException e) {

‎src/main/java/org/apache/maven/plugins/javadoc/JavadocReport.java

Lines changed: 6 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,9 @@
2828

2929
import org.apache.maven.doxia.sink.Sink;
3030
import org.apache.maven.doxia.sink.SinkFactory;
31-
import org.apache.maven.doxia.siterenderer.RenderingContext;
31+
import org.apache.maven.doxia.siterenderer.DocumentRenderingContext;
3232
import org.apache.maven.doxia.siterenderer.sink.SiteRendererSink;
33+
import org.apache.maven.doxia.tools.SiteTool;
3334
import org.apache.maven.plugin.MojoExecutionException;
3435
import org.apache.maven.plugin.MojoFailureException;
3536
import org.apache.maven.plugins.annotations.Execute;
@@ -117,11 +118,6 @@ public String getDescription(Locale locale) {
117118
}
118119

119120
/** {@inheritDoc} */
120-
@Override
121-
public void generate(org.codehaus.doxia.sink.Sink sink, Locale locale) throws MavenReportException {
122-
generate(sink, null, locale);
123-
}
124-
125121
public void generate(Sink sink, Locale locale) throws MavenReportException {
126122
generate(sink, null, locale);
127123
}
@@ -236,9 +232,6 @@ public boolean canGenerateReport() {
236232

237233
canGenerate = canGenerateReport(files);
238234
}
239-
if (getLog().isDebugEnabled()) {
240-
getLog().debug(" canGenerateReport = " + canGenerate + " for project " + this.project);
241-
}
242235
return canGenerate;
243236
}
244237

@@ -293,16 +286,16 @@ public void doExecute() throws MojoExecutionException, MojoFailureException {
293286

294287
String filename = getOutputName() + ".html";
295288

296-
Locale locale = Locale.getDefault();
289+
Locale locale = SiteTool.DEFAULT_LOCALE;
297290

298291
try {
299-
// TODO Replace null with real value
300-
RenderingContext docRenderingContext = new RenderingContext(outputDirectory, filename, null);
292+
String reportMojoInfo = mojoExecution.getPlugin().getId() + ":" + mojoExecution.getGoal();
293+
DocumentRenderingContext docRenderingContext =
294+
new DocumentRenderingContext(outputDirectory, filename, reportMojoInfo);
301295

302296
SiteRendererSink sink = new SiteRendererSink(docRenderingContext);
303297

304298
generate(sink, null, locale);
305-
306299
} catch (MavenReportException | RuntimeException e) {
307300
failOnError("An error has occurred in " + getName(Locale.ENGLISH) + " report generation", e);
308301
}

‎src/test/java/org/apache/maven/plugins/javadoc/JavadocReportTest.java

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1052,15 +1052,6 @@ public void testValidateOptions() throws Exception {
10521052
assertTrue("No wrong charset catch", e.getMessage().contains("Unsupported option <charset/>"));
10531053
}
10541054

1055-
// locale
1056-
testPom = unit.resolve("validate-options-test/wrong-locale-test-plugin-config.xml");
1057-
mojo = lookupMojo(testPom);
1058-
try {
1059-
mojo.execute();
1060-
fail("No wrong locale catch");
1061-
} catch (MojoExecutionException e) {
1062-
assertTrue("No wrong locale catch", e.getMessage().contains("Unsupported option <locale/>"));
1063-
}
10641055
testPom = unit.resolve("validate-options-test/wrong-locale-with-variant-test-plugin-config.xml");
10651056
mojo = lookupMojo(testPom);
10661057
mojo.execute();

‎src/test/resources/unit/validate-options-test/wrong-locale-test-plugin-config.xml

Lines changed: 0 additions & 51 deletions
This file was deleted.

0 commit comments

Comments
 (0)