Merge pull request #3542 from mapfish/GSGGR-43

Throw exception for missing javadoc when generating the docs

Author: lanseg

Diff for: core/src/main/java/org/mapfish/print/config/Configuration.java

@@ -315,6 +315,7 @@ public final File getDirectory() {
     return this.configurationFile.getAbsoluteFile().getParentFile();
   }
 
+  /** Sets the configuration file. */
   public final void setConfigurationFile(final File configurationFile) {
     this.configurationFile = configurationFile;
   }
@@ -692,6 +693,7 @@ public SmtpConfig getSmtp() {
     return smtp;
   }
 
+  /** Sets the smtp config. */
   public void setSmtp(final SmtpConfig smtp) {
     this.smtp = smtp;
   }

Diff for: core/src/main/java/org/mapfish/print/config/Template.java

@@ -147,6 +147,7 @@ public final String getReportTemplate() {
     return this.reportTemplate;
   }
 
+  /** Sets the report template. */
   public final void setReportTemplate(final String reportTemplate) {
     this.reportTemplate = reportTemplate;
   }
@@ -184,38 +185,47 @@ public final void setTableData(final String tableData) {
     this.tableDataKey = tableData;
   }
 
+  /** Returns table data key. */
   public final String getTableDataKey() {
     return this.tableDataKey;
   }
 
+  /** Returns JDBC url. */
   public final String getJdbcUrl() {
     return this.jdbcUrl;
   }
 
+  /** Sets the JDBC url. */
   public final void setJdbcUrl(final String jdbcUrl) {
     this.jdbcUrl = jdbcUrl;
   }
 
+  /** Returns the JDBC drivers. */
   public final Set<String> getJdbcDrivers() {
     return this.jdbcDrivers;
   }
 
+  /** Sets the JDBC drivers. */
   public final void setJdbcDrivers(final Set<String> jdbcDrivers) {
     this.jdbcDrivers = jdbcDrivers;
   }
 
+  /** Returns the JDBC driver. */
   public final String getJdbcUser() {
     return this.jdbcUser;
   }
 
+  /** Sets the JDBC user. */
   public final void setJdbcUser(final String jdbcUser) {
     this.jdbcUser = jdbcUser;
   }
 
+  /** Gets the JDBC password. */
   public final String getJdbcPassword() {
     return this.jdbcPassword;
   }
 
+  /** Sets the JDBC password. */
   public final void setJdbcPassword(final String jdbcPassword) {
     this.jdbcPassword = jdbcPassword;
   }
@@ -387,6 +397,7 @@ public final boolean isMapExport() {
     return this.mapExport;
   }
 
+  /** Sets the map export. */
   public final void setMapExport(final boolean mapExport) {
     this.mapExport = mapExport;
   }

Diff for: core/src/main/java/org/mapfish/print/map/geotools/AbstractFeatureSourceLayerPlugin.java

@@ -83,6 +83,7 @@ public Style load(
     };
   }
 
+  /** Sets the style parser for this feature source layer. */
   public final void setParser(final StyleParser parser) {
     this.parser = parser;
   }

Diff for: core/src/main/java/org/mapfish/print/processor/http/matcher/HostMatcher.java

@@ -53,6 +53,7 @@ public final boolean matches(final MatchInfo matchInfo)
   protected abstract Optional<Boolean> tryOverrideValidation(MatchInfo matchInfo)
       throws UnknownHostException, SocketException, MalformedURLException;
 
+  /** Port of this host matcher. */
   public final void setPort(final int port) {
     this.port = port;
   }

Diff for: docs/src/main/groovy/Javadoc7Parser.groovy

@@ -58,7 +58,7 @@ class Javadoc7Parser {
                     throw new IllegalArgumentException(errorHandler(obj, objectName))
                 }
             } else {
-                return ""
+                throw new IllegalArgumentException(errorHandler(obj, objectName))
             }
         }
     }