[jira] [Commented] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread JIRA

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

Frédéric Donckels commented on WICKET-4881:
---

I downloaded your project and reproduced the issue.
See here: http://www.screenr.com/GbU7
(sorry about the sound, I turned the microphone off, but that didn't prevent it 
from recording some noise)

 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Comment Edited] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread JIRA

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

Frédéric Donckels edited comment on WICKET-4881 at 11/26/12 8:22 AM:
-

I downloaded your project and reproduced the issue (using mvn jetty:run).
See here: http://www.screenr.com/GbU7
(sorry about the sound, I turned the microphone off, but that didn't prevent it 
from recording some noise)

  was (Author: fdonckels):
I downloaded your project and reproduced the issue.
See here: http://www.screenr.com/GbU7
(sorry about the sound, I turned the microphone off, but that didn't prevent it 
from recording some noise)
  
 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread Martin Grigorov (JIRA)

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

Martin Grigorov commented on WICKET-4881:
-

I cannot reproduce the problem here as well.
I use IE9 in compatibility modes and it didn't break with any of them. I cannot 
install IE8 here.
The source code of the demo application is too big to make a full code review. 
A brief look at it didn't show anything suspicious.

 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


git commit: WICKET-4888 introduce FencedFeedbackPanel

2012-11-26 Thread mgrigorov
Updated Branches:
  refs/heads/master a0b2dd194 - e8db4bbfb


WICKET-4888 introduce FencedFeedbackPanel

Make the metadata key final.


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

Branch: refs/heads/master
Commit: e8db4bbfb64443252ec2906d308f0df80b54286e
Parents: a0b2dd1
Author: Martin Tzvetanov Grigorov mgrigo...@apache.org
Authored: Mon Nov 26 10:12:16 2012 +0100
Committer: Martin Tzvetanov Grigorov mgrigo...@apache.org
Committed: Mon Nov 26 10:12:16 2012 +0100

--
 .../wicket/feedback/FencedFeedbackPanel.java   |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/wicket/blob/e8db4bbf/wicket-core/src/main/java/org/apache/wicket/feedback/FencedFeedbackPanel.java
--
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/feedback/FencedFeedbackPanel.java 
b/wicket-core/src/main/java/org/apache/wicket/feedback/FencedFeedbackPanel.java
index c39250a..9b60ea1 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/feedback/FencedFeedbackPanel.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/feedback/FencedFeedbackPanel.java
@@ -79,7 +79,7 @@ public class FencedFeedbackPanel extends FeedbackPanel
 {
private static final long serialVersionUID = 1L;
 
-   private static MetaDataKeyInteger FENCE_KEY = new 
MetaDataKeyInteger()
+   private static final MetaDataKeyInteger FENCE_KEY = new 
MetaDataKeyInteger()
{
private static final long serialVersionUID = 1L;
};



[jira] [Commented] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread Andrea Del Bene (JIRA)

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

Andrea Del Bene commented on WICKET-4881:
-

I suspect you may have a more general problem with AJAX and IE. Does the rest 
of the application work fine with AJAX? Does data pagination return any error?

 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Created] (WICKET-4889) Label constructor should accept Serializable as label

2012-11-26 Thread Emond Papegaaij (JIRA)
Emond Papegaaij created WICKET-4889:
---

 Summary: Label constructor should accept Serializable as label
 Key: WICKET-4889
 URL: https://issues.apache.org/jira/browse/WICKET-4889
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 6.3.0
Reporter: Emond Papegaaij
Assignee: Emond Papegaaij


The fixed-label constructor of Label current only accepts strings, which is 
very limited. Label has no problems in rendering other serializable fixed 
values through the use of converters.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread JIRA

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

Frédéric Donckels commented on WICKET-4881:
---

Copy-pasted from the ##wicket IRC channel :
inside the catch block of FunctionsExecuter, the exception message is Out of 
stack space
depth is 41 / current is 41
of course ,  I have no idea how this is supposed to work...
the call stack is just a pretty large bunch of anonymous JScript functions

actually, the issue happens in the processEvaluation function, of 
Wicket.Ajax.Call when calling eval on the Javascript provided in the Ajax 
response
It's odd though: if I change the number of items per page in the pagination, 
everything is ok until itemsPerPage = 8 (lower is ok)

 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread Martin Grigorov (JIRA)

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

Martin Grigorov commented on WICKET-4881:
-

Can you try the JavaScript from this comment
https://issues.apache.org/jira/browse/WICKET-4675?focusedCommentId=13426609page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-13426609
in your IE8 instance ?
What is the number that it gives ?

 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread JIRA

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

Frédéric Donckels commented on WICKET-4881:
---

LOG: 272 and then it outputs Out of memory (which is pretty odd and a 
different message). That seems quite low to run out of stack space in a Win7 64 
bit environment and 8gb of memory...

 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (WICKET-4881) IE 8 : error when handling Wicket Ajax Response

2012-11-26 Thread Martin Grigorov (JIRA)

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

Martin Grigorov commented on WICKET-4881:
-

Each browser has different capabilities.
See 
http://stackoverflow.com/questions/7826992/browser-javascript-stack-size-limit

It depends on how much memory you use in each stack frame.
The test that you ran is very simple and I'm not sure why 272 is the max for 
you. It could be some configuration property of IE ...

The only solution that I see is to use setTimeout() when the depth is lower. 
The current threshold is 1000 recursive calls. Pre-Wicket-4675 it was 50. But 
it fails for you at 41 which indeed is very low.

 IE 8 : error when handling Wicket Ajax Response
 ---

 Key: WICKET-4881
 URL: https://issues.apache.org/jira/browse/WICKET-4881
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0, 6.3.0
 Environment: Internet Explorer 8, Windows 7 64 bits
Reporter: Frédéric Donckels
 Attachments: wicket_bug_report.7z, wicket_bug_report.zip


 I have a problem with Wicket 6.2.0/6.3.0 and IE (this works perfectly with 
 Chrome or Firefox). I have a list of items, with Ajax links in the list. The 
 response to the Ajax link modifies a panel from the page, and redraws the 
 list (its container).
 IE does at least some of the changes correctly, but outputs an error, and it 
 seems that subsequent Ajax request fail:
(... some UI code ...)
(... Jquery and Wicket ajax references ...)
 script type=text/javascript id=wicket-ajax-base-url
 /*![CDATA[*/
 Wicket.Ajax.baseUrl=other/bictables/list;
 /*]^]^*/
 /script
 /head]]/header-contributionevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-AMEND,e:click,c:AMENDea});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-interaction-view-toolbar-RETURN,e:click,c:RETURNec});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-view,e:click,c:viewf2});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-actions-edit,e:click,c:editf3});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-0-reorder-down,e:click,c:downf4});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-view,e:click,c:viewf5});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-actions-edit,e:click,c:editf6});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-up,e:click,c:upf7});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-1-reorder-down,e:click,c:downf8});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-view,e:click,c:viewf9});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-2-actions-edit,e:click,c:editfa});]]
