bundle_id with space causes most activities to disappear

[dsd]

Edit an activity.info file, adding another word to the bundle_id assignment, like:

bundle_id = org.laptop.Log hello

Restart Sugar. Most activities won't appear on the home screen, since Sugar has choked on the following exception:

Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/jarabe/desktop/favoritesview.py", line 133, in __connect_to_bundle_registry_cb
    info.get_activity_version()):
  File "/usr/lib/python2.7/site-packages/jarabe/model/bundleregistry.py", line 312, in is_bundle_favorite
    key = self._get_favorite_key(bundle_id, version)
  File "/usr/lib/python2.7/site-packages/jarabe/model/bundleregistry.py", line 118, in _get_favorite_key
    raise ValueError('bundle_id cannot contain spaces')
ValueError: bundle_id cannot contain spaces

Reproduced on Sugar-0.98 and 0.96.

[AlanJAS]

This is a bad bug! Sugar cannot fail for a incorrect activity.info field.
What happens with other fields?

[SAMdroid]

I'm working on a fix for the bug at _get_favorite_key() (As part of google code in).
It is sort of working but another bug is just exposed downstream; dbus will not accept spaces in ids either. This causes dbus apps to crash (log...).

[Sam]

Fixes #4634, issue where spaces in bundle ids crashed sugar

Changeset: f43d5f8d24819fa19f2e49bd14cd1b51ee44c2cc