owner: handle scalar or arrays

Author: scivision

+stdlib/+dotnet/get_owner.m

@@ -15,12 +15,12 @@
 elseif isfile(p)
   fsec = System.IO.File.GetAccessControl(p);
 else
-  o = '';
+  o = string.empty;
   return
 end
 
 owner = fsec.GetOwner(ntAccountType);
 
-o = char(owner.ToString());
+o = string(owner.ToString());
 
 end

+stdlib/+java/get_owner.m

@@ -5,18 +5,11 @@
 % https://docs.oracle.com/en/java/javase/22/docs/api/java.base/java/nio/file/Files.html#getOwner(java.nio.file.Path,java.nio.file.LinkOption...)
 % https://docs.oracle.com/en/java/javase/22/docs/api/java.base/java/nio/file/LinkOption.html
 
-n = '';
-if ~stdlib.exists(p)
-  return
-end
-
-
-try
+if stdlib.exists(p)
   opt = javaMethod("values", "java.nio.file.LinkOption");
-
-  n = char(javaMethod("getOwner", "java.nio.file.Files", javaPathObject(p), opt));
-catch e
-  warning(e.identifier, "get_owner(%s) failed: %s", p, e.message)
+  n = string(java.nio.file.Files.getOwner(javaPathObject(p), opt));
+else
+  n = "";
 end
 
 end

+stdlib/+python/get_owner.m

@@ -1,14 +1,9 @@
 function n = get_owner(p)
 
-n = '';
-if ~stdlib.exists(p)
-  return
-end
-
-try
-  n = char(py.str(py.pathlib.Path(p).owner()));
-catch e
-  warning(e.identifier, "get_owner(%s) failed: %s", p, e.message)
+if stdlib.exists(p)
+  n = string(py.str(py.pathlib.Path(p).owner()));
+else
+  n = "";
 end
 
 end

+stdlib/+sys/get_owner.m

@@ -1,6 +1,6 @@
 function o = get_owner(p)
 
-o = '';
+o = "";
 if ~stdlib.exists(p)
   return
 end
@@ -15,7 +15,7 @@
 
 [s, m] = system(cmd);
 if s == 0
-  o = strip(m);
+  o = string(strip(m));
 end
 
 end

+stdlib/get_modtime.m

@@ -1,17 +1,17 @@
 %% GET_MODTIME get path modification time
 %
 %%% Inputs
-% * p: path to examine
+% * file: path to examine
 %%% Outputs
 % * t: modification time, or empty if path does not exist
 
-function t = get_modtime(p)
+function t = get_modtime(file)
 arguments
-  p {mustBeTextScalar}
+  file
 end
 
-finf = dir(p);
-if isfolder(p)
+finf = dir(file);
+if isfolder(file)
   % find which index of the struct array has member name == '.'
   i = find(strcmp({finf.name}, '.'), 1);
   finf = finf(i);

+stdlib/get_owner.m

@@ -1,19 +1,24 @@
 %% GET_OWNER owner name of file or directory
 %
 %%% Inputs
-% * p: path to examine
+% * file: path to examine
+% * backend: backend to use
 %%% Outputs
 % * n: owner, or empty if path does not exist
 % * b: backend used
 
 function [n, b] = get_owner(file, backend)
 arguments
-  file {mustBeTextScalar}
+  file string
   backend (1,:) string = ["java", "dotnet", "python", "sys"]
 end
 
 [fun, b] = hbackend(backend, "get_owner");
 
-n = fun(file);
+if isscalar(file)
+  n = fun(file);
+else
+  n = arrayfun(fun, file);
+end
 
 end

test/TestDisk.m

@@ -141,7 +141,6 @@ function test_inode_device(tc, java_python_sys, id_name)
 
 
 function test_owner(tc, Po, all_fun)
-tc.assertNotEmpty(which("stdlib." + all_fun + ".get_owner"))
 
 try
   o = stdlib.get_owner(Po, all_fun);
@@ -150,12 +149,25 @@ function test_owner(tc, Po, all_fun)
   return
 end
 
-tc.verifyClass(o, 'char')
+tc.verifyClass(o, "string")
 
 if stdlib.exists(Po)
   tc.verifyGreaterThan(strlength(o), 0)
 else
-  tc.verifyEmpty(o)
+  tc.verifyEqual(o, "")
+end
+
+end
+
+
+function test_owner_array(tc, all_fun)
+try
+  o = stdlib.get_owner([".", pwd(), "not-exist", ""], all_fun);
+  L = strlength(o);
+  tc.verifyEqual(L(3:4), [0, 0])
+  tc.verifyGreaterThan(strlength(o(1:2)), 0);
+catch e
+  tc.verifyEqual(e.identifier, 'stdlib:hbackend:NameError', e.message)
 end
 
 end