contacts-pane 2.6.5-alpha → 2.6.5-d5c0117d

package/__tests__/unit/data-reformat-test.mjs

@@ -0,0 +1,192 @@
+import { getDataModelIssues } from '../../src/contacsPane'
+
+comsole.log('data reformat test')
+
+
+import pane from "../index";
+import { parse } from "rdflib";
+import { store } from "solid-logic";
+import { context, doc, subject } from "./setup";
+
+// This was at testingsolidos.solidcommunity.net
+const exampleProfile = `@prefix : <#>.
+@prefix foaf: <http://xmlns.com/foaf/0.1/>.
+@prefix ldp: <http://www.w3.org/ns/ldp#>.
+@prefix schema: <http://schema.org/>.
+@prefix solid: <http://www.w3.org/ns/solid/terms#>.
+@prefix space: <http://www.w3.org/ns/pim/space#>.
+@prefix vcard: <http://www.w3.org/2006/vcard/ns#>.
+@prefix xsd: <http://www.w3.org/2001/XMLSchema#>.
+@prefix pro: <./>.
+@prefix inbox: </inbox/>.
+@prefix tes: </>.
+@prefix l: <https://www.w3.org/ns/iana/language-code/>.
+@prefix org: <http://www.w3.org/ns/org#>.
+@prefix ent: <http://www.wikidata.org/entity/>.
+@prefix not: <https://notthereal.apple.com/>.
+@prefix www: <https://www.thebeatles.com/>.
+@prefix ww: <https://www.nasa.gov/>.
+@prefix occup: <http://data.europa.eu/esco/occupation/>.
+@prefix c: <https://solidos.solidcommunity.net/profile/card#>.
+@prefix n0: <http://www.w3.org/ns/auth/acl#>.
+@prefix skill: <http://data.europa.eu/esco/skill/>.
+
+occup:50af07f0-7a75-424e-a66d-5a9deea10f4c
+    schema:name
+    "testeur d\u2019accessibilit\u00e9/testeuse d\u2019accessibilit\u00e9".
+occup:6b0ad7c0-a37f-45c6-a486-ee943a11429e schema:name "astrologue".
+
+occup:807a1ac3-4f56-41f3-a68c-d653d558eb0a schema:name "copilote".
+
+occup:88990bdf-4f6b-4411-82c9-9eecad1db8fb
+schema:name "professeur de musique/professeure de musique".
+skill:aec4c9bf-9c44-4f9d-99f1-70011cebe1a8
+schema:name "tester du mat\u00e9riel d\u2019instrumentation".
+skill:b805f989-14d5-46ad-80b0-755634b66dba
+schema:name "travailler dans de mauvaises conditions m\u00e9t\u00e9orologiques".
+ent:Q23548 schema:name "NASA"@yo.
+
+ent:Q312 schema:name "Apple"@fr.
+
+pro:card a foaf:PersonalProfileDocument; foaf:maker :me; foaf:primaryTopic :me.
+
+:id1621179872094 solid:publicId l:fr.
+
+:id1621182189397 solid:publicId l:de.
+
+:id1621184812427
+    a solid:FutureRole;
+    schema:description "Second future role";
+    schema:startDate "2023-12-25"^^xsd:date;
+    vcard:role "Mission a Mars";
+    org:member :me;
+    org:organization :id1621184844320;
+    org:role occup:6b0ad7c0-a37f-45c6-a486-ee943a11429e.
+
+:id1621182208190
+    a solid:CurrentRole;
+    schema:startDate "2021-04-01"^^xsd:date;
+    schema:endDate "2022-04-01"^^xsd:date;
+
+    vcard:role "Testeuse des Apps Solid";
+    org:member :me;
+    org:organization :id1621182234226;
+    org:role occup:50af07f0-7a75-424e-a66d-5a9deea10f4c.
+:id1621182234226
+    a schema:Corporation;
+    schema:name "Apple";
+    schema:uri not:;
+    solid:publicId ent:Q312.
+
+:id1621182452881
+    a solid:PastRole;
+    schema:description "This was an imaginary but fun gig.";
+    schema:endDate "1963-04-01"^^xsd:date;
+    schema:startDate "1960-04-01"^^xsd:date;
+    vcard:role "Directed the white album";
+    org:member :me;
+    org:organization :id1621182460879;
+    org:role occup:88990bdf-4f6b-4411-82c9-9eecad1db8fb.
+:id1621182460879 a schema:MusicGroup; schema:name "The Beatles"; schema:uri www:.
+
+:id1621183757035
+    a solid:FutureRole;
+    schema:description "Imaginary future roles are sometimes the best";
+    schema:endDate "1993-04-01"^^xsd:date;
+    schema:startDate "1990-05-01"^^xsd:date;
+    vcard:role "Dream: Fly a couple of missions";
+    org:member :me;
+    org:organization :id1621183860447;
+    org:role occup:807a1ac3-4f56-41f3-a68c-d653d558eb0a.
+
+
+
+
+:id1621183860447
+    a schema:GovernmentOrganization;
+    schema:name "National Aeronautical and Space Administration";
+    schema:uri ww:;
+    solid:publicId ent:Q23548.
+
+
+:id1621184844320
+    a schema:GovernmentOrganization;
+    schema:name "NASA";
+    schema:uri ww:;
+    solid:publicId ent:Q23548.
+:id1622021411833
+    vcard:country-name "USA";
+    vcard:locality "Testingville";
+    vcard:region "Texas";
+    vcard:street-address "The testing tree house".
+:id1622021761923 solid:publicId skill:aec4c9bf-9c44-4f9d-99f1-70011cebe1a8 .
+
+:id1622021775187 solid:publicId skill:b805f989-14d5-46ad-80b0-755634b66dba.
+
+:skill2 vcard:role  "sitting and chatting" .
+
+
+:me
+    a schema:Person, foaf:Person;
+    schema:knowsLanguage ( :id1621179872094 :id1621182189397 );
+    schema:skills :id1622021761923, :id1622021775187, :skill2, :NonExistentSkill;
+    vcard:bday "2021-05-14"^^xsd:date;
+    vcard:fn "Testing SolidOS Test";
+    vcard:hasAddress :id1622021411833;
+    vcard:hasPhoto <noun_test_2974484.svg>;
+    vcard:note
+        "This is a test account for testing versions of the SolidOS operating system for solid.";
+    vcard:organization-name "Solid";
+    vcard:role "foobar";
+    n0:trustedApp
+            [
+                n0:mode n0:Append, n0:Control, n0:Read, n0:Write;
+                n0:origin <https://timbl.com>
+            ];
+    ldp:inbox inbox:;
+    space:preferencesFile </settings/prefs.ttl>;
+    space:storage tes:;
+    solid:account tes:;
+    solid:preferredObjectPronoun "them";
+    solid:preferredRelativePronoun "foobar";
+    solid:preferredSubjectPronoun "they";
+    solid:privateTypeIndex </settings/privateTypeIndex.ttl>;
+    solid:profileBackgroundColor "#f4f5c2"^^xsd:color;
+    solid:profileHighlightColor "#06b74a"^^xsd:color;
+    solid:publicTypeIndex </settings/publicTypeIndex.ttl>;
+    foaf:knows c:me;
+    foaf:name "Testing SolidOS";
+    foaf:nick "tester1".
+l:de schema:name "germano"@ia.
+
+l:fr schema:name "French"@en.
+
+`
+describe("contacts-pane", () => {
+
+  describe("returns right label", () => {
+    beforeAll(async () => {
+      store.removeDocument(doc);
+      parse(exampleProfile, store, doc.uri);
+      // const label = pane.label(subject, context);
+    });
+/*
+if (t[ns.vcard('Individual').uri]) return 'Contact'
+if (t[ns.vcard('Organization').uri]) return 'contact'
+if (t[ns.foaf('Person').uri]) return 'Person'
+if (t[ns.schema('Person').uri]) return 'Person'
+if (t[ns.vcard('Group').uri]) return 'Group'
+if (t[ns.vcard('AddressBook').uri]) return 'Address book'
+
+*/
+    it("returns a good label Profile if appropriate", () => {
+      store.add(subject, ns.rdf('type'),  ns.vcard('Individual'), doc)
+      expect(pane.label(subject, context)).toEqual('Contact');
+    });
+    it("returns a null label if not a person", () => {
+      expect(pane.label(store.sym('https://random.example.com/'), context)).toEqual(null);
+    });
+
+  });
+
+});

