You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Juergen Donnerstag (JIRA)" <ji...@apache.org> on 2009/01/17 16:07:59 UTC
[jira] Resolved: (WICKET-2012) Not possible to call getRowCount()
in override of isVisible() for DataView
[ https://issues.apache.org/jira/browse/WICKET-2012?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Juergen Donnerstag resolved WICKET-2012.
----------------------------------------
Resolution: Fixed
Fix Version/s: 1.4-RC2
Assignee: Juergen Donnerstag
introduced new method getItemCount which is extracted from getRowCount but still uses the cached count if possible.
> Not possible to call getRowCount() in override of isVisible() for DataView
> --------------------------------------------------------------------------
>
> Key: WICKET-2012
> URL: https://issues.apache.org/jira/browse/WICKET-2012
> Project: Wicket
> Issue Type: Bug
> Components: wicket
> Affects Versions: 1.3.5
> Reporter: Peter Thomas
> Assignee: Juergen Donnerstag
> Fix For: 1.4-RC2
>
>
> I am using a DataView and I was trying to override isVisible() and use "this.getRowCount()" to decide if the table should be rendered or not. But this results in a stack overflow because DataView#getRowCount() internally calls isVisibleInHierarchy().
> The use case is trying to hide a table header row when there are no search results. The implementation of getRowCount() in AbstractPageableView (Wicket 1.3.5) starts like this:
> if (!isVisibleInHierarchy())
> {
> return 0;
> }
> Was wondering - is that part really necessary? I wanted to use an enclosure but that also results in isVisible() calls all over the place. I could work around this for now by using a separate web markup container for hiding the table header row.
> My message to the mailing list: http://www.nabble.com/trouble-controlling-visibility-of-dataview-td21265778.html#a21265778
> I just searched Nabble and found this had come up before:
> http://www.nabble.com/enclosures-and-dataview-td14258879.html#a14258879
> Let me know if you need a quickstart or more information.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.