get_owner: simplify, str2func

Author: scivision

+stdlib/+dotnet/get_owner.m

@@ -5,7 +5,7 @@
 % This is not yet possible with .NET on Unix, even with .NET 10.
 % It would require Pinvoke or external Mono.Unix
 
-try
+if ispc() && stdlib.has_dotnet()
   ntAccountType = System.Type.GetType('System.Security.Principal.NTAccount');
 
   if isfolder(file)
@@ -20,8 +20,8 @@
   owner = fsec.GetOwner(ntAccountType);
 
   o = char(owner.ToString());
-catch e
-  o = dotnetException(e);
+else
+  o = missing;
 end
 
 end

+stdlib/+java/get_owner.m

@@ -1,29 +1,23 @@
 %% JAVA.GET_OWNER get owner of file
 
-
 function n = get_owner(file)
 
 % 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 = missing;
-if stdlib.strempty(file)
-  return
-end
-
 % Java 1.8 benefits from absolute.
 % We only saw this issue with R2025a on windows-2025 GA runner image.
 %
 % if stdlib.exists() was not adequate here, as on some CI systems, despite the same setup on a laptop working.
 % stdlib.exists() was true, the Java function threw java.nio.file.NoSuchFileException.
 %
 % this try-catch is faster and more robust
-try
+if stdlib.has_java()
   p = javaAbsolutePath(file);
   opt = javaMethod('values', 'java.nio.file.LinkOption');
   n = char(javaMethod('getOwner', 'java.nio.file.Files', p, opt));
-catch e
-  javaException(e);
+else
+  n = missing;
 end
 
 end

+stdlib/+python/get_owner.m

@@ -1,15 +1,9 @@
 function n = get_owner(file)
 
-n = missing;
-
-if stdlib.strempty(file)
-  return
-end
-
-try
+if stdlib.has_python()
   n = char(py.pathlib.Path(file).owner());
-catch e
-  pythonException(e);
+else
+  n = missing;
 end
 
 end

+stdlib/+shell/get_owner.m

@@ -11,11 +11,9 @@
 end
 
 % Windows needs exists() rather than just ~strempty()
-if stdlib.exists(file)
-  [s, m] = system(cmd);
-  if s == 0
-    o = deblank(m);
-  end
+[s, m] = system(cmd);
+if s == 0
+  o = deblank(m);
 end
 
 end

+stdlib/get_owner.m

@@ -9,27 +9,16 @@
 
 function [r, b] = get_owner(file, backend)
 arguments
-  file {mustBeTextScalar}
+  file {mustBeTextScalar,mustBeFileOrFolder}
   backend (1,:) string = ["java", "dotnet", "python", "shell"]
 end
 
 r = missing;
 
 for b = backend
-  switch b
-    case 'java'
-      r = stdlib.java.get_owner(file);
-    case 'dotnet'
-      r = stdlib.dotnet.get_owner(file);
-    case 'python'
-      if stdlib.has_python()
-        r = stdlib.python.get_owner(file);
-      end
-    case 'shell'
-      r = stdlib.shell.get_owner(file);
-    otherwise
-      error('stdlib:get_owner:ValueError', 'Unknown backend: %s', b)
-  end
+
+  f = str2func("stdlib." + b + ".get_owner");
+  r = f(file);
 
   if ~ismissing(r)
     return

test/TestDisk.m

@@ -159,7 +159,7 @@ function test_remove_file(tc)
 
 function test_device(tc, Ps, B_jps)
 if ~stdlib.exists(Ps)
-  tc.verifyError(@() stdlib.device(Ps), 'MATLAB:validators:mustBeFileOrFolder')
+  tc.verifyError(@() stdlib.device(Ps, B_jps), 'MATLAB:validators:mustBeFileOrFolder')
 else
   [i, b] = stdlib.device(Ps, B_jps);
   if ismember(B_jps, stdlib.Backend().select('device'))
@@ -177,7 +177,7 @@ function test_device(tc, Ps, B_jps)
 function test_inode(tc, Ps, B_jps)
 
 if ~stdlib.exists(Ps)
-  tc.verifyError(@() stdlib.inode(Ps), 'MATLAB:validators:mustBeFileOrFolder')
+  tc.verifyError(@() stdlib.inode(Ps, B_jps), 'MATLAB:validators:mustBeFileOrFolder')
 else
   [i, b] = stdlib.inode(Ps, B_jps);
   if ismember(B_jps, stdlib.Backend().select('inode'))
@@ -194,6 +194,11 @@ function test_inode(tc, Ps, B_jps)
 
 
 function test_owner(tc, Ps, B_jdps)
+
+if ~stdlib.exists(Ps)
+  tc.verifyError(@() stdlib.get_owner(Ps, B_jdps), 'MATLAB:validators:mustBeFileOrFolder')
+else
+
 [o, b] = stdlib.get_owner(Ps, B_jdps);
 tc.assertMatches(b, B_jdps)
 
@@ -208,6 +213,8 @@ function test_owner(tc, Ps, B_jdps)
   tc.verifyEqual(o, missing)
 end
 
+end % stdlib.exists
+
 end