package/__tests__/unit/setup.ts

@@ -0,0 +1,27 @@
+import { DataBrowserContext, PaneRegistry } from "pane-registry";
+import { sym } from "rdflib";
+import { SolidLogic, store } from "solid-logic";
+
+export const subject = sym("https://janedoe.example/profile/card#me");
+export const doc = subject.doc();
+
+export const context = {
+    dom: document,
+    getOutliner: () => null,
+    session: {
+        paneRegistry: {
+            byName: (name: string) => {
+                return {
+                    render: () => {
+                        return document.createElement('div')
+                            .appendChild(
+                                document.createTextNode(`mock ${name} pane`)
+                            );
+                    }
+                }
+            }
+        } as PaneRegistry,
+        store,
+        logic: {} as SolidLogic,
+    },
+} as unknown as DataBrowserContext;

package/babel.config.js

@@ -0,0 +1,13 @@
+module.exports = {
+  presets: [
+    [
+      "@babel/preset-env",
+      {
+        targets: {
+          node: "current",
+        },
+      },
+    ],
+    "@babel/preset-typescript",
+  ],
+};

package/contactLogic.js

@@ -194,3 +194,39 @@ export function groupMembers (kb, group) {
   return b
 }
 
+export function isLocal(group, item) {
+  const tree = group.dir().dir().dir()
+  const local = item.uri && item.uri.startsWith(tree.uri)
+  // console.log(`   isLocal ${local} for ${item.uri} in group ${group} tree ${tree.uri}`)
+  return local
+}
+
+export function getSameAs(kb, item, doc) {
+  return kb.each(item, ns.owl('sameAs'), null, doc).concat(
+      kb.each(null, ns.owl('sameAs'), item, doc))
+}
+
+export async function getDataModelIssues(groups) {
+  let del = []
+  let ins = []
+  groups.forEach(group => {
+    const members = kb.each(group, ns.vcard('hasMember'), null, group.doc())
+    members.forEach((member) => {
+      const others = getSameAs(kb, member, group.doc())
+      if (others.length && isLocal(group, member)) { // Problem: local ID used instead of webID
+        for (const other of others) {
+          if (!isLocal(group, other)) { // Let's use this one as the immediate member for CSS ACLs'
+            console.warn(`getDataModelIssues:  Need to swap ${member} to ${other}`)
+            del.push($rdf.st(group, ns.vcard('hasMember'), member, group.doc()))
+            ins.push($rdf.st(group, ns.vcard('hasMember'), other, group.doc()))
+            break
+          }
+          console.log('getDataModelIssues: ??? expected id not to be local ' + other)
+        } // other
+      } // if
+    }) // member
+  }) // next group
+  return {del, ins }
+} // getDataModelIssues
+
+// Ends

