Updated Branches:
  refs/heads/master c0b25b980 -> 06a726a84

WICKET-4556 Remove IResourceSettings#getUseDefaultResourceAggregator


Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/06a726a8
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/06a726a8
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/06a726a8

Branch: refs/heads/master
Commit: 06a726a84b18dcd3824b9f73b07b28319fe688a8
Parents: c0b25b9
Author: Martin Tzvetanov Grigorov <[email protected]>
Authored: Mon May 14 16:06:08 2012 +0200
Committer: Martin Tzvetanov Grigorov <[email protected]>
Committed: Mon May 14 16:06:08 2012 +0200

----------------------------------------------------------------------
 .../main/java/org/apache/wicket/Application.java   |   12 +++++-------
 .../apache/wicket/settings/IResourceSettings.java  |   15 ---------------
 .../wicket/settings/def/ResourceSettings.java      |   14 --------------
 3 files changed, 5 insertions(+), 36 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/wicket/blob/06a726a8/wicket-core/src/main/java/org/apache/wicket/Application.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/Application.java 
b/wicket-core/src/main/java/org/apache/wicket/Application.java
index 98078ea..bf16fa9 100644
--- a/wicket-core/src/main/java/org/apache/wicket/Application.java
+++ b/wicket-core/src/main/java/org/apache/wicket/Application.java
@@ -1667,18 +1667,16 @@ public abstract class Application implements 
UnboundListener, IEventSink
         *            the response Wicket created
         * @return the response Wicket should use in IHeaderContributor 
traversal
         */
-       public final IHeaderResponse decorateHeaderResponse(IHeaderResponse 
response)
+       public final IHeaderResponse decorateHeaderResponse(final 
IHeaderResponse response)
        {
-               if (getResourceSettings().getUseDefaultResourceAggregator())
-               {
-                       response = new ResourceAggregator(response);
-               }
+               final IHeaderResponse aggregatingResponse = new 
ResourceAggregator(response);
 
                if (headerResponseDecorator == null)
                {
-                       return response;
+                       return aggregatingResponse;
                }
-               return headerResponseDecorator.decorate(response);
+
+               return headerResponseDecorator.decorate(aggregatingResponse);
        }
 
        /**

http://git-wip-us.apache.org/repos/asf/wicket/blob/06a726a8/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java 
b/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java
index 225f41c..122348b 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/settings/IResourceSettings.java
@@ -351,21 +351,6 @@ public interface IResourceSettings extends 
IPropertiesFactoryContext
        ICssCompressor getCssCompressor();
 
        /**
-        * Allows to disable the default {@linkplain ResourceAggregator 
resource aggregation}. If this
-        * setting is disabled, you will have to take care of implementing 
resource dependencies and
-        * bundles yourself.
-        * 
-        * @param useDefaultResourceAggregator
-        *            The new value for the setting
-        */
-       void setUseDefaultResourceAggregator(boolean 
useDefaultResourceAggregator);
-
-       /**
-        * @return Whether to use the default {@linkplain 
org.apache.wicket.markup.head.ResourceAggregator resource aggregator}.
-        */
-       boolean getUseDefaultResourceAggregator();
-
-       /**
         * Sets whether to use pre-minified resources when available. Minified 
resources are detected by
         * name. The minified version of {@code x.js} is expected to be called 
{@code x.min.js}. For css
         * files, the same convention is used: {@code x.min.css} is the 
minified version of

http://git-wip-us.apache.org/repos/asf/wicket/blob/06a726a8/wicket-core/src/main/java/org/apache/wicket/settings/def/ResourceSettings.java
----------------------------------------------------------------------
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/settings/def/ResourceSettings.java
 
b/wicket-core/src/main/java/org/apache/wicket/settings/def/ResourceSettings.java
index 63e78dc..3909335 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/settings/def/ResourceSettings.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/settings/def/ResourceSettings.java
@@ -132,8 +132,6 @@ public class ResourceSettings implements IResourceSettings
        // application these settings are bound to
        private final Application application;
 
-       private boolean useDefaultResourceAggregator = true;
-
        private boolean useMinifiedResources = true;
 
        private Comparator<? super RecordedHeaderItem> headerItemComparator = 
new PriorityFirstComparator(
@@ -559,18 +557,6 @@ public class ResourceSettings implements IResourceSettings
        }
 
        @Override
-       public void setUseDefaultResourceAggregator(boolean 
useDefaultResourceAggregator)
-       {
-               this.useDefaultResourceAggregator = 
useDefaultResourceAggregator;
-       }
-
-       @Override
-       public boolean getUseDefaultResourceAggregator()
-       {
-               return useDefaultResourceAggregator;
-       }
-
-       @Override
        public void setUseMinifiedResources(boolean useMinifiedResources)
        {
                this.useMinifiedResources = useMinifiedResources;

Reply via email to