Spring/spring-framework
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

TilesConfigurer defensively expects null from getResources in case of no resources found

Browse Source 
Also includes order preservation for resource results with Tiles 2 as well as retrieval failure logging with Tiles 3.

Issue: SPR-12362
(cherry picked from commit c8b8dc5)
This commit is contained in:
Juergen Hoeller
2014-11-01 12:55:53 +01:00
parent f812998826
commit d53b67f5cb
4 changed files with 35 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
spring-webmvc/src/main/java/org/springframework/web/servlet/view/tiles2/SpringTilesApplicationContextFactory.java
View File

@@ -1,5 +1,5 @@
/*
* Copyright 2002-2010 the original author or authors.
* Copyright 2002-2014 the original author or authors.
*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -19,8 +19,8 @@ package org.springframework.web.servlet.view.tiles2;
import java.io.IOException;
import java.net.URL;
import java.util.Enumeration;
import java.util.HashSet;
import java.util.LinkedHashMap;
import java.util.LinkedHashSet;
import java.util.Map;
import java.util.Set;
import javax.servlet.ServletContext;
@@ -32,6 +32,8 @@ import org.apache.tiles.servlet.context.ServletTilesApplicationContext;
import org.springframework.core.io.Resource;
import org.springframework.core.io.support.ResourcePatternResolver;
import org.springframework.util.CollectionUtils;
import org.springframework.util.ObjectUtils;
import org.springframework.web.context.support.ServletContextResourcePatternResolver;
/**
@@ -89,7 +91,7 @@ public class SpringTilesApplicationContextFactory extends AbstractTilesApplicati
public URL getResource(String path) throws IOException {
URL retValue = null;
Set<URL> urlSet = getResources(path);
if (urlSet != null && !urlSet.isEmpty()) {
if (!CollectionUtils.isEmpty(urlSet)) {
retValue = urlSet.iterator().next();
}
return retValue;
@@ -99,8 +101,8 @@ public class SpringTilesApplicationContextFactory extends AbstractTilesApplicati
public Set<URL> getResources(String path) throws IOException {
Set<URL> urlSet = null;
Resource[] resources = this.resolver.getResources(path);
if (resources != null && resources.length > 0) {
urlSet = new HashSet<URL>();
if (!ObjectUtils.isEmpty(resources)) {
urlSet = new LinkedHashSet<URL>(resources.length);
for (Resource resource : resources) {
urlSet.add(resource.getURL());
}

6
spring-webmvc/src/main/java/org/springframework/web/servlet/view/tiles2/TilesConfigurer.java
View File

@@ -25,6 +25,7 @@ import java.util.LinkedList;
import java.util.List;
import java.util.Map;
import java.util.Properties;
import java.util.Set;
import javax.servlet.ServletContext;
import javax.servlet.jsp.JspFactory;
@@ -420,7 +421,10 @@ public class TilesConfigurer implements ServletContextAware, InitializingBean, D
try {
List<URL> result = new LinkedList<URL>();
for (String definition : definitions) {
result.addAll(applicationContext.getResources(definition));
Set<URL> resources = applicationContext.getResources(definition);
if (resources != null) {
result.addAll(resources);
}
}
return result;
}