[jira] Created: (WICKET-1752) There is a problem using wicket standard DateTimeField, when on same page there is InMethod DataGrid control

2008-07-17 Thread Emil Genov (JIRA)
There is a problem using wicket standard DateTimeField, when on same page there 
is InMethod DataGrid control


 Key: WICKET-1752
 URL: https://issues.apache.org/jira/browse/WICKET-1752
 Project: Wicket
  Issue Type: Bug
  Components: wicket-datetime, wicket-extensions
Affects Versions: 1.3.4, 1.3.3, 1.3.2
 Environment: Windows XP, Java 5.0, Tomcat 5.5, Firefox 2.13
Reporter: Emil Genov


When on same wicket page we use both DateTimeField and InMethod DataGrid 
control, pressing of drop-down calendar button does not work. It do not show 
drop-down calendar. Source files to reproduce it attached

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (WICKET-1752) There is a problem using wicket standard DateTimeField, when on same page there is InMethod DataGrid control

2008-07-17 Thread Emil Genov (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1752?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Emil Genov updated WICKET-1752:
---

Attachment: TableTestPage.java

 There is a problem using wicket standard DateTimeField, when on same page 
 there is InMethod DataGrid control
 

 Key: WICKET-1752
 URL: https://issues.apache.org/jira/browse/WICKET-1752
 Project: Wicket
  Issue Type: Bug
  Components: wicket-datetime, wicket-extensions
Affects Versions: 1.3.2, 1.3.3, 1.3.4
 Environment: Windows XP, Java 5.0, Tomcat 5.5, Firefox 2.13
Reporter: Emil Genov
 Attachments: TableTestPage.html, TableTestPage.java


 When on same wicket page we use both DateTimeField and InMethod DataGrid 
 control, pressing of drop-down calendar button does not work. It do not show 
 drop-down calendar. Source files to reproduce it attached

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (WICKET-1752) There is a problem using wicket standard DateTimeField, when on same page there is InMethod DataGrid control

2008-07-17 Thread Emil Genov (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1752?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Emil Genov updated WICKET-1752:
---

Attachment: TableTestPage.html

 There is a problem using wicket standard DateTimeField, when on same page 
 there is InMethod DataGrid control
 

 Key: WICKET-1752
 URL: https://issues.apache.org/jira/browse/WICKET-1752
 Project: Wicket
  Issue Type: Bug
  Components: wicket-datetime, wicket-extensions
Affects Versions: 1.3.2, 1.3.3, 1.3.4
 Environment: Windows XP, Java 5.0, Tomcat 5.5, Firefox 2.13
Reporter: Emil Genov
 Attachments: TableTestPage.html, TableTestPage.java


 When on same wicket page we use both DateTimeField and InMethod DataGrid 
 control, pressing of drop-down calendar button does not work. It do not show 
 drop-down calendar. Source files to reproduce it attached

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (WICKET-1737) wicketTester does not find HTML mark-up if custom location is used.

2008-07-17 Thread Karsten Efker (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12614270#action_12614270
 ] 

Karsten Efker commented on WICKET-1737:
---

Yes!

May be this was a refresh problem - now it works!

Thank you very, very much for the fix and best regrads from Germany.

 wicketTester does not find HTML mark-up if custom location is used.
 ---

 Key: WICKET-1737
 URL: https://issues.apache.org/jira/browse/WICKET-1737
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.4
Reporter: Karsten Efker
Assignee: Timo Rantalaiho
Priority: Minor
 Attachments: sample1737-1.3-SNAPSHOT.zip, sample1737.zip


 If a custom HTML mark-up location is used as described in the ApacheWicket 
 Wiki  the wicketTester does not find mark-up to render pages.
 refer to:
 http://cwiki.apache.org/WICKET/control-where-html-files-are-loaded-from.html#ControlwhereHTMLfilesareloadedfrom-InWicket1.3
 Nevertheless running the application with ApacheTomcat 6.0.16 everything 
 renders fine.
 Remark:
 wicketTester has been instanciated using custom test application extended 
 from the application used, overriding newSession for different purpose.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Commented: (WICKET-1737) wicketTester does not find HTML mark-up if custom location is used.

2008-07-17 Thread Karsten Efker (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-1737?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12614284#action_12614284
 ] 

Karsten Efker commented on WICKET-1737:
---

I just added your fix to your live project and now everything works fine with 
JUnit.
Thank you, once again for your support.

 wicketTester does not find HTML mark-up if custom location is used.
 ---

 Key: WICKET-1737
 URL: https://issues.apache.org/jira/browse/WICKET-1737
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.4
Reporter: Karsten Efker
Assignee: Timo Rantalaiho
Priority: Minor
 Attachments: sample1737-1.3-SNAPSHOT.zip, sample1737.zip


 If a custom HTML mark-up location is used as described in the ApacheWicket 
 Wiki  the wicketTester does not find mark-up to render pages.
 refer to:
 http://cwiki.apache.org/WICKET/control-where-html-files-are-loaded-from.html#ControlwhereHTMLfilesareloadedfrom-InWicket1.3
 Nevertheless running the application with ApacheTomcat 6.0.16 everything 
 renders fine.
 Remark:
 wicketTester has been instanciated using custom test application extended 
 from the application used, overriding newSession for different purpose.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Closed: (WICKET-1737) wicketTester does not find HTML mark-up if custom location is used.

2008-07-17 Thread Karsten Efker (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1737?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Karsten Efker closed WICKET-1737.
-

   Resolution: Fixed
Fix Version/s: 1.3.5

 wicketTester does not find HTML mark-up if custom location is used.
 ---

 Key: WICKET-1737
 URL: https://issues.apache.org/jira/browse/WICKET-1737
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.4
Reporter: Karsten Efker
Assignee: Timo Rantalaiho
Priority: Minor
 Fix For: 1.3.5

 Attachments: sample1737-1.3-SNAPSHOT.zip, sample1737.zip


 If a custom HTML mark-up location is used as described in the ApacheWicket 
 Wiki  the wicketTester does not find mark-up to render pages.
 refer to:
 http://cwiki.apache.org/WICKET/control-where-html-files-are-loaded-from.html#ControlwhereHTMLfilesareloadedfrom-InWicket1.3
 Nevertheless running the application with ApacheTomcat 6.0.16 everything 
 renders fine.
 Remark:
 wicketTester has been instanciated using custom test application extended 
 from the application used, overriding newSession for different purpose.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



svn commit: r677632 - in /wicket/trunk/wicket/src/main/java/org/apache/wicket/model: ./ util/

2008-07-17 Thread ivaynberg
Author: ivaynberg
Date: Thu Jul 17 09:31:56 2008
New Revision: 677632

URL: http://svn.apache.org/viewvc?rev=677632view=rev
Log:
model cleanup

Added:
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/AbstractCollectionModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/CollectionModelBase.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/CollectionModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/CollectionModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/ListModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/ListModel.java
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/MapModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/MapModel.java
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/SetModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/SetModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/WildcardCollectionModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/WildcardCollectionModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/WildcardListModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/WildcardListModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/util/WildcardSetModel.java
  - copied, changed from r677623, 
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/WildcardSetModel.java
Removed:

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/CollectionModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/CollectionModelBase.java
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/ListModel.java
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/MapModel.java
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/SetModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/WildcardCollectionModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/WildcardListModel.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/model/WildcardSetModel.java
Modified:
wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java

Modified: wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java?rev=677632r1=677631r2=677632view=diff
==
--- wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java 
(original)
+++ wicket/trunk/wicket/src/main/java/org/apache/wicket/model/Model.java Thu 
Jul 17 09:31:56 2008
@@ -25,14 +25,18 @@
 
 import org.apache.wicket.Component;
 import org.apache.wicket.WicketRuntimeException;
+import org.apache.wicket.model.util.MapModel;
+import org.apache.wicket.model.util.WildcardCollectionModel;
+import org.apache.wicket.model.util.WildcardListModel;
+import org.apache.wicket.model.util.WildcardSetModel;
 import org.apache.wicket.util.lang.Objects;
 
 
 /**
- * codeModel/code is the basic implementation of an codeIModel/code. 
It just wraps a
- * simple model object. The model object must be serializable, as it is stored 
in the session. If
- * you have large objects to store, consider using
- * [EMAIL PROTECTED] org.apache.wicket.model.LoadableDetachableModel} instead 
of this class.
+ * codeModel/code is the basic implementation of an codeIModel/code. 
It just wraps a simple
+ * model object. The model object must be serializable, as it is stored in the 
session. If you have
+ * large objects to store, consider using [EMAIL PROTECTED] 
org.apache.wicket.model.LoadableDetachableModel}
+ * instead of this class.
  * 
  * @author Chris Turner
  * @author Eelco Hillenius
@@ -66,25 +70,29 @@
}
 
/**
+* @param K
+*type of key inside map
+* @param V
+*type of value inside map
 * @param map
 *The Map, which may or may not be Serializable
 * @deprecated see [EMAIL PROTECTED] Model#of(Map)}
 * @return A Model object wrapping the Map
 */
-   @SuppressWarnings(unchecked)
@Deprecated
-public static K, V IModelMapK, V valueOf(final MapK, V map)
+   public static K, V IModelMapK, V valueOf(final MapK, V map)
{
-return of(map);
+   return of(map);
}
 
/**
+* @param C
+*type of object inside list
 * @param list
 *  

[jira] Assigned: (WICKET-1749) Want to add SignInPanel_ja.html

2008-07-17 Thread Igor Vaynberg (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1749?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Igor Vaynberg reassigned WICKET-1749:
-

Assignee: Martijn Dashorst

 Want to add SignInPanel_ja.html
 ---

 Key: WICKET-1749
 URL: https://issues.apache.org/jira/browse/WICKET-1749
 Project: Wicket
  Issue Type: Improvement
  Components: wicket-auth-roles
Affects Versions: 1.3.4
 Environment: platform independent.
Reporter: Ryo Ichinose
Assignee: Martijn Dashorst
Priority: Minor
 Attachments: SignInPanel_ja.html

   Original Estimate: 24h
  Remaining Estimate: 24h

 There are several localized version of SignInPanel.html in current 
 wicket-auth-roles package, but SignInPanel_ja.html (japanese version of 
 SignInPanel.html) is not introduced yet...
 I want developers of wicket-auth-roles to add SignInPanel_ja.html, which 
 might make japanese users have more fun with the package.
 I can provide SignInPanel_ja.html if necessary (e.g. there are no japanese 
 speakers among developers).
 Thanks.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



svn commit: r677653 - in /wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http: WicketFilter.java WicketServlet.java

2008-07-17 Thread ivaynberg
Author: ivaynberg
Date: Thu Jul 17 10:54:58 2008
New Revision: 677653

URL: http://svn.apache.org/viewvc?rev=677653view=rev
Log:
WICKET-1753

Modified:

wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketFilter.java

wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketServlet.java

Modified: 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketFilter.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketFilter.java?rev=677653r1=677652r2=677653view=diff
==
--- 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketFilter.java
 (original)
+++ 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketFilter.java
 Thu Jul 17 10:54:58 2008
@@ -21,7 +21,9 @@
 import java.io.UnsupportedEncodingException;
 import java.text.ParseException;
 import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.Properties;
+import java.util.Set;
 
 import javax.servlet.Filter;
 import javax.servlet.FilterChain;
@@ -57,6 +59,13 @@
 /**
  * Filter for initiating handling of Wicket requests.
  * 
+ * @see WicketServlet for documentation
+ * 
+ * @author Jonathan Locke
+ * @author Timur Mehrvarz
+ * @author Juergen Donnerstag
+ * @author Igor Vaynberg (ivaynberg)
+ * @author Al Maw
  * @author jcompagner
  */
 public class WicketFilter implements Filter
@@ -74,6 +83,13 @@
/** Log. */
private static final Logger log = 
LoggerFactory.getLogger(WicketFilter.class);
 
+
+   /**
+* Name of parameter used to express a comma separated list of paths 
that should be ignored
+*/
+   public static final String IGNORE_PATHS_PARAM = ignorePaths;
+
+
/**
 * The servlet path holder when the WicketSerlvet is used. So that the 
filter path will be
 * computed with the first request. Note: This variable is by purpose 
package protected. See
@@ -133,6 +149,9 @@
 */
private boolean portletOnlyFilter;
 
+   /** set of paths that should be ignored by the wicket filter */
+   private final SetString ignorePaths = new HashSetString();
+
/**
 * Servlet cleanup.
 */
@@ -177,7 +196,21 @@
return;
}
 
-   String relativePath = getRelativePath(httpServletRequest);
+   final String relativePath = getRelativePath(httpServletRequest);
+
+   if (ignorePaths.size()  0  relativePath.length()  0)
+   {
+   for (String path : ignorePaths)
+   {
+   if (relativePath.startsWith(path))
+   {
+   log.debug(Ignoring request {}, 
httpServletRequest.getRequestURL());
+   chain.doFilter(request, response);
+   return;
+   }
+   }
+   }
+
 
if (isWicketRequest(relativePath))
{
@@ -489,6 +522,8 @@
 */
public void init(FilterConfig filterConfig) throws ServletException
{
+   initIgnorePaths(filterConfig);
+
this.filterConfig = filterConfig;
String filterMapping = 
filterConfig.getInitParameter(WicketFilter.FILTER_MAPPING_PARAM);
 
@@ -591,6 +626,28 @@
}
}
 
+   /**
+* initializes the ignore paths parameter
+* 
+* @param filterConfig
+*/
+   private void initIgnorePaths(FilterConfig filterConfig)
+   {
+   String paths = 
filterConfig.getInitParameter(IGNORE_PATHS_PARAM);
+   if (!Strings.isEmpty(paths))
+   {
+   String[] parts = paths.split(,);
+   for (String path : parts)
+   {
+   if (path.startsWith(/))
+   {
+   path = path.substring(1);
+   }
+   ignorePaths.add(path);
+   }
+   }
+   }
+
protected boolean isPortletContextAvailable(FilterConfig config) throws 
ServletException
{
boolean detectPortletContext = false;

Modified: 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketServlet.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketServlet.java?rev=677653r1=677652r2=677653view=diff
==
--- 
wicket/trunk/wicket/src/main/java/org/apache/wicket/protocol/http/WicketServlet.java
 (original)
+++ 

[jira] Assigned: (WICKET-1752) There is a problem using wicket standard DateTimeField, when on same page there is InMethod DataGrid control

2008-07-17 Thread Igor Vaynberg (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1752?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Igor Vaynberg reassigned WICKET-1752:
-

Assignee: Matej Knopp

 There is a problem using wicket standard DateTimeField, when on same page 
 there is InMethod DataGrid control
 

 Key: WICKET-1752
 URL: https://issues.apache.org/jira/browse/WICKET-1752
 Project: Wicket
  Issue Type: Bug
  Components: wicket-datetime, wicket-extensions
Affects Versions: 1.3.2, 1.3.3, 1.3.4
 Environment: Windows XP, Java 5.0, Tomcat 5.5, Firefox 2.13
Reporter: Emil Genov
Assignee: Matej Knopp
 Attachments: TableTestPage.html, TableTestPage.java


 When on same wicket page we use both DateTimeField and InMethod DataGrid 
 control, pressing of drop-down calendar button does not work. It do not show 
 drop-down calendar. Source files to reproduce it attached

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



[jira] Updated: (WICKET-1737) wicketTester does not find HTML mark-up if custom location is used.

2008-07-17 Thread Timo Rantalaiho (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1737?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Timo Rantalaiho updated WICKET-1737:


Affects Version/s: 1.4-M3
Fix Version/s: 1.4-M4

You're welcome, and thanks to your for reporting and testing. I now applied 
this to trunk (1.4) as well.

 wicketTester does not find HTML mark-up if custom location is used.
 ---

 Key: WICKET-1737
 URL: https://issues.apache.org/jira/browse/WICKET-1737
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.4, 1.4-M3
Reporter: Karsten Efker
Assignee: Timo Rantalaiho
Priority: Minor
 Fix For: 1.3.5, 1.4-M4

 Attachments: sample1737-1.3-SNAPSHOT.zip, sample1737.zip


 If a custom HTML mark-up location is used as described in the ApacheWicket 
 Wiki  the wicketTester does not find mark-up to render pages.
 refer to:
 http://cwiki.apache.org/WICKET/control-where-html-files-are-loaded-from.html#ControlwhereHTMLfilesareloadedfrom-InWicket1.3
 Nevertheless running the application with ApacheTomcat 6.0.16 everything 
 renders fine.
 Remark:
 wicketTester has been instanciated using custom test application extended 
 from the application used, overriding newSession for different purpose.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



svn commit: r677812 - /wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java

2008-07-17 Thread thrantal
Author: thrantal
Date: Thu Jul 17 20:38:42 2008
New Revision: 677812

URL: http://svn.apache.org/viewvc?rev=677812view=rev
Log:
Fixed missing @Overrides, redundant thises and raw types
- no functional changes

Modified:

wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java

Modified: 
wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java?rev=677812r1=677811r2=677812view=diff
==
--- 
wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java
 (original)
+++ 
wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java
 Thu Jul 17 20:38:42 2008
@@ -67,14 +67,16 @@
super(name);
}
 
-   protected void setUp() throws Exception
+   @Override
+protected void setUp() throws Exception
{
super.setUp();
application = new WicketTester();
application.startPage(MockPage.class);
}
 
-   protected void tearDown() throws Exception
+   @Override
+protected void tearDown() throws Exception
{
application.destroy();
}
@@ -86,7 +88,7 @@
{
// Validate the document
String document = 
application.getServletResponse().getDocument();
-   DiffUtil.validatePage(document, this.getClass(), 
MockPage_expectedResult.html, true);
+   DiffUtil.validatePage(document, getClass(), 
MockPage_expectedResult.html, true);
 
// Inspect the page  model
MockPage p = (MockPage)application.getLastRenderedPage();
@@ -102,12 +104,12 @@
session.info(Message);
session.info(Not rendered);
FeedbackMessages feedbackMessages = 
session.getFeedbackMessages();
-   Iterator iterator = feedbackMessages.iterator();
-   FeedbackMessage message = (FeedbackMessage)iterator.next();
+   IteratorFeedbackMessage iterator = 
feedbackMessages.iterator();
+   FeedbackMessage message = iterator.next();
message.markRendered();
feedbackMessages.clear(RENDERED_MESSAGES);
assertEquals(1, feedbackMessages.size());
-   message = (FeedbackMessage)iterator.next();
+   message = iterator.next();
message.markRendered();
feedbackMessages.clear(RENDERED_MESSAGES);
assertEquals(0, feedbackMessages.size());
@@ -124,7 +126,7 @@
// Now request that we click the link
application.setupRequestAndResponse();
MockPage p = (MockPage)application.getLastRenderedPage();
-   Link link = (Link)p.get(actionLink);
+   Link? link = (Link?)p.get(actionLink);
application.getServletRequest().setRequestToComponent(link);
application.processRequestCycle();
 
@@ -139,7 +141,7 @@
 */
// Validate the document
String document = 
application.getServletResponse().getDocument();
-   DiffUtil.validatePage(document, this.getClass(), 
MockPage_expectedResult2.html, true);
+   DiffUtil.validatePage(document, getClass(), 
MockPage_expectedResult2.html, true);
 
// Inspect the page  model
p = (MockPage)application.getLastRenderedPage();




svn commit: r677813 - /wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java

2008-07-17 Thread thrantal
Author: thrantal
Date: Thu Jul 17 20:40:15 2008
New Revision: 677813

URL: http://svn.apache.org/viewvc?rev=677813view=rev
Log:
Fixed indentation to use tab as elsewhere
- no functional changes

Modified:

wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java

Modified: 
wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java?rev=677813r1=677812r2=677813view=diff
==
--- 
wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java
 (original)
+++ 
wicket/trunk/wicket/src/test/java/org/apache/wicket/protocol/http/MockWebApplicationTest.java
 Thu Jul 17 20:40:15 2008
@@ -36,7 +36,7 @@
 /**
  * Simple application that demonstrates the mock http application code (and 
checks that it is
  * working)
- * 
+ *
  * @author Chris Turner
  */
 public class MockWebApplicationTest extends TestCase
@@ -58,9 +58,8 @@
 
/**
 * Create the test.
-* 
-* @param name
-*The test name
+*
+* @param name The test name
 */
public MockWebApplicationTest(String name)
{
@@ -68,7 +67,7 @@
}
 
@Override
-protected void setUp() throws Exception
+   protected void setUp() throws Exception
{
super.setUp();
application = new WicketTester();
@@ -76,7 +75,7 @@
}
 
@Override
-protected void tearDown() throws Exception
+   protected void tearDown() throws Exception
{
application.destroy();
}
@@ -91,7 +90,7 @@
DiffUtil.validatePage(document, getClass(), 
MockPage_expectedResult.html, true);
 
// Inspect the page  model
-   MockPage p = (MockPage)application.getLastRenderedPage();
+   MockPage p = (MockPage) application.getLastRenderedPage();
Assert.assertEquals(Link should have been clicked 0 times, 0, 
p.getLinkClickCount());
}
 
@@ -125,8 +124,8 @@
 
// Now request that we click the link
application.setupRequestAndResponse();
-   MockPage p = (MockPage)application.getLastRenderedPage();
-   Link? link = (Link?)p.get(actionLink);
+   MockPage p = (MockPage) application.getLastRenderedPage();
+   Link? link = (Link?) p.get(actionLink);
application.getServletRequest().setRequestToComponent(link);
application.processRequestCycle();
 
@@ -144,13 +143,13 @@
DiffUtil.validatePage(document, getClass(), 
MockPage_expectedResult2.html, true);
 
// Inspect the page  model
-   p = (MockPage)application.getLastRenderedPage();
+   p = (MockPage) application.getLastRenderedPage();
Assert.assertEquals(Link should have been clicked 1 time, 1, 
p.getLinkClickCount());
}
 
public void testProvidesDefaultResourceFinderIfNotSetByApplication()
{
-
Assert.assertNotNull(Application.get().getResourceSettings().getResourceFinder());
+   
Assert.assertNotNull(Application.get().getResourceSettings().getResourceFinder());
}
 
public void testHonorsResourceFinderSettingsSetByApplication()
@@ -163,14 +162,15 @@
}
 
@Override
-public String toString()
+   public String toString()
{
return customResourceFinder;
}
};
-   WebApplication wicketApplication = new DummyWebApplication() {
+   WebApplication wicketApplication = new DummyWebApplication()
+   {
@Override
-protected void init()
+   protected void init()
{
IResourceSettings resourceSettings = 
getResourceSettings();

resourceSettings.setResourceFinder(customResourceFinder);




[jira] Assigned: (WICKET-1528) ModalWindow / Mousepointer with Opera

2008-07-17 Thread Igor Vaynberg (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1528?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Igor Vaynberg reassigned WICKET-1528:
-

Assignee: Matej Knopp

 ModalWindow / Mousepointer with Opera
 -

 Key: WICKET-1528
 URL: https://issues.apache.org/jira/browse/WICKET-1528
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
 Environment: Opera 9.2x
Reporter: Jan Kriesten
Assignee: Matej Knopp

 With Opera there's always a right arrow over ModalWindow instead of the 
 default.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.



svn commit: r677818 - /wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/link/ExternalLink.java

2008-07-17 Thread ivaynberg
Author: ivaynberg
Date: Thu Jul 17 21:30:42 2008
New Revision: 677818

URL: http://svn.apache.org/viewvc?rev=677818view=rev
Log:
WICKET-1535

Modified:

wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/link/ExternalLink.java

Modified: 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/link/ExternalLink.java
URL: 
http://svn.apache.org/viewvc/wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/link/ExternalLink.java?rev=677818r1=677817r2=677818view=diff
==
--- 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/link/ExternalLink.java
 (original)
+++ 
wicket/trunk/wicket/src/main/java/org/apache/wicket/markup/html/link/ExternalLink.java
 Thu Jul 17 21:30:42 2008
@@ -192,7 +192,7 @@
else
{
// or generate an onclick JS 
handler directly
-   tag.put(onclick, 
window.location.href=' + url + ';);
+   tag.put(onclick, 
window.location.href=' + url + ';return false;);
}
}
}




[jira] Resolved: (WICKET-1535) ExternalLink JavaScript not working in FF 3

2008-07-17 Thread Igor Vaynberg (JIRA)

 [ 
https://issues.apache.org/jira/browse/WICKET-1535?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Igor Vaynberg resolved WICKET-1535.
---

   Resolution: Fixed
Fix Version/s: 1.3.5
 Assignee: Igor Vaynberg

there is nothing wrong with adding return false;

 ExternalLink JavaScript not working in FF 3
 ---

 Key: WICKET-1535
 URL: https://issues.apache.org/jira/browse/WICKET-1535
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.3.3
 Environment: WinXP; Firefox 3.0 Beta 5
Reporter: Gwyn Evans
Assignee: Igor Vaynberg
Priority: Minor
 Fix For: 1.3.5, 1.4-M4


 If the link target isn't a proper anchor, the code generates a JS on-click 
 handler, i.e. 
tag.put(onclick, window.location.href=' + url + ';);
 which generates code such as 
   onclick='window.location.href=http://news.bbc.co.uk;;'
 The problem is that when this is used with a button /in a form/, while this 
 works with IE  FF2, it appears that FF3 needs a 'return false;' at the end 
 of the JS before it works, otherwise it just treats the button as a 'submit' 
 action.
 So, is there any down-side to changing the above line in 
   \org\apache\wicket\markup\html\link\ExternalLink.java
 to be 
   tag.put(onclick, window.location.href=' + url + '; return false;); ?
 /Gwyn

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.