[jira] Resolved: (WICKET-2405) componentID.PatternValidator settings in properties file cannot be picked up

2009-07-30 Thread Igor Vaynberg (JIRA)

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

Igor Vaynberg resolved WICKET-2405.
---

Resolution: Won't Fix
  Assignee: Igor Vaynberg

1.4 rc7 is not the latest version. retest with 1.4.0 and reopen if still broken.

> componentID.PatternValidator settings in properties file cannot be picked up
> 
>
> Key: WICKET-2405
> URL: https://issues.apache.org/jira/browse/WICKET-2405
> Project: Wicket
>  Issue Type: Bug
>  Components: wicket
>Affects Versions: 1.4-RC7
> Environment: jaunty, 1.4-rc7. JDK 1.6.0_13-b03
>Reporter: jason wang
>Assignee: Igor Vaynberg
>
> Code:
> final RequiredTextField mobile = new 
> RequiredTextField("mobile");
> mobile.setLabel(new Model("mobile")).add(new 
> PatternValidator("^[1-9]([0-9]{8,14})"));
> Properties file:
> mobile.PatternValidator = Please input a valid international phone number.
> Results:
> always output  does not match pattern '^[1-9]([0-9]{8,14})'.
> WIth the same code, build with 1.4 rc1, worked fine, build with rc7, problem 
> occurred.

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



[jira] Created: (WICKET-2405) componentID.PatternValidator settings in properties file cannot be picked up

2009-07-30 Thread jason wang (JIRA)
componentID.PatternValidator settings in properties file cannot be picked up


 Key: WICKET-2405
 URL: https://issues.apache.org/jira/browse/WICKET-2405
 Project: Wicket
  Issue Type: Bug
  Components: wicket
Affects Versions: 1.4-RC7
 Environment: jaunty, 1.4-rc7. JDK 1.6.0_13-b03
Reporter: jason wang


Code:

final RequiredTextField mobile = new 
RequiredTextField("mobile");
mobile.setLabel(new Model("mobile")).add(new 
PatternValidator("^[1-9]([0-9]{8,14})"));

Properties file:

mobile.PatternValidator = Please input a valid international phone number.

Results:

always output  does not match pattern '^[1-9]([0-9]{8,14})'.


WIth the same code, build with 1.4 rc1, worked fine, build with rc7, problem 
occurred.

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



[jira] Commented: (WICKET-2376) In some cases, org.apache.wicket.extensions.markup.html.repeater.data.table.NavigatorToolbar is not displayed even when there are multiple pages in the DataTable it is

2009-07-30 Thread Kelsey Francis (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12737305#action_12737305
 ] 

Kelsey Francis commented on WICKET-2376:


This is related to WICKET-2175. I'm experiencing this bug too.

The problem appears to be that callOnBeforeRenderIfNotVisible() in 
NavigationToolbar never gets called, because NavigationToolbar is contained 
within a DataTable.ToolbarContainer, which is itself contained in a 
DataTable.ToolbarsContainer.

ToolbarsContainer's isVisible() returns false, since it doesn't currently 
contain any visible toolbars, and since callOnBeforeRenderIfNotVisible() is not 
overridden in either ToolbarsContainer and ToolbarContainer, onBeforeRender() 
in those components is never called, and thus onBeforeRender() is never called 
in NavigationToolbar, so it remains hidden.

So, once NavigationToolbar is hidden, it never can become visible again. 