(... lots of similar code from the table ...)
 evaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-actions-edit,e:click,c:edit136});]]/evaluateevaluate![CDATA[Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-17-reorder-up,e:click,c:up137});]]/evaluate/ajax-response
 INFO: returned focused element: 
 http://localhost:8080/payments/app/other/bictables/list?3#
 INFO: returned focused element: null
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating javascript: 
 [object Error], text: 
 Wicket.Ajax.ajax({u:./list?3-1.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-view,e:click,c:view10d});
 ERROR: FunctionsExecuter.processNext: [object Error]
 ERROR: Wicket.Ajax.Call.processEvaluation: Exception evaluating 
 javascript: [object Error], text: 
 Wicket.Ajax.ajax({u:./list?2-4.IBehaviorListener.0-entities-entitiesList-bictables-7-actions-
   view,e:click,c:view2cb});
 ERROR: FunctionsExecuter.processNext: [object Error]
 all subsequent Ajax requests lead to :
 INFO: Channel '0' is busy - scheduling the callback to be executed when the 
 previous request finish
 Is this a bug in Wicket, in IE? Any way to circumvent it if it's in IE?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Created] (WICKET-4890) Bad validation messages after WICKET-2128

2012-11-26 Thread Jonas (JIRA)
Jonas created WICKET-4890:
-

 Summary: Bad validation messages after WICKET-2128
 Key: WICKET-4890
 URL: https://issues.apache.org/jira/browse/WICKET-4890
 Project: Wicket
  Issue Type: Bug
Reporter: Jonas
Priority: Minor


In most languages (at least the ones that I speak), the error messages used by 
validators don't sound right anymore after WICKET-2128 was fixed. Replacing 
'${input}' with '${label}' (what seems to have been done) doesn't yield useful 
error messages.

I see the following options:
a) Improve the messages the way Martin Grigorov suggested it in WICKET-2128, 
e.g.
'IConverter=The value of '${label}' is not a valid ${type}.' 
(instead of just 'IConverter='${label}' is not a valid ${type}.')
Interestingly, all the languages that use .xml properties (e.g. 
Application_bg.properties.xml) files seem to have been excluded (by accident?) 
from the changes done in WICKET-2128, as they still refer to ${input}.

b) revert the changes from WICKET-2128 completely. I think the new messages 
only fit well in scenarios where there's only a single feedback panel per page, 
which is anyway not very userfriendly. In cases when you display the error 
messages in separate feedback components next to the corresponding 
FormComponents, it is anyway clear which field is the culprit, and users don't 
need to search for the field that has caused the error.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Created] (WICKET-4891) UrlRenderer.renderRelativeUrl misbehavior if the filterPath is composed.

2012-11-26 Thread Jordi Deu-Pons (JIRA)
Jordi Deu-Pons created WICKET-4891:
--

 Summary: UrlRenderer.renderRelativeUrl misbehavior if the 
filterPath is composed.
 Key: WICKET-4891
 URL: https://issues.apache.org/jira/browse/WICKET-4891
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.3.0
Reporter: Jordi Deu-Pons
Priority: Minor


Using an application with a composed filterPath like:

filter-mapping
filter-namewicket.jsessionid/filter-name
url-pattern/demo/test/*/url-pattern
/filter-mapping 

Wicket fails to properly render a bookmarkable link.

I've debuged my app and the problem is here:

UrlRenderer.removeCommonPrefixes()
Line 309:

String filterPath = request.getFilterPath();
if (filterPath != null  segments.isEmpty() == false)
{
if (filterPath.equals(UrlUtils.normalizePath(segments.get(0
{
LOG.debug(Removing the filter path '{}' from '{}', filterPath, 
segments);
segments.remove(0);
}
}

This code assumes that the filter path is a single segment path, not a composed 
path.

I've only reached this bug browsing my website with the cookies disable. This 
is why it's not critical.




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Created] (WICKET-4892) Provide helpful exception message in RequestCycle#urlFor(Class, PageParameters)

2012-11-26 Thread Peter Parson (JIRA)
Peter Parson created WICKET-4892:


 Summary: Provide helpful exception message in 
RequestCycle#urlFor(Class, PageParameters)
 Key: WICKET-4892
 URL: https://issues.apache.org/jira/browse/WICKET-4892
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0
Reporter: Peter Parson


When something goes wrong while trying to get the URL for 
pageClass/pageParameters using RequestCycle, the exception message and its 
stacktrace is not very helpful in tracking down the component actually causing 
the problem. (See attached stacktrace that I while rendering a 
BookmarkablePageLink with a null parameter.)

I suggest to catch any exceptions thrown by urlFor(IRequestHandler) and 
throwing a WicketRuntimeException having pageClass and pageparamters in its 
message. (See attached patch)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (WICKET-4892) Provide helpful exception message in RequestCycle#urlFor(Class, PageParameters)

2012-11-26 Thread Peter Parson (JIRA)

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

Peter Parson updated WICKET-4892:
-

Attachment: urlFor-stacktrace.txt
RequestCycle-urlFor-exception.patch

Example stacktrace and patch for helpful exception message

 Provide helpful exception message in RequestCycle#urlFor(Class, 
 PageParameters)
 ---

 Key: WICKET-4892
 URL: https://issues.apache.org/jira/browse/WICKET-4892
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.2.0
Reporter: Peter Parson
  Labels: exception, exception-handling, patch
 Attachments: RequestCycle-urlFor-exception.patch, 
 urlFor-stacktrace.txt


 When something goes wrong while trying to get the URL for 
 pageClass/pageParameters using RequestCycle, the exception message and its 
 stacktrace is not very helpful in tracking down the component actually 
 causing the problem. (See attached stacktrace that I while rendering a 
 BookmarkablePageLink with a null parameter.)
 I suggest to catch any exceptions thrown by urlFor(IRequestHandler) and 
 throwing a WicketRuntimeException having pageClass and pageparamters in its 
 message. (See attached patch)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (WICKET-4892) Provide helpful exception message in RequestCycle#urlFor(Class, PageParameters)

2012-11-26 Thread Peter Parson (JIRA)

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

Peter Parson updated WICKET-4892:
-

Issue Type: Improvement  (was: Bug)

 Provide helpful exception message in RequestCycle#urlFor(Class, 
 PageParameters)
 ---

 Key: WICKET-4892
 URL: https://issues.apache.org/jira/browse/WICKET-4892
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 6.2.0
Reporter: Peter Parson
  Labels: exception, exception-handling, patch
 Attachments: RequestCycle-urlFor-exception.patch, 
 urlFor-stacktrace.txt


 When something goes wrong while trying to get the URL for 
 pageClass/pageParameters using RequestCycle, the exception message and its 
 stacktrace is not very helpful in tracking down the component actually 
 causing the problem. (See attached stacktrace that I while rendering a 
 BookmarkablePageLink with a null parameter.)
 I suggest to catch any exceptions thrown by urlFor(IRequestHandler) and 
 throwing a WicketRuntimeException having pageClass and pageparamters in its 
 message. (See attached patch)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (WICKET-4892) Provide helpful exception message in RequestCycle#urlFor(Class, PageParameters)

2012-11-26 Thread Peter Parson (JIRA)

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

Peter Parson updated WICKET-4892:
-

Priority: Minor  (was: Major)

 Provide helpful exception message in RequestCycle#urlFor(Class, 
 PageParameters)
 ---

 Key: WICKET-4892
 URL: https://issues.apache.org/jira/browse/WICKET-4892
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 6.2.0
Reporter: Peter Parson
Priority: Minor
  Labels: exception, exception-handling, patch
 Attachments: RequestCycle-urlFor-exception.patch, 
 urlFor-stacktrace.txt


 When something goes wrong while trying to get the URL for 
 pageClass/pageParameters using RequestCycle, the exception message and its 
 stacktrace is not very helpful in tracking down the component actually 
 causing the problem. (See attached stacktrace that I while rendering a 
 BookmarkablePageLink with a null parameter.)
 I suggest to catch any exceptions thrown by urlFor(IRequestHandler) and 
 throwing a WicketRuntimeException having pageClass and pageparamters in its 
 message. (See attached patch)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (WICKET-4891) UrlRenderer.renderRelativeUrl misbehavior if the filterPath is composed.

2012-11-26 Thread Jordi Deu-Pons (JIRA)

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

Jordi Deu-Pons updated WICKET-4891:
---

Attachment: WICKET-4891.tar.gz

Minimal example demo application that fails if you browse it with the cookies 
disable

 UrlRenderer.renderRelativeUrl misbehavior if the filterPath is composed.
 

 Key: WICKET-4891
 URL: https://issues.apache.org/jira/browse/WICKET-4891
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.3.0
Reporter: Jordi Deu-Pons
Priority: Minor
 Attachments: WICKET-4891.tar.gz


 Using an application with a composed filterPath like:
 filter-mapping
   filter-namewicket.jsessionid/filter-name
   url-pattern/demo/test/*/url-pattern
 /filter-mapping 
 Wicket fails to properly render a bookmarkable link.
 I've debuged my app and the problem is here:
 UrlRenderer.removeCommonPrefixes()
 Line 309:
 String filterPath = request.getFilterPath();
 if (filterPath != null  segments.isEmpty() == false)
 {
 if (filterPath.equals(UrlUtils.normalizePath(segments.get(0
 {
 LOG.debug(Removing the filter path '{}' from '{}', filterPath, 
 segments);
 segments.remove(0);
 }
 }
 This code assumes that the filter path is a single segment path, not a 
 composed path.
 I've only reached this bug browsing my website with the cookies disable. This 
 is why it's not critical.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (WICKET-4890) Bad validation messages after WICKET-2128

2012-11-26 Thread Jonas (JIRA)

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

Jonas updated WICKET-4890:
--

Affects Version/s: 6.3.0

 Bad validation messages after WICKET-2128
 -

 Key: WICKET-4890
 URL: https://issues.apache.org/jira/browse/WICKET-4890
 Project: Wicket
  Issue Type: Bug
Affects Versions: 6.3.0
Reporter: Jonas
Priority: Minor

 In most languages (at least the ones that I speak), the error messages used 
 by validators don't sound right anymore after WICKET-2128 was fixed. 
 Replacing '${input}' with '${label}' (what seems to have been done) doesn't 
 yield useful error messages.
 I see the following options:
 a) Improve the messages the way Martin Grigorov suggested it in WICKET-2128, 
 e.g.
 'IConverter=The value of '${label}' is not a valid ${type}.' 
 (instead of just 'IConverter='${label}' is not a valid ${type}.')
 Interestingly, all the languages that use .xml properties (e.g. 
 Application_bg.properties.xml) files seem to have been excluded (by 
 accident?) from the changes done in WICKET-2128, as they still refer to 
 ${input}.
 b) revert the changes from WICKET-2128 completely. I think the new messages 
 only fit well in scenarios where there's only a single feedback panel per 
 page, which is anyway not very userfriendly. In cases when you display the 
 error messages in separate feedback components next to the corresponding 
 FormComponents, it is anyway clear which field is the culprit, and users 
 don't need to search for the field that has caused the error.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (WICKET-4891) UrlRenderer.renderRelativeUrl misbehavior if the filterPath is composed.

2012-11-26 Thread Jordi Deu-Pons (JIRA)

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

Jordi Deu-Pons commented on WICKET-4891:


I've created a pull request:
https://github.com/apache/wicket/pull/22

From the source code I can deduce that it can happen the same with the context 
path, so I also check this at the pull request.

 UrlRenderer.renderRelativeUrl misbehavior if the filterPath is composed.
 

 Key: WICKET-4891
 URL: https://issues.apache.org/jira/browse/WICKET-4891
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.3.0
Reporter: Jordi Deu-Pons
Priority: Minor
 Attachments: WICKET-4891.tar.gz


 Using an application with a composed filterPath like:
 filter-mapping
   filter-namewicket.jsessionid/filter-name
   url-pattern/demo/test/*/url-pattern
 /filter-mapping 
 Wicket fails to properly render a bookmarkable link.
 I've debuged my app and the problem is here:
 UrlRenderer.removeCommonPrefixes()
 Line 309:
 String filterPath = request.getFilterPath();
 if (filterPath != null  segments.isEmpty() == false)
 {
 if (filterPath.equals(UrlUtils.normalizePath(segments.get(0
 {
 LOG.debug(Removing the filter path '{}' from '{}', filterPath, 
 segments);
 segments.remove(0);
 }
 }
 This code assumes that the filter path is a single segment path, not a 
 composed path.
 I've only reached this bug browsing my website with the cookies disable. This 
 is why it's not critical.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Updated] (WICKET-4891) UrlRenderer.renderRelativeUrl misbehavior if the filterPath is composed.

2012-11-26 Thread Jordi Deu-Pons (JIRA)

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

Jordi Deu-Pons updated WICKET-4891:
---

Attachment: WICKET-4891.patch

A patch 

 UrlRenderer.renderRelativeUrl misbehavior if the filterPath is composed.
 

 Key: WICKET-4891
 URL: https://issues.apache.org/jira/browse/WICKET-4891
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.3.0
Reporter: Jordi Deu-Pons
Priority: Minor
 Attachments: WICKET-4891.patch, WICKET-4891.tar.gz


 Using an application with a composed filterPath like:
 filter-mapping
   filter-namewicket.jsessionid/filter-name
   url-pattern/demo/test/*/url-pattern
 /filter-mapping 
 Wicket fails to properly render a bookmarkable link.
 I've debuged my app and the problem is here:
 UrlRenderer.removeCommonPrefixes()
 Line 309:
 String filterPath = request.getFilterPath();
 if (filterPath != null  segments.isEmpty() == false)
 {
 if (filterPath.equals(UrlUtils.normalizePath(segments.get(0
 {
 LOG.debug(Removing the filter path '{}' from '{}', filterPath, 
 segments);
 segments.remove(0);
 }
 }
 This code assumes that the filter path is a single segment path, not a 
 composed path.
 I've only reached this bug browsing my website with the cookies disable. This 
 is why it's not critical.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[jira] [Commented] (WICKET-4879) Implementing channels in wicket-atmosphere

2012-11-26 Thread Zala Goupil (JIRA)

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

Zala Goupil commented on WICKET-4879:
-

Good evening,

I've just tried it. The behaviour is the same: if three clients are connected 
and that only one's filter returns true, then the same message goes to all the 
three clients. If two of them return true, then those two send the message to 
the three clients, and so on.

Personally, I'd like to be able to have the message sent by one client 
broadcasted to only this client. Of course, the use case of one client sends 
to all is useful, I just wish it was not the only one possible.

Anyway, your contextAwareFilter is useful, thanks for that !

I've tried myself to implement a Broadcaster-based message-sending in your 
code, but for the moment, I failed. Too bad. The one thing I miss is how to 
provide an AtmosphereResource to my Broadcaster.

Anyway, keep up the good work! :-)

 Implementing channels in wicket-atmosphere
 --

 Key: WICKET-4879
 URL: https://issues.apache.org/jira/browse/WICKET-4879
 Project: Wicket
  Issue Type: New Feature
  Components: wicket-atmosphere
Affects Versions: 6.3.0
 Environment: Tomcat / Jetty
Reporter: Zala Goupil
Assignee: Emond Papegaaij
 Fix For: 6.4.0

 Attachments: wicket-atmosphere-quickstart.zip


 Good evening,
 Now that both Tomcat  Jetty support native WebSockets, it would be a 
 killer-feature to implement Comet / WebSockets channels. i.e. being able to 
 have a channel id and to post a message to one client only or to all the 
 clients who subscribed to a certain channel.
 For the moment, it is possible to do it, but to the (expensive) price of 
 having a PerRequestBroadcastFilter and tweaking it if we manage to do it. But 
 it would be great to have a link to the Broadcaster API from Atmosphere, 
 which I think, supports this kind of things.That would probably be way more 
 efficient than PerRequestBroadcastFilter tweaking.
 As of today, when posting a message to the event bus, all pages who have the 
 @Subscribed-decorated method suitable to process this message will do it, 
 which is not always desirable.
 I'm looking forward to hear from you.
 Regards

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


git commit: WICKET-4879: make it possible to post to a single atmosphere resource

2012-11-26 Thread papegaaij
Updated Branches:
  refs/heads/master e8db4bbfb - a5421ec48


WICKET-4879: make it possible to post to a single atmosphere resource


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

Branch: refs/heads/master
Commit: a5421ec488020a3835d49962e1f772dc1435e230
Parents: e8db4bb
Author: Emond Papegaaij papega...@apache.org
Authored: Mon Nov 26 21:59:15 2012 +0100
Committer: Emond Papegaaij papega...@apache.org
Committed: Mon Nov 26 21:59:22 2012 +0100

--
 .../wicket/atmosphere/AtmosphereBehavior.java  |   31 ---
 .../org/apache/wicket/atmosphere/EventBus.java |   69 +++---
 2 files changed, 72 insertions(+), 28 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/wicket/blob/a5421ec4/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereBehavior.java
--
diff --git 
a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereBehavior.java
 
b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereBehavior.java
index 4d18093..87e47b2 100644
--- 
a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereBehavior.java
+++ 
b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/AtmosphereBehavior.java
@@ -22,6 +22,7 @@ import org.apache.wicket.Application;
 import org.apache.wicket.Component;
 import org.apache.wicket.IResourceListener;
 import org.apache.wicket.MetaDataKey;
+import org.apache.wicket.Page;
 import org.apache.wicket.WicketRuntimeException;
 import org.apache.wicket.ajax.json.JSONException;
 import org.apache.wicket.ajax.json.JSONObject;
@@ -33,7 +34,6 @@ import 
org.apache.wicket.protocol.http.servlet.ServletWebRequest;
 import org.apache.wicket.request.cycle.RequestCycle;
 import org.apache.wicket.request.mapper.parameter.PageParameters;
 import org.apache.wicket.resource.CoreLibrariesContributor;
-import org.atmosphere.cpr.ApplicationConfig;
 import org.atmosphere.cpr.AtmosphereResource;
 import org.atmosphere.cpr.AtmosphereResourceEvent;
 import org.atmosphere.cpr.AtmosphereResourceEventListener;
@@ -105,7 +105,7 @@ public class AtmosphereBehavior extends Behavior
 
// Grab a Meteor
Meteor meteor = Meteor.build(request.getContainerRequest());
-   String uuid = getUUID(meteor.getAtmosphereResource());
+   String uuid = meteor.getAtmosphereResource().uuid();
component.getPage().setMetaData(ATMOSPHERE_UUID, uuid);
findEventBus().registerPage(uuid, component.getPage());
 
@@ -179,7 +179,7 @@ public class AtmosphereBehavior extends Behavior
log.info(String.format(%s connection dropped from ip 
%s:%s, transport == null
? websocket : transport, req.getRemoteAddr(), 
req.getRemotePort()));
}
-   
findEventBus().unregisterConnection(getUUID(event.getResource()));
+   findEventBus().unregisterConnection(event.getResource().uuid());
}
 
@Override
@@ -210,20 +210,25 @@ public class AtmosphereBehavior extends Behavior
}
 
/**
+* Find the Atmosphere UUID for the suspended connection for the given 
page (if any).
+* 
+* @param page
+* @return The UUID of the Atmosphere Resource, or null if no resource 
is suspended for the
+* page.
+*/
+   public static String getUUID(Page page)
+   {
+   return page.getMetaData(ATMOSPHERE_UUID);
+   }
+
+   /**
 * @param resource
 * @return the unique id for the given suspended connection
+* @deprecated use {@link AtmosphereResource#uuid()}
 */
+   @Deprecated
public static String getUUID(AtmosphereResource resource)
{
-   Object trackingId = 
resource.getRequest().getHeader(HeaderConfig.X_ATMOSPHERE_TRACKING_ID);
-   if (trackingId != null  !trackingId.equals(0))
-   return trackingId.toString();
-
-   trackingId = resource.getRequest().getAttribute(
-   ApplicationConfig.SUSPENDED_ATMOSPHERE_RESOURCE_UUID);
-   if (trackingId != null  !trackingId.equals(0))
-   return trackingId.toString();
-
-   return resource.getRequest().getHeader(Sec-WebSocket-Key);
+   return resource.uuid();
}
 }
\ No newline at end of file


git commit: WICKET-4879: null check for the resource

2012-11-26 Thread papegaaij
Updated Branches:
  refs/heads/master a5421ec48 - bef99f313


WICKET-4879: null check for the resource


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

Branch: refs/heads/master
Commit: bef99f3135ee545d11cfad41733a1c6a82f54dfb
Parents: a5421ec
Author: Emond Papegaaij papega...@apache.org
Authored: Mon Nov 26 22:04:06 2012 +0100
Committer: Emond Papegaaij papega...@apache.org
Committed: Mon Nov 26 22:04:06 2012 +0100

--
 .../org/apache/wicket/atmosphere/EventBus.java |9 +++--
 1 files changed, 7 insertions(+), 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/wicket/blob/bef99f31/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
--
diff --git 
a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
 
b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
index f8efd06..4901c7a 100644
--- 
a/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
+++ 
b/wicket-experimental/wicket-atmosphere/src/main/java/org/apache/wicket/atmosphere/EventBus.java
@@ -185,14 +185,19 @@ public class EventBus implements UnboundListener
/**
 * Post an event to a single resource. This will invoke the event 
handlers on all components on
 * the page with the suspended connection. The resulting AJAX update 
(if any) is pushed to the
-* client. You can find the UUID via {@link 
AtmosphereBehavior#getUUID(Page)}.
+* client. You can find the UUID via {@link 
AtmosphereBehavior#getUUID(Page)}. If no resource
+* exists with the given UUID, no post is performed.
 * 
 * @param event
 * @param resourceUuid
 */
public void post(Object event, String resourceUuid)
{
-   post(event, 
AtmosphereResourceFactory.getDefault().find(resourceUuid));
+   AtmosphereResource resource = 
AtmosphereResourceFactory.getDefault().find(resourceUuid);
+   if (resource != null)
+   {
+   post(event, resource);
+   }
}
 
/**



[jira] [Commented] (WICKET-4879) Implementing channels in wicket-atmosphere

2012-11-26 Thread Emond Papegaaij (JIRA)

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

Emond Papegaaij commented on WICKET-4879:
-

I don't see how you could return true for one client and false for another 
(other than using something like random). filter() is used with no receiver 
context at all. It is meant to be used to filter certain types of events (other 
than the actual instance type). For example, if you have an event that contains 
progress of a job and you only want your component to receive the events if the 
progress is 80% or higher, that's when you use filter(). The same filter should 
always return true for the same events.

Can you take a look at commit a5421ec488020a3835d49962e1f772dc1435e230? I've 
added methods to post an event to a single resource, either with a resource 
instance or via its uuid (which I think makes more sense). You can get the UUID 
with AtmosphereResource.getUUID(page).

 Implementing channels in wicket-atmosphere
 --

 Key: WICKET-4879
 URL: https://issues.apache.org/jira/browse/WICKET-4879
 Project: Wicket
  Issue Type: New Feature
  Components: wicket-atmosphere
Affects Versions: 6.3.0
 Environment: Tomcat / Jetty
Reporter: Zala Goupil
Assignee: Emond Papegaaij
 Fix For: 6.4.0

 Attachments: wicket-atmosphere-quickstart.zip


 Good evening,
 Now that both Tomcat  Jetty support native WebSockets, it would be a 
 killer-feature to implement Comet / WebSockets channels. i.e. being able to 
 have a channel id and to post a message to one client only or to all the 
 clients who subscribed to a certain channel.
 For the moment, it is possible to do it, but to the (expensive) price of 
 having a PerRequestBroadcastFilter and tweaking it if we manage to do it. But 
 it would be great to have a link to the Broadcaster API from Atmosphere, 
 which I think, supports this kind of things.That would probably be way more 
 efficient than PerRequestBroadcastFilter tweaking.
 As of today, when posting a message to the event bus, all pages who have the 
 @Subscribed-decorated method suitable to process this message will do it, 
 which is not always desirable.
 I'm looking forward to hear from you.
 Regards

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


git commit: WICKET-4889: added a constructor that takes Serializable

2012-11-26 Thread papegaaij
Updated Branches:
  refs/heads/master bef99f313 - 3d5fba8ca


WICKET-4889: added a constructor that takes Serializable


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

Branch: refs/heads/master
Commit: 3d5fba8cada3fa8aae6ea60cd55af084e1c76fcc
Parents: bef99f3
Author: Emond Papegaaij papega...@apache.org
Authored: Mon Nov 26 22:41:20 2012 +0100
Committer: Emond Papegaaij papega...@apache.org
Committed: Mon Nov 26 22:41:20 2012 +0100

--
 .../org/apache/wicket/markup/html/basic/Label.java |   19 ++-
 1 files changed, 18 insertions(+), 1 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/wicket/blob/3d5fba8c/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
--
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
index 43a7556..b88f44e 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
@@ -16,6 +16,8 @@
  */
 package org.apache.wicket.markup.html.basic;
 
+import java.io.Serializable;
+
 import org.apache.wicket.markup.ComponentTag;
 import org.apache.wicket.markup.MarkupStream;
 import org.apache.wicket.markup.html.WebComponent;
@@ -95,6 +97,21 @@ public class Label extends WebComponent
}
 
