Journal Listview BuddyPalette: traceback

[erikos]

Traceback (most recent call last):
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 1216, in __motion_notify_event_cb
    self.notify_mouse_enter()
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 910, in notify_mouse_enter
    self._ensure_palette_exists()
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 905, in _ensure_palette_exists
    palette = self.parent.create_palette()
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/jarabe/journal/listview.py", line 668, in create_palette
    if row[self._model_column_index] is not None:
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/gi/overrides/Gtk.py", line 1136, in __getitem__
    return self.model.get_value(self.iter, key)
  File "/home/erikos/sugar-build/install/lib/python2.7/site-packages/gi/types.py", line 47, in function
    return info.invoke(*args, **kwargs)
TypeError: unknown type (null)

pygtk-doc: ​http://www.pygtk.org/docs/pygtk/class-pygtktreemodelrow.html

[manuq]

The bug is in CellRendererBuddy create_palette method, fails for column indexes 9 and 11 which are COLUMN_BUDDY_* in listmodel.py . It is getting null.

[manuq]

Another traceback, this time moving the mouse around a Journal entry from a shared activity.

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 1279, in __motion_notify_event_cb
    self.notify_mouse_enter()
  File "/usr/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 929, in notify_mouse_enter
    self._ensure_palette_exists()
  File "/usr/lib/python2.7/site-packages/sugar3/graphics/palettewindow.py", line 924, in _ensure_palette_exists
    palette = self.parent.create_palette()
  File "/usr/lib/python2.7/site-packages/jarabe/journal/listview.py", line 679, in create_palette
    nick, xo_color = row[self._model_column_index]
ValueError: too many values to unpack

[manuq]

Our custom TreeModel do_get_value method is returning the right output for the buddies column. It returns:

1. None if there is no buddy
2. (buddy-name, buddy-color) if the buddy for the column exist

However in the create_palette method of CellRendererBuddy we get the first traceback for case 1. And only the buddy-name for the case 2, that's the why of the "too many values to unpack".

Going into pygobjact code we can see that the row[self._model_column_index] internally ​calls an override, which calls the get_value of the custom model, so we can change:

-        if row[self._model_column_index] is not None:
-            nick, xo_color = row[self._model_column_index]
+        if row.model.get_value(row.iter, self._model_column_index) is not None:
+            nick, xo_color = row.model.get_value(row.iter, self._model_column_index)

That gives us the same bug. But turns out that replacing get_value by do_get_value fixes it:

-        if row[self._model_column_index] is not None:
-            nick, xo_color = row[self._model_column_index]
+        if row.model.do_get_value(row.iter, self._model_column_index) is not None:
+            nick, xo_color = row.model.do_get_value(row.iter, self._model_column_index)

With the latter change the tracebacks are gone and I can see the palette for a Journal entry that was shared. It has the correct buddy color and name.

[manuq]

Workaround patch.

[manuq]

Change the model to fix the icon in the Jounal list view

[manuq]

The proposed patch makes the buddy icon in the list visible too. Unlike the workaround, that only fixed the icon inside the palette.

[manuq]

Still there is a drawback, all rows display the same icon when the first one is displayed.

[manuq]

Filed upstream bug: ​https://bugzilla.gnome.org/show_bug.cgi?id=689277

Pushed the workaround 9e494ef0 .

[manuq]

Reopening to track the upstream bug.

[greenfeld]

Buddy icons work in the journal view (as well as the Network view) for shared activities in OLPC 13.1.0 os20/Sugar 0.98.2.

[dnarvaez]

Seems good to close since we have a workaround.

[dnarvaez]

Milestone 1.0 deleted