Merge pull request #53 from SolidOS/returnClass

Return class with typeindex data

Author: timea-solid

src/inbox/inboxLogic.ts

@@ -3,13 +3,13 @@ import { InboxLogic } from "../types";
 import { getArchiveUrl } from "../util/utils";
 
 export function createInboxLogic(store, profileLogic, utilityLogic, containerLogic, aclLogic): InboxLogic {
-  
+
     async function createInboxFor(peerWebId: string, nick: string) {
       const myWebId: NamedNode = await profileLogic.loadMe();
       const podRoot: NamedNode = await profileLogic.getPodRoot(myWebId);
       const ourInbox = `${podRoot.value}p2p-inboxes/${encodeURIComponent(nick)}/`;
       await containerLogic.createContainer(ourInbox);
-      const aclDocUrl = await aclLogic.findAclDocUrl(ourInbox);
+      // const aclDocUrl = await aclLogic.findAclDocUrl(ourInbox);
       await utilityLogic.setSinglePeerAccess({
         ownerWebId: myWebId.value,
         peerWebId,
@@ -55,4 +55,4 @@ export function createInboxLogic(store, profileLogic, utilityLogic, containerLog
     getNewMessages,
     markAsRead
   }
-}
+}

src/profile/profileLogic.ts

@@ -11,7 +11,7 @@ export function createProfileLogic(store, authn, utilityLogic): ProfileLogic {
     /**
      * loads the preference without throwing errors - if it can create it it does so.
      * remark: it still throws error if it cannot load profile.
-     * @param user 
+     * @param user
      * @returns undefined if preferenceFile cannot be returned or NamedNode if it can find it or create it
      */
     async function silencedLoadPreferences(user: NamedNode): Promise <NamedNode | undefined> {
@@ -25,7 +25,7 @@ export function createProfileLogic(store, authn, utilityLogic): ProfileLogic {
     /**
      * loads the preference without returning different errors if it cannot create or load it.
      * remark: it also throws error if it cannot load profile.
-     * @param user 
+     * @param user
      * @returns undefined if preferenceFile cannot be an Error or NamedNode if it can find it or create it
      */
     async function loadPreferences (user: NamedNode): Promise <NamedNode> {
@@ -48,9 +48,8 @@ export function createProfileLogic(store, authn, utilityLogic): ProfileLogic {
             throw err
         }
 
-        let response
         try {
-            response = await store.fetcher.load(preferencesFile as NamedNode)
+            await store.fetcher.load(preferencesFile as NamedNode)
         } catch (err) { // Maybe a permission problem or origin problem
             const msg = `Unable to load preference of user ${user}: ${err}`
             debug.warn(msg)
@@ -123,4 +122,3 @@ export function createProfileLogic(store, authn, utilityLogic): ProfileLogic {
         silencedLoadPreferences
     }
 }
-

src/typeIndex/typeIndexLogic.ts

@@ -2,7 +2,7 @@ import { NamedNode, st, sym } from 'rdflib'
 import { ScopedApp, TypeIndexLogic, TypeIndexScope } from '../types'
 import * as debug from "../util/debug"
 import { ns as namespace } from '../util/ns'
-import { newThing, uniqueNodes } from "../util/utils"
+import { newThing } from "../util/utils"
 
 export function createTypeIndexLogic(store, authn, profileLogic, utilityLogic): TypeIndexLogic {
     const ns = namespace
@@ -155,29 +155,30 @@ export function createTypeIndexLogic(store, authn, profileLogic, utilityLogic):
 
     async function getScopedAppsFromIndex(scope: TypeIndexScope, theClass: NamedNode | null): Promise<ScopedApp[]> {
         const index = scope.index
+        const results: ScopedApp[] = []
         const registrations = store.statementsMatching(null, ns.solid('instance'), null, index)
             .concat(store.statementsMatching(null, ns.solid('instanceContainer'), null, index))
             .map(st => st.subject)
-        const relevant = theClass ? registrations.filter(reg => store.any(reg, ns.solid('forClass'), null, index)?.sameTerm(theClass))
-            : registrations
-        const directInstances = relevant.map(reg => store.each(reg, ns.solid('instance'), null, index).map(one => sym(one.value))).flat()
-        let instances = uniqueNodes(directInstances)
-
-        const instanceContainers = relevant.map(
-            reg => store.each(reg, ns.solid('instanceContainer'), null, index).map(one => sym(one.value))).flat()
-
-        //  instanceContainers may be deprocatable if no one has used them
-        const containers = uniqueNodes(instanceContainers)
-        if (containers.length > 0) { console.log('@@ getScopedAppsFromIndex containers ', containers) }
-        for (let i = 0; i < containers.length; i++) {
-            const cont = containers[i]
-            await store.fetcher.load(cont)
-            const contents = store.each(cont, ns.ldp('contains'), null, cont).map(one => sym(one.value))
-            instances = instances.concat(contents)
+        for (const reg of registrations) {
+          const klass = store.any(reg, ns.solid('forClass'), null, index)
+          if (!theClass || klass.sameTerm(theClass)) {
+            const instances = store.each(reg, ns.solid('instance'), null, index)
+            for (const instance of instances) {
+              results.push({ instance, type: klass, scope })
+            }
+            const containers = store.each(reg, ns.solid('instanceContainer'), null, index)
+            for (const cont of containers) {
+              await store.fetcher.load(cont)
+              const contents = store.each(cont, ns.ldp('contains'), null, cont).map(one => sym(one.value))
+              for (const instance of contents) {
+                results.push({ instance: sym(instance.value), type: klass,  scope })
+              }
+            }
+          }
         }
-        return instances.map(instance => { return { instance, scope } })
+        return results
     }
-    
+
     return {
         registerInTypeIndex,
         getRegistrations,

src/types.ts

@@ -33,7 +33,7 @@ export interface SolidNamespace {
 }
 
 export type TypeIndexScope = { label: string, index: NamedNode, agent: NamedNode }
-export type ScopedApp = { instance: NamedNode, scope: TypeIndexScope }
+export type ScopedApp = { instance: NamedNode, type: NamedNode, scope: TypeIndexScope }
 
 export interface NewPaneOptions {
     me?: NamedNode;
@@ -119,4 +119,3 @@ export interface SolidLogic {
     updatePromise: (del: Array<Statement>, ins: Array<Statement>) => Promise<void>,
     clearStore: () => void
 }
-