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.