/**
+* Convenience constructor. Same as Label(String, 
Model.of(Serializable))
+* 
+* @param id
+*See Component
+* @param label
+*The label text or object, converted to a string via the 
{@link IConverter}.
+* 
+* @see org.apache.wicket.Component#Component(String, IModel)
+*/
+   public Label(final String id, Serializable label)
+   {
+   this(id, Model.of(label));
+   }
+
+   /**
 * @param id
 * @param model
 * @see org.apache.wicket.Component#Component(String, IModel)
@@ -127,4 +144,4 @@ public class Label extends WebComponent
tag.setType(TagType.OPEN);
}
}
-}
\ No newline at end of file
+}



[jira] [Resolved] (WICKET-4889) Label constructor should accept Serializable as label

2012-11-26 Thread Emond Papegaaij (JIRA)

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

Emond Papegaaij resolved WICKET-4889.
-

   Resolution: Fixed
Fix Version/s: 6.4.0

A new constructor is added.

 Label constructor should accept Serializable as label
 -

 Key: WICKET-4889
 URL: https://issues.apache.org/jira/browse/WICKET-4889
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 6.3.0
Reporter: Emond Papegaaij
Assignee: Emond Papegaaij
 Fix For: 6.4.0


 The fixed-label constructor of Label current only accepts strings, which is 
 very limited. Label has no problems in rendering other serializable fixed 
 values through the use of converters.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


git commit: WICKET-4889: added a constructor that takes Serializable

2012-11-26 Thread mgrigorov
Updated Branches:
  refs/heads/master 3d5fba8ca - e9d8e33ff


WICKET-4889: added a constructor that takes Serializable

Fix a javadoc link.
Add a TODO to remove the constructor that accepts String in Wicket 7. 
Deprecating it now will lead to ugly warnings in the user apps.


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

Branch: refs/heads/master
Commit: e9d8e33ffc51651a7862c7213d71058ab2a3c6f2
Parents: 3d5fba8
Author: Martin Tzvetanov Grigorov mgrigo...@apache.org
Authored: Mon Nov 26 22:52:18 2012 +0100
Committer: Martin Tzvetanov Grigorov mgrigo...@apache.org
Committed: Mon Nov 26 22:52:18 2012 +0100

--
 .../org/apache/wicket/markup/html/basic/Label.java |3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/wicket/blob/e9d8e33f/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
--
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
index b88f44e..1423dd5 100644
--- a/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
+++ b/wicket-core/src/main/java/org/apache/wicket/markup/html/basic/Label.java
@@ -91,6 +91,7 @@ public class Label extends WebComponent
 * 
 * @see org.apache.wicket.Component#Component(String, IModel)
 */
