fix overrides.zcml: set skin; fix resource indexing: avoid error when principal not found

loops/organize/util.py

@@ -84,8 +84,9 @@ def getInternalPrincipal(id, context=None, pau=None):
     raise PrincipalLookupError(id)
 
 
-def getPrincipalForUserId(id, context=None):
-    auth = component.getUtility(IAuthentication, context=context)
+def getPrincipalForUserId(id, context=None, auth=None):
+    if auth is None:
+        auth = component.getUtility(IAuthentication, context=context)
     try:
         return auth.getPrincipal(id)
     except PrincipalLookupError:

loops/overrides.zcml

@@ -4,6 +4,8 @@
 
   <!-- ZCML declarations to override the default definitions -->
 
+  <browser:defaultSkin name="Loops" />
+
   <browser:containerViews
       for="zope.app.folder.interfaces.IFolder"
       contents="zope.ManageContent"

loops/resource.py

@@ -588,12 +588,15 @@ class IndexAttributes(object):
                         self.creators()).strip()
 
     def creators(self):
+        from loops.organize.util import getPrincipalForUserId
         cr = IZopeDublinCore(self.context).creators or []
-        pau = component.getUtility(IAuthentication)
+        pau = component.getUtility(IAuthentication, context=self.context)
         creators = []
         for c in cr:
-            principal = pau.getPrincipal(c)
-            if principal is not None:
+            principal = getPrincipalForUserId(c, auth=pau)
+            if principal is None:
+                creators.append(c)
+            else:
                 creators.append(principal.title)
         return creators