> In some cases, 
> org.apache.wicket.extensions.markup.html.repeater.data.table.NavigatorToolbar 
> is not displayed even when there are multiple pages in the DataTable it is 
> attached to
> ---
>
> Key: WICKET-2376
> URL: https://issues.apache.org/jira/browse/WICKET-2376
> Project: Wicket
>  Issue Type: Bug
>  Components: wicket
>Affects Versions: 1.4-RC7
> Environment: Windows Vista, JDK 6 Update 14
>Reporter: Vikash Madhow
>
> The following can recreate the problem:
> 1. Create a DataTable with a NavigationToolbar and a page length less than 
> what the data provider will return, i.e., the data table will have more than 
> one page initially.
> 2. Add two buttons to the page; the first one, when submitted, will change 
> the criteria of the data provider to return less rows than what can be 
> displayed in one page of the DataTable and reload the page, i.e., after 
> pressing the button, the page will be reloaded with the datatable having only 
> one page and, therefore, the NavigationToolbar will not be visible anymore.
> 3. The second button will change the criteria of the data provider back to 
> what it was originally, i.e., it will have more rows than can be displayed in 
> one page of the DataTable, and reload the page. At that point, the 
> NavigationToolbar should be shown again, but it isn't.
> I've found that the bug is due to the fact that the onBeforeRender() method 
> of NavigationToolbar which have the following line for determining visibility 
> of the toolbar:
>   setVisible(table.getPageCount() > 1);
> is not called when the 2nd button is pressed in the above scenario.
> I've been able to work-around this problem by subclassing NavigationToolbar 
> and overidding isVisible() method (which is called many times however, and 
> might cause some performance problems) and added the above line of code as 
> such:
> @Override
> public boolean isVisible()
> {
> return getTable().getPageCount() > 1;
> }

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



[jira] Updated: (WICKET-2376) In some cases, org.apache.wicket.extensions.markup.html.repeater.data.table.NavigatorToolbar is not displayed even when there are multiple pages in the DataTable it is at

2009-07-30 Thread Kelsey Francis (JIRA)

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

Kelsey Francis updated WICKET-2376:
---

Comment: was deleted

(was: This is related to WICKET-2175. I'm experiencing this bug too.

The problem appears to be that callOnBeforeRenderIfNotVisible() in 
NavigationToolbar never gets called, because NavigationToolbar is contained 
within a DataTable.ToolbarContainer, which is itself contained in a 
DataTable.ToolbarsContainer.

ToolbarsContainer's isVisible() returns false, since it doesn't currently 
contain any visible toolbars, and since callOnBeforeRenderIfNotVisible() is not 
overridden in either ToolbarsContainer and ToolbarContainer, onBeforeRender() 
in those components is never called, and thus onBeforeRender() is never called 
in NavigationToolbar, so it remains hidden.

So, once NavigationToolbar is hidden, it never can become visible again.)

> In some cases, 
> org.apache.wicket.extensions.markup.html.repeater.data.table.NavigatorToolbar 
> is not displayed even when there are multiple pages in the DataTable it is 
> attached to
> ---
>
> Key: WICKET-2376
> URL: https://issues.apache.org/jira/browse/WICKET-2376
> Project: Wicket
>  Issue Type: Bug
>  Components: wicket
>Affects Versions: 1.4-RC7
> Environment: Windows Vista, JDK 6 Update 14
>Reporter: Vikash Madhow
>
> The following can recreate the problem:
> 1. Create a DataTable with a NavigationToolbar and a page length less than 
> what the data provider will return, i.e., the data table will have more than 
> one page initially.
> 2. Add two buttons to the page; the first one, when submitted, will change 
> the criteria of the data provider to return less rows than what can be 
> displayed in one page of the DataTable and reload the page, i.e., after 
> pressing the button, the page will be reloaded with the datatable having only 
> one page and, therefore, the NavigationToolbar will not be visible anymore.
> 3. The second button will change the criteria of the data provider back to 
> what it was originally, i.e., it will have more rows than can be displayed in 
> one page of the DataTable, and reload the page. At that point, the 
> NavigationToolbar should be shown again, but it isn't.
> I've found that the bug is due to the fact that the onBeforeRender() method 
> of NavigationToolbar which have the following line for determining visibility 
> of the toolbar:
>   setVisible(table.getPageCount() > 1);
> is not called when the 2nd button is pressed in the above scenario.
> I've been able to work-around this problem by subclassing NavigationToolbar 
> and overidding isVisible() method (which is called many times however, and 
> might cause some performance problems) and added the above line of code as 
> such:
> @Override
> public boolean isVisible()
> {
> return getTable().getPageCount() > 1;
> }

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



[jira] Commented: (WICKET-2376) In some cases, org.apache.wicket.extensions.markup.html.repeater.data.table.NavigatorToolbar is not displayed even when there are multiple pages in the DataTable it is

2009-07-30 Thread Kelsey Francis (JIRA)

[ 
https://issues.apache.org/jira/browse/WICKET-2376?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12737302#action_12737302
 ] 

Kelsey Francis commented on WICKET-2376:


