You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2015/11/18 23:32:06 UTC
[1/2] wicket git commit: Mark ConditionalCommentFilter as deprecated
Repository: wicket
Updated Branches:
refs/heads/wicket-7.x a7acc843d -> 89c285a15
Mark ConditionalCommentFilter as deprecated
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/a91ed6fd
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/a91ed6fd
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/a91ed6fd
Branch: refs/heads/wicket-7.x
Commit: a91ed6fdbb6de798a7e0dc9332699a6d9e79beac
Parents: a7acc84
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Wed Nov 18 23:31:33 2015 +0100
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Wed Nov 18 23:31:33 2015 +0100
----------------------------------------------------------------------
.../wicket/markup/parser/filter/ConditionalCommentFilter.java | 2 ++
1 file changed, 2 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/a91ed6fd/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/ConditionalCommentFilter.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/ConditionalCommentFilter.java b/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/ConditionalCommentFilter.java
index 0fc289a..b677f7a 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/ConditionalCommentFilter.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/ConditionalCommentFilter.java
@@ -37,7 +37,9 @@ import org.slf4j.LoggerFactory;
*
* @author Pedro Santos
* @author Juergen Donnerstag
+ * @deprecated This class is removed in Wicket 8.0. It did nothing since a long time
*/
+@Depracated
public class ConditionalCommentFilter extends AbstractMarkupFilter
{
private static final Logger log = LoggerFactory.getLogger(ConditionalCommentFilter.class);
[2/2] wicket git commit: WICKET-6036 Failure to process markup with
nested tags inside a Label
Posted by mg...@apache.org.
WICKET-6036 Failure to process markup with nested tags inside a Label
Move the new test from ComponentQueueingCompatibilityTest to ComponentQueueingTest.
Fix the exception message.
Remove some trailing spaces.
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/89c285a1
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/89c285a1
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/89c285a1
Branch: refs/heads/wicket-7.x
Commit: 89c285a1520e9d008fff9f4452fab6ba611b17da
Parents: a91ed6f
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Wed Nov 18 23:28:41 2015 +0100
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Wed Nov 18 23:31:57 2015 +0100
----------------------------------------------------------------------
.../java/org/apache/wicket/MarkupContainer.java | 5 ++-
.../markup/parser/filter/HtmlHandler.java | 2 +-
.../ComponentQueueingCompatibilityTest.java | 33 --------------------
.../wicket/queueing/ComponentQueueingTest.java | 11 ++++++-
4 files changed, 13 insertions(+), 38 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/89c285a1/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java b/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
index 2d3a118..6dd9d7e 100644
--- a/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
+++ b/wicket-core/src/main/java/org/apache/wicket/MarkupContainer.java
@@ -2111,7 +2111,7 @@ public abstract class MarkupContainer extends Component implements Iterable<Comp
if (child != null)
{
- addDequeuedComponent(child, tag);
+ addDequeuedComponent(child, tag);
}
}
@@ -2165,8 +2165,7 @@ public abstract class MarkupContainer extends Component implements Iterable<Comp
}
} while ((close = dequeue.takeTag()) != null);
- throw new IllegalStateException(String.format(
- "Could not find the closing for '%s'", tag));
+ throw new IllegalStateException(String.format("Could not find the closing tag for '%s'", tag));
}
/** @see IQueueRegion#newDequeueContext() */
http://git-wip-us.apache.org/repos/asf/wicket/blob/89c285a1/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/HtmlHandler.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/HtmlHandler.java b/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/HtmlHandler.java
index 9288409..c5c6f89 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/HtmlHandler.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/parser/filter/HtmlHandler.java
@@ -110,7 +110,7 @@ public final class HtmlHandler extends AbstractMarkupFilter
setContainsWicketIdFlag(tag);
// Push onto stack
- stack.push(tag);
+ stack.push(tag);
}
else if (tag.isClose())
{
http://git-wip-us.apache.org/repos/asf/wicket/blob/89c285a1/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingCompatibilityTest.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingCompatibilityTest.java b/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingCompatibilityTest.java
deleted file mode 100644
index 7cc703b..0000000
--- a/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingCompatibilityTest.java
+++ /dev/null
@@ -1,33 +0,0 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements. See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You under the Apache License, Version 2.0
- * (the "License"); you may not use this file except in compliance with
- * the License. You may obtain a copy of the License at
- *
- * http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-package org.apache.wicket.queueing;
-
-import org.apache.wicket.util.tester.WicketTestCase;
-import org.junit.Test;
-
-public class ComponentQueueingCompatibilityTest extends WicketTestCase
-{
- /**
- * https://issues.apache.org/jira/browse/WICKET-6036
- */
- @Test
- public void nestedTags()
- {
- IncorrectCloseTagPanel p = new IncorrectCloseTagPanel("test");
- tester.startComponentInPage(p);
- }
-}
http://git-wip-us.apache.org/repos/asf/wicket/blob/89c285a1/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingTest.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingTest.java b/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingTest.java
index 463999d..3039a27 100644
--- a/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingTest.java
+++ b/wicket-core/src/test/java/org/apache/wicket/queueing/ComponentQueueingTest.java
@@ -19,7 +19,6 @@ package org.apache.wicket.queueing;
import static org.apache.wicket.queueing.WicketMatchers.hasPath;
import static org.hamcrest.Matchers.is;
import static org.hamcrest.Matchers.nullValue;
-import static org.junit.Assert.*;
import java.util.ArrayList;
@@ -723,6 +722,16 @@ public class ComponentQueueingTest extends WicketTestCase
tester.assertContains("title");
}
+ /**
+ * https://issues.apache.org/jira/browse/WICKET-6036
+ */
+ @Test
+ public void nestedTags()
+ {
+ IncorrectCloseTagPanel p = new IncorrectCloseTagPanel("test");
+ tester.startComponentInPage(p);
+ }
+
private static class A extends WebMarkupContainer
{
public A()