+   // TODO Wicket 7 remove this constructor. The one with Serializable is 
the replacement
public Label(final String id, String label)
{
this(id, new ModelString(label));
@@ -102,7 +103,7 @@ public class Label extends WebComponent
 * @param id
 *See Component
 * @param label
-*The label text or object, converted to a string via the 
{@link IConverter}.
+*The label text or object, converted to a string via the 
{@link org.apache.wicket.util.convert.IConverter}.
 * 
 * @see org.apache.wicket.Component#Component(String, IModel)
 */



[jira] [Commented] (WICKET-4879) Implementing channels in wicket-atmosphere

2012-11-26 Thread Zala Goupil (JIRA)

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

Zala Goupil commented on WICKET-4879:
-

Thanks again! I've tried this new code, but unfortunately the situation is the 
same: all the connected resources are refreshed when issuing: 
Application.get().getEventBus().post(MyObject, pageUuid), not just the one that 
I'm expecting.

Sorry about that, I don't know what could be wrong.

 Implementing channels in wicket-atmosphere
 --

 Key: WICKET-4879
 URL: https://issues.apache.org/jira/browse/WICKET-4879
 Project: Wicket
  Issue Type: New Feature
  Components: wicket-atmosphere
Affects Versions: 6.3.0
 Environment: Tomcat / Jetty
Reporter: Zala Goupil
Assignee: Emond Papegaaij
 Fix For: 6.4.0

 Attachments: wicket-atmosphere-quickstart.zip


 Good evening,
 Now that both Tomcat  Jetty support native WebSockets, it would be a 
 killer-feature to implement Comet / WebSockets channels. i.e. being able to 
 have a channel id and to post a message to one client only or to all the 
 clients who subscribed to a certain channel.
 For the moment, it is possible to do it, but to the (expensive) price of 
 having a PerRequestBroadcastFilter and tweaking it if we manage to do it. But 
 it would be great to have a link to the Broadcaster API from Atmosphere, 
 which I think, supports this kind of things.That would probably be way more 
 efficient than PerRequestBroadcastFilter tweaking.
 As of today, when posting a message to the event bus, all pages who have the 
 @Subscribed-decorated method suitable to process this message will do it, 
 which is not always desirable.
 I'm looking forward to hear from you.
 Regards

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


[CONF] Apache Wicket Migration to Wicket 6.0

2012-11-26 Thread confluence







Migration to Wicket 6.0
Page edited by Daniel Huang


 Changes (1)
 




...
  AjaxRequestTarget target = requestCycle.find(AjaxRequestTarget.class); {code} 
RequestCycle.get().find(Class? extends IRequestHandler) can be used to find the currently running or a scheduled IRequestHandler for other types too. 
 h3. IHeaderResponse, including decorators and filters 
...


Full Content

Migrating to Wicket 6.0



Environment
Repeaters
Form Processing

Validation
Feedback Storage Refactoring
Feedback messages now use ${label} instead of ${input}
added onAfterSubmit to Buttons and SubmitLinks

Ajax

Use JQuery as a backing library for Wicket Ajax functionality
AjaxRequestTarget is an interface

IHeaderResponse, including decorators and filters
Order of Header contributions and wicket:head
package.properties renamed to wicket-package.properties
List of renamed classes and methods
Refactorings
o.a.w.IComponentSource and o.a.w.ComponentSourceEntry are removed
o.a.w.Component#onMarkupAttached() has been dropped in favour of o.a.w.Component#onInitialize().
IInitializers are initialized before Application#init()
PackageResourceReference can load a minified version of a package resource
ResourceStreamLocator and ResourceFinder have been cleaned up
Minor changes


Environment


	Wicket 6.0 requires at least Java 6




Repeaters

	`IDataProvider` was converted to using `long` instead of `int` to better line up with JPA and other persistence frameworks. This has caused a rather large cascade of `int` to `long` changes all over the repeater packages (WICKET-1175).
	All classes in wicket-extensions which are related to repeaters with sorting behavior have improved generics
related to the sorting property (WICKET-4535). In most cases you just have to add ', String' to the previous parameterized type. For example: IColumnEntity becomes IColumnEntity, String. But now it is possible to use other type for the sort property than java.lang.String too.



Form Processing

Validation


	`StringValidator` will add the `maxlength` attribute if added to a component attached to an input tag
	MinimumValidator and MaximumValidator are replaced by RangeValidator.minimum and RangeValidator.maximum respectively
	`AbstractValidator` has been removed
	`ValidationError` now makes creating standard error keys (classname and classname.subtype) easier. Before one had to do: new ValidationError().addMessageKey(validator.getClass().getSimpleName()) to construct the default key, now you can simply say new ValidationError(validator) and it will do the above for you. Also, for more specific keys one had to do new ValidationError().addMessageKey(validator.getClass().getSimpleName()+".variant"), now you can do new ValidationError(validator, "variant").
	Most validators provide a `decorate(ValidationError, Validatable)` method for overriding how they report errors. For example, to add an extra resource key for the error message to StringValidator one can do 


class MyStringValidator extends StringValidator {
ValidationError decorate(ValidationError error, IValidatable validatable) {
   error.addKey("mystringerror");
   return error;
}
}


	org.apache.wicket.validation.RawValidationError is introduced. It can be used to bring a raw Serializable object which is registered as FeedbackMessage with org.apache.wicket.Component#error(Serializable).
	IFormValidator can have its own resource bundle as other IValidator implementations. WICKET-3879



Feedback Storage Refactoring


	Feedback messages reported against components are now stored in component's metadata rather then in session. This will allow them to survive across requests, see WICKET-2705.
	Previously the cleanup of messages was controlled by RequestCycle#cleanupFeedbackMessages(), now it is controlled by an IFeedbackMessageFilter set via IApplicationSettings#setFeedbackMessageCleanupFilter(). When session and components are detached they will run this filter on any stored messages, all messages accepted by the filter are removed. The default filter will accept any rendered message, so by default feedback messages are removed once rendered.
	Form components will clear any stored messages when validate() is called. This will remove any unredered, but now stale, messages.
	Since messages are now stored in session and in component tree retrieving them is not as simple as Session.getFeedbackMessages(). To assist with this common usecase please see FeedbackCollector.
	In test land cleanup of feedback messages was controlled by RequestCycle#set/isCleanupFeedbackMessagesOnDetach(), but this method is now deprecated. Instead, in testland you can now call 

[jira] [Commented] (WICKET-4879) Implementing channels in wicket-atmosphere

2012-11-26 Thread Emond Papegaaij (JIRA)

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

Emond Papegaaij commented on WICKET-4879:
-

Do you think you can make a quickstart for that? If so, please create a new 
ticket. However, I suspect the problem lies somewhere else, because the code in 
EventBus does not write anything to the other resources.

 Implementing channels in wicket-atmosphere
 --

 Key: WICKET-4879
 URL: https://issues.apache.org/jira/browse/WICKET-4879
 Project: Wicket
  Issue Type: New Feature
  Components: wicket-atmosphere
Affects Versions: 6.3.0
 Environment: Tomcat / Jetty
Reporter: Zala Goupil
Assignee: Emond Papegaaij
 Fix For: 6.4.0

 Attachments: wicket-atmosphere-quickstart.zip


 Good evening,
 Now that both Tomcat  Jetty support native WebSockets, it would be a 
 killer-feature to implement Comet / WebSockets channels. i.e. being able to 
 have a channel id and to post a message to one client only or to all the 
 clients who subscribed to a certain channel.
 For the moment, it is possible to do it, but to the (expensive) price of 
 having a PerRequestBroadcastFilter and tweaking it if we manage to do it. But 
 it would be great to have a link to the Broadcaster API from Atmosphere, 
 which I think, supports this kind of things.That would probably be way more 
 efficient than PerRequestBroadcastFilter tweaking.
 As of today, when posting a message to the event bus, all pages who have the 
 @Subscribed-decorated method suitable to process this message will do it, 
 which is not always desirable.
 I'm looking forward to hear from you.
 Regards

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


git commit: WICKET-4884 keep locale untouched but do custom conversion instead

2012-11-26 Thread svenmeier
Updated Branches:
  refs/heads/master e9d8e33ff - 9de66fbf5


WICKET-4884 keep locale untouched but do custom conversion instead


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

Branch: refs/heads/master
Commit: 9de66fbf5701de16d28cf6ea78a1c2a62065a987
Parents: e9d8e33
Author: svenmeier svenme...@apache.org
Authored: Tue Nov 27 08:38:27 2012 +0100
Committer: svenmeier svenme...@apache.org
Committed: Tue Nov 27 08:38:27 2012 +0100

--
 .../wicket/markup/html/form/NumberTextField.java   |   69 
 .../markup/html/form/NumberTextFieldTest.java  |  144 +++
 2 files changed, 177 insertions(+), 36 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/wicket/blob/9de66fbf/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
--
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
index ae3706b..0e88c7b 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
@@ -16,14 +16,14 @@
  */
 package org.apache.wicket.markup.html.form;
 
-import java.text.NumberFormat;
 import java.util.Locale;
 
 import org.apache.wicket.markup.ComponentTag;
 import org.apache.wicket.model.IModel;
+import org.apache.wicket.util.convert.ConversionException;
 import org.apache.wicket.util.convert.IConverter;
-import org.apache.wicket.util.convert.converter.AbstractDecimalConverter;
 import org.apache.wicket.util.lang.Numbers;
+import org.apache.wicket.util.lang.Objects;
 import org.apache.wicket.util.value.IValueMap;
 import org.apache.wicket.validation.validator.RangeValidator;
 
@@ -33,8 +33,8 @@ import org.apache.wicket.validation.validator.RangeValidator;
  * p
  * Automatically validates the input against the configured {@link 
#setMinimum(N) min} and
  * {@link #setMaximum(N) max} attributes. If any of them is codenull/code 
then respective
- * MIN_VALUE or MAX_VALUE for the number type is used. If the number type has 
no minimum and/or maximum
- * value then {@link Double#MIN_VALUE} and {@link Double#MAX_VALUE} are used 
respectfully.
+ * MIN_VALUE or MAX_VALUE for the number type is used. If the number type has 
no minimum and/or
+ * maximum value then {@link Double#MIN_VALUE} and {@link Double#MAX_VALUE} 
are used respectfully.
  * 
  * @param N
  *the type of the number
@@ -43,13 +43,6 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
 {
private static final long serialVersionUID = 1L;
 
-   /**
-* A special locale which is used to render decimal number formats that 
are HTML5 compliant.
-*
-* See a 
href=http://dev.w3.org/html5/markup/datatypes.html#common.data.float;HTML5 
number format/a
-*/
-   private static final Locale HTML5_LOCALE = new Locale(en, , 
wicket-html5);
-
private RangeValidatorN validator;
 
private N minimum;
@@ -149,7 +142,7 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
else
{
ClassN numberType = getNumberType();
-   result = (N) Numbers.getMinValue(numberType);
+   result = (N)Numbers.getMinValue(numberType);
}
return result;
}
@@ -164,7 +157,7 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
else
{
ClassN numberType = getNumberType();
-   result = (N) Numbers.getMaxValue(numberType);
+   result = (N)Numbers.getMaxValue(numberType);
}
return result;
}
@@ -174,7 +167,7 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
ClassN numberType = getType();
if (numberType == null  getModelObject() != null)
{
-   numberType = (ClassN) getModelObject().getClass();
+   numberType = (ClassN)getModelObject().getClass();
}
return numberType;
}
@@ -188,8 +181,7 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
 
if (minimum != null)
{
-   IConverterN converter = getConverter(getNumberType());
-   attributes.put(min, 

git commit: WICKET-4884 keep locale untouched but do custom conversion instead

2012-11-26 Thread svenmeier
Updated Branches:
  refs/heads/wicket-1.5.x 4d6b76dee - 121f2e2a1


WICKET-4884 keep locale untouched but do custom conversion instead


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

Branch: refs/heads/wicket-1.5.x
Commit: 121f2e2a1139105a3ae21b3a9a74ae392b454ec3
Parents: 4d6b76d
Author: svenmeier svenme...@apache.org
Authored: Tue Nov 27 08:38:37 2012 +0100
Committer: svenmeier svenme...@apache.org
Committed: Tue Nov 27 08:38:37 2012 +0100

--
 .../wicket/markup/html/form/NumberTextField.java   |   94 +++---
 .../markup/html/form/NumberTextFieldTest.java  |  143 +++
 2 files changed, 206 insertions(+), 31 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/wicket/blob/121f2e2a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
--
diff --git 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
index ae3706b..4f0adbb 100644
--- 
a/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
+++ 
b/wicket-core/src/main/java/org/apache/wicket/markup/html/form/NumberTextField.java
@@ -16,15 +16,18 @@
  */
 package org.apache.wicket.markup.html.form;
 
-import java.text.NumberFormat;
 import java.util.Locale;
+import java.util.Map;
 
 import org.apache.wicket.markup.ComponentTag;
 import org.apache.wicket.model.IModel;
+import org.apache.wicket.util.convert.ConversionException;
 import org.apache.wicket.util.convert.IConverter;
-import org.apache.wicket.util.convert.converter.AbstractDecimalConverter;
+import org.apache.wicket.util.lang.Classes;
 import org.apache.wicket.util.lang.Numbers;
+import org.apache.wicket.util.lang.Objects;
 import org.apache.wicket.util.value.IValueMap;
+import org.apache.wicket.validation.ValidationError;
 import org.apache.wicket.validation.validator.RangeValidator;
 
 /**
@@ -33,8 +36,8 @@ import org.apache.wicket.validation.validator.RangeValidator;
  * p
  * Automatically validates the input against the configured {@link 
#setMinimum(N) min} and
  * {@link #setMaximum(N) max} attributes. If any of them is codenull/code 
then respective
- * MIN_VALUE or MAX_VALUE for the number type is used. If the number type has 
no minimum and/or maximum
- * value then {@link Double#MIN_VALUE} and {@link Double#MAX_VALUE} are used 
respectfully.
+ * MIN_VALUE or MAX_VALUE for the number type is used. If the number type has 
no minimum and/or
+ * maximum value then {@link Double#MIN_VALUE} and {@link Double#MAX_VALUE} 
are used respectfully.
  * 
  * @param N
  *the type of the number
@@ -44,9 +47,10 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
private static final long serialVersionUID = 1L;
 
/**
-* A special locale which is used to render decimal number formats that 
are HTML5 compliant.
-*
-* See a 
href=http://dev.w3.org/html5/markup/datatypes.html#common.data.float;HTML5 
number format/a
+* A special locale which is used to parse decimal number formats that 
are HTML5 compliant.
+* 
+* See a 
href=http://dev.w3.org/html5/markup/datatypes.html#common.data.float;HTML5 
number
+* format/a
 */
private static final Locale HTML5_LOCALE = new Locale(en, , 
wicket-html5);
 
@@ -149,7 +153,7 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
else
{
ClassN numberType = getNumberType();
-   result = (N) Numbers.getMinValue(numberType);
+   result = (N)Numbers.getMinValue(numberType);
}
return result;
}
@@ -164,7 +168,7 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
else
{
ClassN numberType = getNumberType();
-   result = (N) Numbers.getMaxValue(numberType);
+   result = (N)Numbers.getMaxValue(numberType);
}
return result;
}
@@ -174,7 +178,7 @@ public class NumberTextFieldN extends Number  
ComparableN extends TextField
ClassN numberType = getType();
if (numberType == null  getModelObject() != null)
{
-   numberType = (ClassN) getModelObject().getClass();
+   numberType = (ClassN)getModelObject().getClass();
}
return 

[jira] [Closed] (WICKET-4884) ValidationError messages for NumberTextFields with minimum/maximum are always English

2012-11-26 Thread Sven Meier (JIRA)

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

Sven Meier closed WICKET-4884.
--

   Resolution: Fixed
Fix Version/s: 1.5.10
   6.4.0
 Assignee: Sven Meier

The locale is now kept untouched.

 ValidationError messages for NumberTextFields with minimum/maximum are always 
 English
 -

 Key: WICKET-4884
 URL: https://issues.apache.org/jira/browse/WICKET-4884
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 6.1.1, 6.3.0
 Environment: Ubuntu 12.04
Reporter: Dirk Bachmann
Assignee: Sven Meier
Priority: Minor
 Fix For: 6.4.0, 1.5.10

 Attachments: myproject.zip


 Although my Form's Locale is, e.g., de_DE, the error message generated from 
 AbstractRangeValidator#validate(IValidatable) is always English.
 My German custom message for resource key RangeValidator.range in the 
 Page's resource bundle gets ignored too - the default custom message from the 
 resource bundle is used instead.
 Reason might be that Component#getLocale() invoked from 
 Localizer#getStringIgnoreSettings(String, Component, IModel, Locale, String, 
 String) always returns NumberTextField#HTML5_LOCALE (fix for WICKET-3591).
 Pls. see attached quickstart.
 I'm a Wicket newbie, so could very well be that I'm missing something.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira


buildbot failure in ASF Buildbot on wicket-master

2012-11-26 Thread buildbot
The Buildbot has detected a new failure on builder wicket-master while building 
wicket.
Full details are available at:
 http://ci.apache.org/builders/wicket-master/builds/889

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: hemera_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch master] 9de66fbf5701de16d28cf6ea78a1c2a62065a987
Blamelist: svenmeier svenme...@apache.org

BUILD FAILED: failed compile

sincerely,
 -The Buildbot





buildbot failure in ASF Buildbot on wicket-branch-1.5.x

2012-11-26 Thread buildbot
The Buildbot has detected a new failure on builder wicket-branch-1.5.x while 
building wicket.
Full details are available at:
 http://ci.apache.org/builders/wicket-branch-1.5.x/builds/368

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: hemera_ubuntu

Build Reason: scheduler
Build Source Stamp: [branch wicket-1.5.x] 
121f2e2a1139105a3ae21b3a9a74ae392b454ec3
Blamelist: svenmeier svenme...@apache.org

BUILD FAILED: failed compile

sincerely,
 -The Buildbot





buildbot success in ASF Buildbot on wicket-master

2012-11-26 Thread buildbot
The Buildbot has detected a restored build on builder wicket-master while 
building ASF Buildbot.
Full details are available at:
 http://ci.apache.org/builders/wicket-master/builds/890

Buildbot URL: http://ci.apache.org/

Buildslave for this Build: hemera_ubuntu

Build Reason: forced: by IRC user martin-g on channel ##wicket: None
Build Source Stamp: HEAD
Blamelist: 

Build succeeded!

sincerely,
 -The Buildbot