This is related to WICKET-2175. I'm experiencing this bug too.

The problem appears to be that callOnBeforeRenderIfNotVisible() in 
NavigationToolbar never gets called, because NavigationToolbar is contained 
within a DataTable.ToolbarContainer, which is itself contained in a 
DataTable.ToolbarsContainer.

ToolbarsContainer's isVisible() returns false, since it doesn't currently 
contain any visible toolbars, and since callOnBeforeRenderIfNotVisible() is not 
overridden in either ToolbarsContainer and ToolbarContainer, onBeforeRender() 
in those components is never called, and thus onBeforeRender() is never called 
in NavigationToolbar, so it remains hidden.

So, once NavigationToolbar is hidden, it never can become visible again.

> In some cases, 
> org.apache.wicket.extensions.markup.html.repeater.data.table.NavigatorToolbar 
> is not displayed even when there are multiple pages in the DataTable it is 
> attached to
> ---
>
> Key: WICKET-2376
> URL: https://issues.apache.org/jira/browse/WICKET-2376
> Project: Wicket
>  Issue Type: Bug
>  Components: wicket
>Affects Versions: 1.4-RC7
> Environment: Windows Vista, JDK 6 Update 14
>Reporter: Vikash Madhow
>
> The following can recreate the problem:
> 1. Create a DataTable with a NavigationToolbar and a page length less than 
> what the data provider will return, i.e., the data table will have more than 
> one page initially.
> 2. Add two buttons to the page; the first one, when submitted, will change 
> the criteria of the data provider to return less rows than what can be 
> displayed in one page of the DataTable and reload the page, i.e., after 
> pressing the button, the page will be reloaded with the datatable having only 
> one page and, therefore, the NavigationToolbar will not be visible anymore.
> 3. The second button will change the criteria of the data provider back to 
> what it was originally, i.e., it will have more rows than can be displayed in 
> one page of the DataTable, and reload the page. At that point, the 
> NavigationToolbar should be shown again, but it isn't.
> I've found that the bug is due to the fact that the onBeforeRender() method 
> of NavigationToolbar which have the following line for determining visibility 
> of the toolbar:
>   setVisible(table.getPageCount() > 1);
> is not called when the 2nd button is pressed in the above scenario.
> I've been able to work-around this problem by subclassing NavigationToolbar 
> and overidding isVisible() method (which is called many times however, and 
> might cause some performance problems) and added the above line of code as 
> such:
> @Override
> public boolean isVisible()
> {
> return getTable().getPageCount() > 1;
> }

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



[jira] Created: (WICKET-2404) Quickstart for 1.4 uses 1.3 dtd in HomePage.html

2009-07-30 Thread Erik van Oosten (JIRA)
Quickstart for 1.4 uses 1.3 dtd in HomePage.html


 Key: WICKET-2404
 URL: https://issues.apache.org/jira/browse/WICKET-2404
 Project: Wicket
  Issue Type: Bug
  Components: wicket-quickstart
Affects Versions: 1.4.0
Reporter: Erik van Oosten
Priority: Trivial


The generated HomePage.html contains the following header:

http://wicket.apache.org/dtds.data/wicket-xhtml1.3-strict.dtd"; >

That should be:

http://wicket.apache.org/dtds.data/wicket-xhtml1.4-strict.dtd"; >


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



[jira] Created: (WICKET-2403) org.apache.wicket.markup.html.tree.WicketTreeModel should have a constructor taking a TreeModel

2009-07-30 Thread Florent Legendre (JIRA)
org.apache.wicket.markup.html.tree.WicketTreeModel should have a constructor 
taking a TreeModel
---

 Key: WICKET-2403
 URL: https://issues.apache.org/jira/browse/WICKET-2403
 Project: Wicket
  Issue Type: Improvement
  Components: wicket
Affects Versions: 1.4-RC6
Reporter: Florent Legendre


org.apache.wicket.markup.html.tree.WicketTreeModel has no constructor yet. It 
forces the user to do the job a constructor should typically take care of 
consisting of:


private static IModel getModelForTreeModel(final TreeModel treeModel){
final WicketTreeModel wicketTreeModel = new WicketTreeModel();
wicketTreeModel.setObject(treeModel);
return wicketTreeModel;
}


Nice if such a constructor could make it in 1.4 final.

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