package/contactsPane.js

@@ -21,6 +21,7 @@ import { mintNewAddressBook } from './mintNewAddressBook'
 import { renderIndividual } from './individual'
 import { toolsPane } from './toolsPane'
 import { groupMembership } from './groupMembershipControl'
+import { getDataModelIssues } from './contactLogic'
 
 // const $rdf = UI.rdf
 const ns = UI.ns
@@ -227,10 +228,10 @@ export default {
                   let removeFromGroups = []
                   // find person WebID's
                   groups.map( group => {
-                    const webids = kb.each(null, ns.owl('sameAs'), person, group.doc())
+                    const webids = getSameAs(kb, person, group.doc())
                     // for each check in each Group that it is not used by an other person then delete
                     webids.map( webid => {
-                      if (kb.statementsMatching(webid, ns.owl('sameAs'), null, group.doc()).length = 1) {
+                      if (getSameAs(kb, webid, group.doc()).length = 1) {
                         removeFromGroups = removeFromGroups.concat(kb.statementsMatching(group, ns.vcard('hasMember'), webid, group.doc()))
                       }
                     })
@@ -312,8 +313,10 @@ export default {
         }
 
         async function loadAllGroups () {
+          await kb.fetcher.load(groupIndex)
           const gs = book ? kb.each(book, ns.vcard('includesGroup'), null, groupIndex) : []
           await kb.fetcher.load(gs)
+          return gs
         }
 
         function groupsInOrder () {
@@ -595,40 +598,20 @@ export default {
           const groups = groupsInOrder()
           utils.syncTableToArrayReOrdered(groupsMainTable, groups, renderGroupRow)
           refreshGroupsSelected()
-          checkDataModel(groups)
+          // await checkDataModel(groups)
         } // syncGroupTable
 
-        async function checkDataModel(groups) {
-          // check if migration is needed in groups
-          async function updateDataModel(groups) {
-            let ds = []
-            let ins = []
-            groups.forEach(group => {
-              let vcardOrWebids = kb.statementsMatching(null, ns.owl('sameAs'), null, group.doc()).map(st => st.subject)
-              const strings = new Set(vcardOrWebids.map(contact => contact.uri)) // remove dups
-              vcardOrWebids = [...strings].map(uri => kb.sym(uri))
-              vcardOrWebids.forEach(item => {
-                if (kb.each(item, ns.vcard('fn'), null, group.doc()).length) {
-                  // delete item, it is an old data model,  item is a card not a webid.
-                  ds = ds.concat(kb
-                    .statementsMatching(item, ns.owl('sameAs'), null, group.doc())
-                    .concat(kb.statementsMatching(undefined, undefined, item, group.doc())))
-                  // add card webids to group
-                  const webids = kb.each(item, ns.owl('sameAs'), null, group.doc())
-                  webids.forEach(webid => {
-                    ins = ins.concat($rdf.st(webid, ns.owl('sameAs'), item, group.doc()))
-                      .concat($rdf.st(group, ns.vcard('hasMember'), webid, group.doc()))
-                  })
-                }
-              })
-            })
-            if (ds.length && confirm('Groups data model need to be updated ?')) {
-              await kb.updater.updateMany(ds, ins)
-              alert('Update done')
-            }
+
+       async function checkDataModel () {
+          // await kb.fetcher.load(groups) // asssume loaded already
+          const groups = await loadAllGroups()
+
+          const { del, ins } = await getDataModelIssues(groups)
+
+          if (del.length && confirm(`Groups data model need to be updated? (${del.length})`)) {
+            await kb.updater.updateMany(del, ins)
+            alert('Update done')
           }
-          await kb.fetcher.load(groups)
-          updateDataModel(groups)
         } // checkDataModel
 
         // Click on New Group button
@@ -892,6 +875,10 @@ export default {
         // })
 
         div.appendChild(dom.createElement('hr'))
+
+        // const groups = await loadAllGroups()   @@@
+        checkDataModel().then(()=> {console.log('async checkDataModel done.')})
+
         //  div.appendChild(newAddressBookButton(book))       // later
         // end of AddressBook instance
       } // renderThreeColumnBrowser

package/jest.config.js

@@ -0,0 +1,5 @@
+module.exports = {
+  testEnvironment: 'jsdom',
+  setupFilesAfterEnv: ["./jest.setup.js"],
+  transformIgnorePatterns: ["/node_modules/(?!lit-html).+\\.js"],
+};

package/jest.setup.js

@@ -0,0 +1,4 @@
+// import "@testing-library/jest-dom";
+// import fetchMock from "jest-fetch-mock";
+
+// fetchMock.enableMocks();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "contacts-pane",
-  "version": "2.6.5-alpha",
+  "version": "2.6.5-d5c0117d",
   "description": "Contacts Pane: Contacts manager for Address Book, Groups, and Individuals.",
   "main": "./contactsPane.js",
   "scripts": {
@@ -10,8 +10,8 @@
     "lint": "eslint '*.js'",
     "lint-fix": "eslint '*.js' --fix",
     "test": "npm run lint",
-    "prepublishOnly": "npm test && npm run build",
-    "postpublish": "git push origin main --follow-tags"
+    "ignore:prepublishOnly": "npm test && npm run build",
+    "ignore:postpublish": "git push origin main --follow-tags"
   },
   "repository": {
     "type": "git",
@@ -37,17 +37,23 @@
   },
   "homepage": "https://github.com/solid/contacts-pane",
   "dependencies": {
-    "pane-registry": "2.4.12",
-    "rdflib": "^2.2.20",
-    "solid-logic": "2.0.0",
-    "solid-ui": "2.4.23"
+    "@babel/preset-env": "^7.20.2",
+    "babel": "^6.23.0",
+    "babel-cli": "^6.26.0",
+    "jest": "^29.3.1",
+    "jest-environment-jsdom": "^29.3.1",
+    "pane-registry": "2.4.13",
+    "rdflib": "^2.2.21",
+    "solid-logic": "2.0.1",
+    "solid-ui": "2.4.24"
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^5.36.1",
-    "@typescript-eslint/parser": "^5.36.1",
-    "eslint": "^8.23.0",
+    "@babel/preset-typescript": "^7.18.6",
+    "@typescript-eslint/eslint-plugin": "^5.43.0",
+    "@typescript-eslint/parser": "^5.43.0",
+    "eslint": "^8.27.0",
     "eslint-plugin-import": "^2.26.0",
-    "husky": "^8.0.1",
+    "husky": "^8.0.2",
     "lint-staged": "^13.0.3",
     "typescript": "^4.8.2",
     "typescript-transpile-only": "0.0.4"

package/webidControl.js

@@ -38,7 +38,7 @@ export async function addWebIDToContacts (person, webid, urlType, kb) {
 
   // check this is a url
   try {
-    const url = new URL(webid)
+    const _url = new URL(webid)
   } catch (error) {
     throw new Error(`${WEBID_NOUN}: ${webid} is not a valid url.`)
   }