Eclipse Community Forums: NatTable » GroupBy example has row count problems

Home » Eclipse Projects » NatTable » GroupBy example has row count problems

GroupBy example has row count problems [message #1064929]

Fri, 21 June 2013 13:11

Eclipse User

I've been integrating the GroupBy functionality into our application and only just now noticed a row counting problem.

If you try the "Column and row grouping/Group by" example, you will note that it has 200 rows when there is no grouping. Now, group on any row. It still has 200 rows, with no added rows for the new tree node rows.

I'll be looking into this further and filing a bug report when I can characterize the problem a bit better. But, meanwhile, is this generally known? Has there been any discussion about it. I couldn't find any.

Thanks.

Re: GroupBy example has row count problems [message #1064936 is a reply to message #1064929]

Fri, 21 June 2013 13:35

Eclipse User

Here's a bit more information...

When I group on "Rating", there are 200 rows. There should be 210 rows.

Now, if I collapse the first node and then expand it, there are 210 rows.

If I then ungroup on "Rating" and scroll to the bottom, it is still attempting to show 210 rows, but the last 10 are having exceptions.

I will hazard a guess that the act of grouping/ungrouping is not firing the proper RowStructureEvent for the table. Trying that now...

Re: GroupBy example has row count problems [message #1064946 is a reply to message #1064936]

Fri, 21 June 2013 15:23

Eclipse User

Phew! It looks like this is resolved by modifying GroupByDataLayer.update() to read:

	@Override
	public void update(Observable o, Object arg) {
		updateTree();
		fireLayerEvent(new RowStructuralRefreshEvent(this)); // <== new line
	}

I'll submit a bug for this.

Re: GroupBy example has row count problems [message #1064948 is a reply to message #1064946]

Fri, 21 June 2013 15:46

Eclipse User

I'm not sitting in front of the code right now. But that event should be fired by the GlazedListsEventLayer and should not be needed to be fired manually in the GroupByDataLayer.

Re: GroupBy example has row count problems [message #1064952 is a reply to message #1064948]

Fri, 21 June 2013 17:11

Eclipse User

I don't doubt that you are right. However, GlazedListsEventLayer isn't involved at all in the "Group by" example.

In my own code, derived from early NatTableBuilder code, the GlazedListsEventLayer wraps the GroupByDataLayer. That suggests to me that GroupByDataLayer needs to do something more dramatic in its update() method to inform GlazedListsEventLayer that it needs to send out an event. Perhaps the odd code in updateTree() is meant to do that, but it doesn't seem sufficient, perhaps because it doesn't change the number of elements in the underlying event list.

Re: GroupBy example has row count problems [message #1064966 is a reply to message #1064952]

Sat, 22 June 2013 02:31

Eclipse User

I will investigate in this after I'm done with another issue.

Re: GroupBy example has row count problems [message #1065108 is a reply to message #1064966]

Mon, 24 June 2013 07:44

Eclipse User

Hi Patrick,

so I investigated in this issue, and you are on the right spot as it seems. Even if the GlazedListsEventLayer is added to the layer stack, there is no correct update of the list elements.

The issue is that
- on performing the update of the tree dependent on the grouping, only UPDATE events are fired by the TreeList
- the GlazedListsEventLayer only handles DELETE and INSERT events as structural changes, for UPDATE there is only a VisualRefreshEvent fired
- the GroupByColumnCommandHandler and the UngroupByColumnCommandHandler are registered against the GroupByHeaderLayer, which means the underlying layers are not informed about changes. Also the commands are also just firing VisualRefreshEvents. They should be RowStructuralChangeEvents.

The solution to simply fire the RowStructuralChangeEvent in the update() method as you suggested would be one valid way to go. Another way would be to move the special command handlers to the GroupByDataLayer and let them fire RowStructuralChangeEvents.

After my investigation I would go with your solution, as the events are usually designed to be fired by the layers rather than the commands. But before I'm modifying things I would like to hear Edwins opinion on this. So could you please open a new ticket and link to this topic? Then I will add my investigation results and my opinion into the ticket, so Edwin is possible to get involved.

Thanks for your testing and investigation efforts on this!

Greez,
Dirk

Re: GroupBy example has row count problems [message #1065140 is a reply to message #1065108]

Mon, 24 June 2013 09:38

Eclipse User

Thanks for your quick attention, Dirk. I've added a link back to this discussion to the defect:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=411413

Re: GroupBy example has row count problems [message #1065142 is a reply to message #1065140]

Mon, 24 June 2013 09:39

Eclipse User

Thanks, I completely forgot that you already opened a ticket. Too much in my head ATM. Smile

Previous Topic:	Disable RowHeader
Next Topic:	NatTable bold border between rows?

Goto Forum:

-=] Back to Top [=-

Current Time: Fri Jul 04 23:48:36 EDT 2025

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter