oldaplib 0.3.8__tar.gz → 0.3.10__tar.gz

{oldaplib-0.3.8 → oldaplib-0.3.10}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: oldaplib
-Version: 0.3.8
+Version: 0.3.10
 Summary: Open Media Access Server Library (Linked Open Data middleware/RESTApi)
 License: GNU Affero General Public License version 3
 Author: Lukas Rosenthaler

{oldaplib-0.3.8 → oldaplib-0.3.10}/oldaplib/src/helpers/observable_set.py

@@ -80,7 +80,7 @@ class ObservableSet(Notify):
     def __str__(self) -> str:
         return str(self._setdata)
 
-    def __eq__(self, other: Iterable) -> bool:
+    def __eq__(self, other: Iterable[Any]) -> bool:
         if isinstance(other, ObservableSet):
             return self._setdata == other._setdata
         elif isinstance(other, set):
@@ -90,7 +90,7 @@ class ObservableSet(Notify):
         else:
             raise OldapErrorNotImplemented(f'Set.__eq__() not implemented for {type(other).__name__}')
 
-    def __or__(self, other: Iterable) -> Self:
+    def __or__(self, other: Iterable[Any]) -> Self:
         if isinstance(other, ObservableSet):
             return ObservableSet(self._setdata.__or__(other._setdata), self._notifier, self._notify_data)
         elif isinstance(other, set):
@@ -100,25 +100,24 @@ class ObservableSet(Notify):
         else:
             raise OldapErrorNotImplemented(f'Set.__or__() not implemented for {type(other).__name__}')
 
-    def __ror__(self, other: Self) -> Self:
-        pass
+    def __ror__(self, other: Iterable[Any]) -> Self:
+        return ObservableSet(set(other).__or__(self._setdata), self._notifier, self._notify_data)
+
+    def __rsub__(self, other: Iterable[Any]) -> Self:
+        return ObservableSet(set(other).__sub__(self._setdata), self._notifier, self._notify_data)
 
-    def __ior__(self, other: Iterable) -> Self:
+    def __ior__(self, other: Iterable[Any]) -> Self:
         tmp_copy = deepcopy(self)
         if isinstance(other, ObservableSet):
             self._setdata.__ior__(other._setdata)
-        elif isinstance(other, set):
-            self._setdata.__ior__(other)
-        elif isinstance(other, Iterable):
-            return ObservableSet(self._setdata.__ior__(set(other)), self._notifier, self._notify_data)
         else:
-            raise OldapErrorNotImplemented(f'Set.i__or__() not implemented for {type(other).__name__}')
+            self._setdata.__ior__(set(other))
         if not self._old_value:
             self._old_value = tmp_copy
         self.notify()
         return self
 
-    def __and__(self, other: Iterable) -> Self:
+    def __and__(self, other: Iterable[Any]) -> Self:
         if isinstance(other, ObservableSet):
             return ObservableSet(self._setdata.__and__(other._setdata), self._notifier, self._notify_data)
         elif isinstance(other, set):
@@ -128,7 +127,7 @@ class ObservableSet(Notify):
         else:
             raise OldapErrorNotImplemented(f'Set.__and__() not implemented for {type(other).__name__}')
 
-    def __iand__(self, other: Iterable) -> Self:
+    def __iand__(self, other: Iterable[Any]) -> Self:
         tmp_copy = deepcopy(self)
         if isinstance(other, ObservableSet):
             self._setdata.__iand__(other._setdata)
@@ -146,7 +145,7 @@ class ObservableSet(Notify):
     def __rsub__(self, other: Self) -> Self:
         pass
 
-    def __sub__(self, other: Iterable) -> Self:
+    def __sub__(self, other: Iterable[Any]) -> Self:
         if isinstance(other, ObservableSet):
             return ObservableSet(self._setdata.__sub__(other._setdata), self.notify, self._notify_data)
         elif isinstance(other, set):
@@ -156,7 +155,7 @@ class ObservableSet(Notify):
         else:
             raise OldapErrorNotImplemented(f'Set.__sub__() not implemented for {type(other).__name__}')
 
-    def __isub__(self, other: Iterable) -> Self:
+    def __isub__(self, other: Iterable[Any]) -> Self:
         tmp_copy = deepcopy(self)
         if isinstance(other, ObservableSet):
             self._setdata.__isub__(other._setdata)
@@ -171,34 +170,45 @@ class ObservableSet(Notify):
         self.notify()
         return self
 
-    def update(self, items: Iterable):
+    @classmethod
+    def coerce(cls, value: Iterable[Any], *, notifier=None, notify_data=None) -> "ObservableSet":
+        return value if isinstance(value, cls) else cls(value, notifier=notifier, notify_data=notify_data)
+
+    def update(self, items: Iterable[Any]):
         tmp_copy = deepcopy(self)
         self._setdata.update(items)
         if not self._old_value:
             self._old_value = tmp_copy
         self.notify()
 
