Peopledirectory: report group ordering broken

[Tres Seaver]

When trying to move an item up in a report group contents view.

AttributeError: 'OOBTreeItems' object has no attribute 'index'
Exception when processing https://karl.soros.org/people/foundations/c2/spin-offs/moveup.html?name=open-society-fund-prague
Referer: https://karl.soros.org/people/foundations/c2/spin-offs/admin.html

Traceback (most recent call last):
  File "/srv/osfkarl/production/13/eggs/pyramid-1.2.1-py2.6.egg/pyramid/tweens.py", line 17, in excview_tween
    response = handler(request)
  File "/srv/osfkarl/production/13/eggs/pyramid_tm-0.5-py2.6.egg/pyramid_tm/__init__.py", line 107, in tm_tween
    return response
  File "/srv/osfkarl/production/13/eggs/pyramid_tm-0.5-py2.6.egg/pyramid_tm/__init__.py", line 75, in __exit__
    return self._retry_or_raise(t, v, tb)
  File "/srv/osfkarl/production/13/eggs/pyramid_tm-0.5-py2.6.egg/pyramid_tm/__init__.py", line 60, in _retry_or_raise
    reraise(t, v, tb) # otherwise reraise the exception
  File "/srv/osfkarl/production/13/eggs/pyramid_tm-0.5-py2.6.egg/pyramid_tm/__init__.py", line 100, in tm_tween
    response = handler(request)
  File "/srv/osfkarl/production/13/eggs/pyramid-1.2.1-py2.6.egg/pyramid/router.py", line 153, in handle_request
    response = view_callable(context, request)
  File "/srv/osfkarl/production/13/eggs/pyramid-1.2.1-py2.6.egg/pyramid/config/views.py", line 186, in _secured_view
    return view(context, request)
  File "/srv/osfkarl/production/13/eggs/pyramid-1.2.1-py2.6.egg/pyramid/config/views.py", line 292, in rendered_view
    result = view(context, request)
  File "/srv/osfkarl/production/13/eggs/karl-3.104-py2.6.egg/karl/views/peopledirectory.py", line 104, in admin_contents_moveup_view
    n = order.index(name)
AttributeError: 'OOBTreeItems' object has no attribute 'index'

[Tres Seaver (tseaver)]

I need to fix this issue in order to implement https://bugs.launchpad.net/karl3/+bug/1131934 correctly.

[Tres Seaver (tseaver)]

Fix committed on 'tseaver-1144889-peopledir_ordering' branch.

[Tres Seaver (tseaver)]

I just tested the 'move up' and 'move down' features on karlstaging (/people/admin.html), after merging the branch to the 'm125' integration branch. All is well.

[Paul Everitt (paul-agendaless)]

Yep, I tested also.