you dont need to use models per se as long as you are rebuilding your component hierarchy on every request. RepeatingView does not do that. so either use models like Jeremy said, or rebuild the component hierarchy using a RefreshingView or a ListView
-igor On Thu, May 27, 2010 at 9:10 AM, Jeremy Thomerson <[email protected]> wrote: > On Thu, May 27, 2010 at 10:53 AM, sahuja <[email protected]> wrote: > >> >> Thanks, Igor. Copied below is the implementation for my toolbar. Could you >> please take a look and see if you can spot a problem with it? >> >> public class TotalsToolbar extends AbstractToolbar >> { >> >> public TotalsToolbar(final DataTable table, final IDataProvider >> dataProvider) >> { >> super(table); >> >> RepeatingView totals = new RepeatingView("totals"); >> add(totals); >> >> AbstractReportingBean row = >> (AbstractReportingBean)dataProvider.iterator(0, 1).next(); >> final IColumn[] columns = table.getColumns(); >> for (int i = 0; i < columns.length; i++) >> { >> final IColumn column = columns[i]; >> >> WebMarkupContainer item = new >> WebMarkupContainer(totals.newChildId()); >> totals.add(item); >> >> WebMarkupContainer total = new WebMarkupContainer("total"); >> >> item.add(total); >> item.setRenderBodyOnly(true); >> >> * String val = >> row.getStringForFieldNamed(column.getSortProperty()); >> if (val == null || val.length() == 0) { >> val = " "; >> total.add(new Label("value", >> val).setEscapeModelStrings(false)); >> } else >> total.add(new Label("value", val)); >> * >> } >> } >> } >> >> > You're not using a model for your label - so you are "pushing" data into it > on creation and never changing the data after it's initially created. This > is easily learned / proven if you put a breakpoint there where you are > creating your label. > > Actually, now that I look at it, you're also not using a model for your > columns in your toolbar. I'm not sure what your AbstractReportingBean is, > but you will probably also need to change your RepeatingView to something > that takes an IModel<List> and creates the columns on the fly - or else > you'll get old data. > > Here's an example for the Label - but you'll have to do the repeater on your > own. > > total.add(new Label("value", new AbstractReadOnlyModel<String>() { > String getObject() { > String val = row.getStringForFieldNamed(column.getSortProperty());; > return StringUtils.isEmpty(val) ? "nbsp;" : val; > } > }); > -- > Jeremy Thomerson > http://www.wickettraining.com > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
