You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tiles.apache.org by ap...@apache.org on 2007/10/17 10:16:43 UTC
svn commit: r585406 - in
/tiles/framework/branches/TILES_2_0_X/tiles-core/src:
main/java/org/apache/tiles/definition/
main/java/org/apache/tiles/definition/digester/
test/java/org/apache/tiles/definition/digester/
Author: apetrelli
Date: Wed Oct 17 01:16:42 2007
New Revision: 585406
URL: http://svn.apache.org/viewvc?rev=585406&view=rev
Log:
Merge from trunk to TILES_2_0_X branch.
TILES-223
Now validation is turned ON by default.
TILES-225
Now the DefinitionsFactoryException is a normal implementation of an extended exception.
Modified:
tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/DefinitionsFactoryException.java
tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java
tiles/framework/branches/TILES_2_0_X/tiles-core/src/test/java/org/apache/tiles/definition/digester/TestDigesterDefinitionsReader.java
Modified: tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/DefinitionsFactoryException.java
URL: http://svn.apache.org/viewvc/tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/DefinitionsFactoryException.java?rev=585406&r1=585405&r2=585406&view=diff
==============================================================================
--- tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/DefinitionsFactoryException.java (original)
+++ tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/DefinitionsFactoryException.java Wed Oct 17 01:16:42 2007
@@ -36,7 +36,6 @@
*/
public DefinitionsFactoryException() {
super();
- this.exception = null;
}
/**
@@ -46,7 +45,6 @@
*/
public DefinitionsFactoryException(String message) {
super(message);
- this.exception = null;
}
@@ -60,8 +58,7 @@
* @param e The exception to be wrapped.
*/
public DefinitionsFactoryException(Exception e) {
- super();
- this.exception = e;
+ super(e);
}
@@ -75,49 +72,6 @@
* @param e The exception to be wrapped.
*/
public DefinitionsFactoryException(String message, Exception e) {
- super(message);
- this.exception = e;
+ super(message, e);
}
-
-
- /**
- * Return a detail message for this exception.
- * <p/>
- * <p>If there is a embedded exception, and if the DefinitionsFactoryException
- * has no detail message of its own, this method will return
- * the detail message from the embedded exception.</p>
- *
- * @return The error or warning message.
- */
- public String getMessage() {
- String message = super.getMessage();
-
- if (message == null && exception != null) {
- return exception.getMessage();
- } else {
- return message;
- }
- }
-
-
- /**
- * Return the embedded exception, if any.
- *
- * @return The embedded exception, or <code>null</code> if there is none.
- */
- public Exception getException() {
- return exception;
- }
-
- //////////////////////////////////////////////////////////////////////
- // Internal state.
- //////////////////////////////////////////////////////////////////////
-
-
- /**
- * Any "wrapped" exception will be exposed when this is serialized.
- *
- * @serial
- */
- private Exception exception;
}
Modified: tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java
URL: http://svn.apache.org/viewvc/tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java?rev=585406&r1=585405&r2=585406&view=diff
==============================================================================
--- tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java (original)
+++ tiles/framework/branches/TILES_2_0_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java Wed Oct 17 01:16:42 2007
@@ -166,9 +166,9 @@
private Map<String, Definition> definitions;
/**
* Should we use a validating XML parser to read the configuration file.
- * Default is <code>false</code>.
+ * Default is <code>true</code>.
*/
- protected boolean validating = false;
+ protected boolean validating = true;
/**
* The set of public identifiers, and corresponding resource names for
* the versions of the configuration file DTDs we know about. There
Modified: tiles/framework/branches/TILES_2_0_X/tiles-core/src/test/java/org/apache/tiles/definition/digester/TestDigesterDefinitionsReader.java
URL: http://svn.apache.org/viewvc/tiles/framework/branches/TILES_2_0_X/tiles-core/src/test/java/org/apache/tiles/definition/digester/TestDigesterDefinitionsReader.java?rev=585406&r1=585405&r2=585406&view=diff
==============================================================================
--- tiles/framework/branches/TILES_2_0_X/tiles-core/src/test/java/org/apache/tiles/definition/digester/TestDigesterDefinitionsReader.java (original)
+++ tiles/framework/branches/TILES_2_0_X/tiles-core/src/test/java/org/apache/tiles/definition/digester/TestDigesterDefinitionsReader.java Wed Oct 17 01:16:42 2007
@@ -198,11 +198,10 @@
* configuration file with invalid XML.
*/
public void testValidatingParameter() {
+ // Testing with default (validation ON).
try {
DefinitionsReader reader = new DigesterDefinitionsReader();
Map<String, String> params = new HashMap<String, String>();
- params.put(DigesterDefinitionsReader.PARSER_VALIDATE_PARAMETER_NAME,
- "true");
reader.init(params);
URL configFile = this.getClass().getClassLoader().getResource(
@@ -217,6 +216,26 @@
if (LOG.isDebugEnabled()) {
LOG.debug("Exception caught, it is OK", e);
}
+ } catch (Exception e) {
+ fail("Exception reading configuration." + e);
+ }
+
+ // Testing with validation OFF.
+ try {
+ DefinitionsReader reader = new DigesterDefinitionsReader();
+ Map<String, String> params = new HashMap<String, String>();
+ params.put(DigesterDefinitionsReader.PARSER_VALIDATE_PARAMETER_NAME,
+ "false");
+ reader.init(params);
+
+ URL configFile = this.getClass().getClassLoader().getResource(
+ "org/apache/tiles/config/invalid-defs.xml");
+ assertNotNull("Config file not found", configFile);
+
+ InputStream source = configFile.openStream();
+ reader.read(source);
+ } catch (DefinitionsFactoryException e) {
+ fail("Should not have thrown an exception." + e);
} catch (Exception e) {
fail("Exception reading configuration." + e);
}