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 2009/06/15 23:12:59 UTC
svn commit: r784982 - in /tiles/framework/branches/TILES_2_1_X/tiles-core:
./ src/main/java/org/apache/tiles/definition/dao/
src/main/java/org/apache/tiles/definition/digester/
src/test/java/org/apache/tiles/config/
src/test/java/org/apache/tiles/defin...
Author: apetrelli
Date: Mon Jun 15 21:12:59 2009
New Revision: 784982
URL: http://svn.apache.org/viewvc?rev=784982&view=rev
Log:
TILES-416
Merge from trunk to TILES_2_1_X branch.
Applied patch by Lukasz Racon.
Modified:
tiles/framework/branches/TILES_2_1_X/tiles-core/ (props changed)
tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/dao/CachingLocaleUrlDefinitionDAO.java
tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java
tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/config/defs-wildcard.xml
tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/definition/MockDefinitionsReader.java
Propchange: tiles/framework/branches/TILES_2_1_X/tiles-core/
------------------------------------------------------------------------------
--- svn:mergeinfo (added)
+++ svn:mergeinfo Mon Jun 15 21:12:59 2009
@@ -0,0 +1 @@
+/tiles/framework/trunk/tiles-core:749981,761052,761061,772781,784975
Modified: tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/dao/CachingLocaleUrlDefinitionDAO.java
URL: http://svn.apache.org/viewvc/tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/dao/CachingLocaleUrlDefinitionDAO.java?rev=784982&r1=784981&r2=784982&view=diff
==============================================================================
--- tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/dao/CachingLocaleUrlDefinitionDAO.java (original)
+++ tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/dao/CachingLocaleUrlDefinitionDAO.java Mon Jun 15 21:12:59 2009
@@ -25,6 +25,7 @@
import java.net.URL;
import java.util.ArrayList;
import java.util.HashMap;
+import java.util.LinkedHashMap;
import java.util.List;
import java.util.Locale;
import java.util.Map;
@@ -234,7 +235,7 @@
String postfix = LocaleUtil.calculatePostfix(customizationKey);
Locale parentLocale = LocaleUtil.getParentLocale(customizationKey);
- localeDefsMap = new HashMap<String, Definition>();
+ localeDefsMap = new LinkedHashMap<String, Definition>();
if (parentLocale != null) {
Map<String, Definition> parentDefs = loadParentDefinitions(parentLocale);
if (parentDefs != null) {
@@ -356,7 +357,7 @@
for (String attributeName : localAttributeNames) {
Attribute attr = d.getLocalAttribute(attributeName);
Attribute nuattr = replaceVarsInAttribute(attr, vars);
-
+
nudef.putAttribute(replace(attributeName, vars), nuattr);
}
}
Modified: tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java
URL: http://svn.apache.org/viewvc/tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java?rev=784982&r1=784981&r2=784982&view=diff
==============================================================================
--- tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java (original)
+++ tiles/framework/branches/TILES_2_1_X/tiles-core/src/main/java/org/apache/tiles/definition/digester/DigesterDefinitionsReader.java Mon Jun 15 21:12:59 2009
@@ -24,7 +24,7 @@
import java.io.IOException;
import java.io.InputStream;
import java.net.URL;
-import java.util.HashMap;
+import java.util.LinkedHashMap;
import java.util.Map;
import org.apache.commons.digester.Digester;
@@ -302,7 +302,7 @@
// This is an instance variable instead of a local variable because
// we want to be able to call the addDefinition method to populate it.
// But we reset the Map here, which, of course, has threading implications.
- definitions = new HashMap<String, Definition>();
+ definitions = new LinkedHashMap<String, Definition>();
if (source == null) {
// Perhaps we should throw an exception here.
Modified: tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/config/defs-wildcard.xml
URL: http://svn.apache.org/viewvc/tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/config/defs-wildcard.xml?rev=784982&r1=784981&r2=784982&view=diff
==============================================================================
--- tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/config/defs-wildcard.xml (original)
+++ tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/config/defs-wildcard.xml Mon Jun 15 21:12:59 2009
@@ -44,6 +44,17 @@
<put-attribute name="footer" value="/common/footer.jsp" />
<put-attribute name="body" value="doc.portal.body" />
</definition>
+
+ <!-- Never used on purpose, if there is the TILES-416 the test will fail -->
+ <definition name="test.def*" template="/test{1}.jsp">
+ <put-attribute name="country" value="default"/>
+ <put-attribute name="title" value="Tiles Library Documentation" />
+ <put-attribute name="header" value="/common/header-sub.jsp" />
+ <put-attribute name="menu" value="doc.menu.main" />
+ <put-attribute name="footer" value="/common/footer.jsp" />
+ <put-attribute name="body" value="doc.portal.body" />
+ </definition>
+
<definition name="test.def*.noAttribute" template="/test{1}.jsp" />
</tiles-definitions>
Modified: tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/definition/MockDefinitionsReader.java
URL: http://svn.apache.org/viewvc/tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/definition/MockDefinitionsReader.java?rev=784982&r1=784981&r2=784982&view=diff
==============================================================================
--- tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/definition/MockDefinitionsReader.java (original)
+++ tiles/framework/branches/TILES_2_1_X/tiles-core/src/test/java/org/apache/tiles/definition/MockDefinitionsReader.java Mon Jun 15 21:12:59 2009
@@ -21,7 +21,7 @@
package org.apache.tiles.definition;
-import java.util.Collections;
+import java.util.LinkedHashMap;
import java.util.Map;
import org.apache.tiles.Definition;
@@ -65,7 +65,7 @@
*/
@SuppressWarnings("unchecked")
public Map<String, Definition> read(Object source) {
- return Collections.EMPTY_MAP;
+ return new LinkedHashMap();
}
/**