You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Sven Meier (JIRA)" <ji...@apache.org> on 2018/09/27 10:42:00 UTC
[jira] [Resolved] (WICKET-6539) Scope fix for DataTable toolbars
[ https://issues.apache.org/jira/browse/WICKET-6539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Sven Meier resolved WICKET-6539.
--------------------------------
Resolution: Fixed
Assignee: Sven Meier
Fix Version/s: 9.0.0
Accessors return a WebMarkupContainer now, so people can still add Behaviors or do whatever.
> Scope fix for DataTable toolbars
> --------------------------------
>
> Key: WICKET-6539
> URL: https://issues.apache.org/jira/browse/WICKET-6539
> Project: Wicket
> Issue Type: Wish
> Components: wicket-extensions
> Affects Versions: 8.0.0-M8
> Reporter: Vít Rozkovec
> Assignee: Sven Meier
> Priority: Minor
> Fix For: 9.0.0
>
>
> {{I've come across one inconsistency in the DataTable class:}}
> {{ /**}}
> {{ * @return the container with the toolbars at the top}}
> {{ */}}
> {{ public final ToolbarsContainer getTopToolbars()}}
> {{ {}}
> {{ return topToolbars;}}
> {{ /**}}
> {{ * @return the container with the toolbars at the bottom}}
> {{ */}}
> {{ public final ToolbarsContainer getBottomToolbars()}}
> {{ {}}
> {{ return bottomToolbars;}}
> ToolbarsContainer is defined as private, so those methods cannot be used anyway. I suggest either changing the visibility of ToolbarsContainer or removing those methods.
> {{ToolbarsContainer is defined as private, so those methods cannot be used anyway. I suggest either changing the visibility of ToolbarsContainer or removing those methods. I've searched for references and there are not any, though I came across one usecase where I needed to refresh toolbars after ajax update.}}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)