-    def intersection_update(self, items: Iterable):
+    def intersection_update(self, items: Iterable[Any]):
         tmp_copy = deepcopy(self)
         self._setdata.intersection_update(items)
         if not self._old_value:
             self._old_value = tmp_copy
         self.notify()
 
-    def difference_update(self, items: Iterable):
+    def difference_update(self, items: Iterable[Any]):
         tmp_copy = deepcopy(self)
         self._setdata.difference_update(items)
         if not self._old_value:
             self._old_value = tmp_copy
         self.notify()
 
-    def symmetric_difference_update(self, items: Iterable):
+    def symmetric_difference_update(self, items: Iterable[Any]):
         tmp_copy = deepcopy(self)
         self._setdata.symmetric_difference_update(items)
         if not self._old_value:
             self._old_value = tmp_copy
         self.notify()
 
+    def replace(self, items: Iterable[Any]) -> None:
+        tmp_copy = deepcopy(self)
+        self._setdata = set(items)
+        if not self._old_value:
+            self._old_value = tmp_copy
+        self.notify()
+
     def add(self, item: Any) -> None:
         tmp_copy = deepcopy(self)
         self._setdata.add(item)

{oldaplib-0.3.8 → oldaplib-0.3.10}/oldaplib/src/objectfactory.py

@@ -443,33 +443,24 @@ class ResourceInstance:
 
     def get_data_permission(self, context: Context, permission: DataPermission) -> bool:
         permission_query = context.sparql_context
-        # language=sparql
         permission_query += f'''
-        SELECT (COUNT(?permset) as ?numOfPermsets)
-        FROM oldap:onto
-        FROM shared:onto
-        FROM {self._graph}:onto
-        FROM NAMED oldap:admin
-        FROM NAMED {self._graph}:data
-        WHERE {{
-            BIND({self._iri.toRdf} as ?iri)
+        ASK {{
             GRAPH {self._graph}:data {{
-                ?iri oldap:grantsPermission ?permset .
+                {self._iri.toRdf} oldap:grantsPermission ?permset .
             }}
-            BIND({self._con.userIri.toRdf} as ?user)
             GRAPH oldap:admin {{
-                ?user oldap:hasPermissions ?permset .
+                {self._con.userIri.toRdf} oldap:hasPermissions ?permset .
                 ?permset oldap:givesPermission ?DataPermission .
                 ?DataPermission oldap:permissionValue ?permval .
             }}
             FILTER(?permval >= {permission.numeric.toRdf})
         }}'''
+
         if self._con.in_transaction():
-            jsonobj = self._con.transaction_query(permission_query)
+            result = self._con.transaction_query(permission_query)
         else:
-            jsonobj = self._con.query(permission_query)
-        res = QueryProcessor(context, jsonobj)
-        return res[0]['numOfPermsets'] > 0
+            result = self._con.query(permission_query)
+        return result['boolean']
 
     def create(self, indent: int = 0, indent_inc: int = 4) -> str:
         result, message = self.check_for_permissions(AdminPermission.ADMIN_CREATE)
@@ -707,12 +698,13 @@ class ResourceInstance:
 
         context = Context(name=self._con.context_name)
         inuse = context.sparql_context
-        inuse = f'''
-        SELECT (COUNT(?res) as ?nres)
-        WHERE {{
-            ?res ?prop {self._iri.toRdf} .
+        inuse += f"""
+        ASK {{
+            GRAPH ?g {{
+                ?res ?prop {self._iri.toRdf} .
+            }}
         }}
-        '''
+        """
 
         context = Context(name=self._con.context_name)
         sparql = context.sparql_context
@@ -730,9 +722,8 @@ class ResourceInstance:
                 self._con.transaction_abort()
                 raise OldapErrorNoPermission(f'No permission to update resource "{self._iri}"')
         try:
-            jsonobj = self._con.transaction_query(inuse)
-            res = QueryProcessor(context, jsonobj)
-            if res[0]['nres'] > 0:
+            result = self._con.transaction_query(inuse)
+            if result['boolean']:
                 raise OldapErrorInUse(f'Resource "{self._iri}" is in use and cannot be deleted.')
         except OldapError:
             self._con.transaction_abort()
@@ -791,9 +782,6 @@ class ResourceInstanceFactory:
 
         self._datamodel = DataModel.read(con=self._con, project=self._project, ignore_cache=True)
 
-        #self._oldap_project = Project.read(self._con, "oldap")
-        #self._oldap_datamodel = DataModel.read(con=self._con, project=self._oldap_project)
-
     def createObjectInstance(self, name: Xsd_NCName | str) -> Type:  ## ToDo: Get name automatically from IRI
         classiri = Xsd_QName(self._project.projectShortName, name)
         resclass = self._datamodel.get(classiri)
@@ -849,7 +837,6 @@ class ResourceInstanceFactory:
             raise OldapErrorNotFound(f'Resource with iri <{iri}> not found.')
         return data
 
-
     def read(self, iri: Iri | str) -> ResourceInstance:
         if not isinstance(iri, Iri):
             iri = Iri(iri, validate=True)
@@ -901,9 +888,18 @@ class ResourceInstanceFactory:
         Instance = self.createObjectInstance(objtype)
         return Instance(iri=iri, **kwargs)
 
-    def search_fulltext(self, s: str, count_only: bool = False, limit: int = 100, offset: int = 0) -> int | dict[Iri, dict[str, Xsd]]:
-        graph = self._project.projectShortName
-        context = Context(name=self._con.context_name)
+    @staticmethod
+    def search_fulltext(con: IConnection,
+                        projectShortName: Xsd_NCName | str,
+                        s: str,
+                        count_only: bool = False,
+                        limit: int = 100,
+                        offset: int = 0) -> int | dict[Iri, dict[str, Xsd]]:
+        if not isinstance(projectShortName, Xsd_NCName):
+            graph = Xsd_NCName(projectShortName)
+        else:
+            graph = projectShortName
+        context = Context(name=con.context_name)
         sparql = context.sparql_context
         if (count_only):
             sparql += "SELECT (COUNT(DISTINCT ?s) as ?numResult)"
@@ -920,7 +916,7 @@ class ResourceInstanceFactory:
                 (datatype(?o) = xsd:string || datatype(?o) = rdf:langString || lang(?o) != ""))
             FILTER(CONTAINS(LCASE(STR(?o)), "{s}"))  # case-insensitive substring match
             GRAPH oldap:admin {{
-    	        {self._con.userIri.toRdf} oldap:hasPermissions ?permset .
+    	        {con.userIri.toRdf} oldap:hasPermissions ?permset .
     	        ?permset oldap:givesPermission ?DataPermission .
     	        ?DataPermission oldap:permissionValue ?permval .
             }}
@@ -929,21 +925,31 @@ class ResourceInstanceFactory:
         '''
         if not count_only:
             sparql += f'LIMIT {limit} OFFSET {offset}'
-        jsonres = self._con.query(sparql)
+        try:
+            jsonres = con.query(sparql)
+        except OldapError:
+            print(sparql)
+            raise
         res = QueryProcessor(context, jsonres)
         if count_only:
-            if isinstance(res[0]['numResult'], Xsd_integer):
-                tmp = cast(Xsd_integer, res[0]['numResult'])
-                return int(tmp)
-            else:
-                raise OldapErrorInconsistency(f'Expected integer as value, got "{res[0]["numResult"]}"')
+            return res[0]['numResult']
         else:
-            result: dict[Iri, dict[str, Xsd]] = {}
-            for r in res:
-                iri = cast(Iri, r['s'])
-                resclass = cast(Iri, r['t'])
-                result[iri] = {'resclass': resclass, 'property': r['p'], 'value': r['o']}
-            return result
+            return {Iri(r['s']): {r['p']: r['o']} for r in res}
+        # jsonres = con.query(sparql)
+        # res = QueryProcessor(context, jsonres)
+        # if count_only:
+        #     if isinstance(res[0]['numResult'], Xsd_integer):
+        #         tmp = cast(Xsd_integer, res[0]['numResult'])
+        #         return int(tmp)
+        #     else:
+        #         raise OldapErrorInconsistency(f'Expected integer as value, got "{res[0]["numResult"]}"')
+        # else:
+        #     result: dict[Iri, dict[str, Xsd]] = {}
+        #     for r in res:
+        #         iri = cast(Iri, r['s'])
+        #         resclass = cast(Iri, r['t'])
+        #         result[iri] = {'resclass': resclass, 'property': r['p'], 'value': r['o']}
+        #     return result
 
 
 

oldaplib-0.3.10/oldaplib/src/version.py

@@ -0,0 +1 @@
+__version__ = "0.3.10"

{oldaplib-0.3.8 → oldaplib-0.3.10}/oldaplib/test/test_objectfactory.py

@@ -645,7 +645,9 @@ class TestObjectFactory(unittest.TestCase):
                      pubDate="1995-09-27",
                      grantsPermission=Iri('oldap:GenericView'))
             b.create()
-        res = factory.search_fulltext('tales')
+        res = ResourceInstanceFactory.search_fulltext(con=self._connection,
+                                                      projectShortName='test',
+                                                      s='tales')
 
         for x, y in res.items():
             print(x, y)

{oldaplib-0.3.8 → oldaplib-0.3.10}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "oldaplib"
-version = "0.3.8"
+version = "0.3.10"
 description = "Open Media Access Server Library (Linked Open Data middleware/RESTApi)"
 authors = ["Lukas Rosenthaler <lukas.rosenthaler@unibas.ch>"]
 license = "GNU Affero General Public License version 3"

oldaplib-0.3.8/oldaplib/src/version.py

@@ -1 +0,0 @@
-__version__ = "0.3.8"