Remove favourite does not work from principal table

[dmartinc]

Removing a favourite works well from a user's personal area.
However it does not when clicking on an "illuminated" star icon:

Service request failed. Request text:
{"version":"1.1","method":"user.favorites.removeUser","params":{"value":[{"id":"2562"}]}}

Traceback (most recent call last):
  File "/usr/lib/python2.4/site-packages/cds_indico-0.97b2-py2.4.egg/MaKaC/services/interface/rpc/json.py", line 106, in process
    result = invokeMethod(str(requestBody["method"]), requestBody.get("params", []), req)
  File "/usr/lib/python2.4/site-packages/cds_indico-0.97b2-py2.4.egg/MaKaC/services/interface/rpc/process.py", line 86, in invokeMethod
    raise ProcessError("ERR-P0", "Error processing method.")
ProcessError: ERR-P0 : Error processing method.

Inner Exception:
Traceback (most recent call last):

  File "/usr/lib/python2.4/site-packages/cds_indico-0.97b2-py2.4.egg/MaKaC/services/interface/rpc/process.py", line 67, in invokeMethod
    result = processRequest(method, params, req)

  File "/usr/lib/python2.4/site-packages/cds_indico-0.97b2-py2.4.egg/MaKaC/services/interface/rpc/process.py", line 45, in processRequest
    result = handler(params, req.get_remote_host(), getSession(req)).process()

  File "/usr/lib/python2.4/site-packages/cds_indico-0.97b2-py2.4.egg/MaKaC/services/implementation/base.py", line 222, in process
    self._checkParams()

  File "/usr/lib/python2.4/site-packages/cds_indico-0.97b2-py2.4.egg/MaKaC/services/implementation/user.py", line 93, in _checkParams
    self._obj = user.AvatarHolder().getById(self._params['id'])

KeyError: 'id'

[jose.pedro.ferreira@…]

commit d0a51f58673028c0a3234b0979755dc7383d8684
Author: Pedro Ferreira <jose.pedro.ferreira@…>
Date: Thu Feb 11 17:25:05 2010 +0100

[FIX] Removing users from favorites

• fixes #182

[dmartinc]

Testing in my PC, looks like the star widget now works to remove a favourite, but now removing it from the user's personal area throws an exception.
It seems both widgets are sending the id in a different way; one directly as "id", the other inside a "value"; so we have to make them do the same thing...

[jose.pedro.ferreira@…]

commit 3422580b3676f103c6dd67c528d8d2c615221305
Author: Pedro Ferreira <jose.pedro.ferreira@…>
Date: Tue Feb 16 15:17:56 2010 +0100

[FIX] Problem removing users in 'favorites' dialog

• fixes #182 once and for all (i hope);

[Pedro Ferreira <jose.pedro.ferreira@…>]

In [d0a51f58673028c0a3234b0979755dc7383d8684]:

[FIX] Removing users from favorites

• fixes #182

[Pedro Ferreira <jose.pedro.ferreira@…>]

In [3422580b3676f103c6dd67c528d8d2c615221305]:

[FIX] Problem removing users in 'favorites' dialog

• fixes #182 once and for all (i hope);