Switch to unified view

a/Allura/allura/model/timeline.py b/Allura/allura/model/timeline.py
...
...
44
def perm_check(user):
44
def perm_check(user):
45
    def _perm_check(activity):
45
    def _perm_check(activity):
46
        """Return True if c.user has 'read' access to this activity,
46
        """Return True if c.user has 'read' access to this activity,
47
        otherwise return False.
47
        otherwise return False.
48
        """
48
        """
49
        allura_id = activity['obj']['activity_extras'].get('allura_id')
49
        extras_dict = activity['obj'].get('activity_extras')
50
        if not extras_dict: return True
51
        allura_id = extras_dict.get('allura_id')
50
        if not allura_id: return True
52
        if not allura_id: return True
51
        classname, _id = allura_id.split(':')
53
        classname, _id = allura_id.split(':')
52
        cls = Mapper.by_classname(classname).mapped_class
54
        cls = Mapper.by_classname(classname).mapped_class
53
        try:
55
        try:
54
            _id = bson.ObjectId(_id)
56
            _id = bson.ObjectId(_id)