Group widget can break profile

[marcus]

[11:24:50] Marcus Povey: bug on groups widget
[11:25:23] … 1) the widget defaults to display 1 but this has no effect on what is displayed
[11:25:47] … 2) if you save it with say 3 - so you get the pageing
[11:25:58] … clicking on the next button totally breaks the page
[11:27:14] … 3) going back to the previous page the next prev buttons disappear

[marcus]

Still an issue ben?

[marcus]

Confirmed that this is still an issue.

[marcus]

This appears to have been fixed in a recent commit.

[costelloc]

I just checked with the svn revision 2532 and the bug is still there.

To reproduce:

1. Become a member of a large number of groups
   
2. Set the group widget in the profile to display a small number of groups (say 2)
   
3. save
   
4. the widget should now display pagination
   
5. click next
   
6. you should see a broken page with only the next page of groups showing
   

I think a better approach would be to display a 'more' link if there are more groups than are displayed in the widget. This would be consistent with the other core plugins. I also noted this only happens directly after saving in the edit mode of the widget. A reload of the page changes the view.

[marcus]

Pagination is no longer showing for me following your instructions, can you confirm this on the latest SVN?

[costelloc]

Change my first instruction to "Become a member of a large number of groups (>10)"

I didn't realize it when I submitted the previous comment, but the pagination fix [svn:2534] is just masking the problem by setting the pagination limit to 10.

Confirmed on svn revision 2537

[marcus]

(In [svn:2539]) Closes #410: Pagination parameter now passed in relationship list view.