Merge branch 'master' of https://github.com/a-schild/nextcloud-java-api

Author: a-schild

Changelog.md

@@ -1,8 +1,13 @@
 # Changelog for nextcloud api
-## Version 13.1.0
-- 2024-05-07
-  - Release 13.1.0
-  - Merged enhancements and cleanups (Thanks to mathiasbosman)
+## Version 13.0.2
+- 2024-03-16
+  - Typo fixes
+  - Javadoc url's
+  - Modifiers should be declared in the correct order
+  - Try-with resources
+  - Use constants where possible
+  - Removed redundant Exception throwing
+  - Simplified assertions in tests
 
 ## Version 13.0.1
 - 2023-09-29

pom.xml

@@ -3,7 +3,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>org.aarboard.nextcloud</groupId>
     <artifactId>nextcloud-api</artifactId>
-    <version>13.1.0-SNAPSHOT</version>
+    <version>13.1.0</version>
     <packaging>jar</packaging>
     <properties>
         <!-- compile time dependencies -->

src/main/java/org/aarboard/nextcloud/api/NextcloudConnector.java

@@ -41,7 +41,7 @@
 
 public class NextcloudConnector {
 
-    private final ServerConfig _serverConfig;
+    private final ServerConfig serverConfig;
     private final ProvisionConnector pc;
     private final FilesharingConnector fc;
     private final ConfigConnector cc;
@@ -73,7 +73,7 @@ public NextcloudConnector(String serverName, boolean useHTTPS, int port, String
 
     /**
      * @param serviceUrl url of the nextcloud instance, e.g.
-     * https://nextcloud.instance.com:8443/cloud
+     * <a href="https://nextcloud.instance.com:8443/cloud">...</a>
      * @param loginName User for login
      * @param password Password for login
      */
@@ -83,32 +83,32 @@ public NextcloudConnector(String serviceUrl, String loginName, String password)
 
     /**
      * @param serviceUrl url of the nextcloud instance, e.g.
-     * https://nextcloud.instance.com:8443/cloud
+     * <a href="https://nextcloud.instance.com:8443/cloud">...</a>
      * @param bearerToken Bearer token for login
      */
     public NextcloudConnector(String serviceUrl, String bearerToken) {
         this(serviceUrl, new AuthenticationConfig(bearerToken));
     }
 
     /**
-     * @param serviceUrl url of the nextcloud instance, e.g.
-     * https://nextcloud.instance.com:8443/cloud
+     * @param originalServiceUrl url of the nextcloud instance, e.g.
+     * <a href="https://nextcloud.instance.com:8443/cloud">...</a>
      * @param authenticationConfig Authentication config
      */
-    public NextcloudConnector(String serviceUrl, AuthenticationConfig authenticationConfig) {
+    public NextcloudConnector(String originalServiceUrl, AuthenticationConfig authenticationConfig) {
         try {
-            URL _serviceUrl = new URL(serviceUrl);
-            boolean useHTTPS = serviceUrl.startsWith("https");
-            _serverConfig = new ServerConfig(_serviceUrl.getHost(), useHTTPS, _serviceUrl.getPort(),
+            URL serviceUrl = new URL(originalServiceUrl);
+            boolean useHTTPS = originalServiceUrl.startsWith("https");
+            this.serverConfig = new ServerConfig(serviceUrl.getHost(), useHTTPS, serviceUrl.getPort(),
                     authenticationConfig);
-            if (!_serviceUrl.getPath().isEmpty()) {
-                _serverConfig.setSubPathPrefix(_serviceUrl.getPath());
+            if (!serviceUrl.getPath().isEmpty()) {
+                this.serverConfig.setSubPathPrefix(serviceUrl.getPath());
             }
-            pc = new ProvisionConnector(_serverConfig);
-            fc = new FilesharingConnector(_serverConfig);
-            cc = new ConfigConnector(_serverConfig);
-            fd = new Folders(_serverConfig);
-            fl = new Files(_serverConfig);
+            pc = new ProvisionConnector(this.serverConfig);
+            fc = new FilesharingConnector(this.serverConfig);
+            cc = new ConfigConnector(this.serverConfig);
+            fd = new Folders(this.serverConfig);
+            fl = new Files(this.serverConfig);
 
         } catch (MalformedURLException e) {
             throw new IllegalArgumentException(e);
@@ -124,12 +124,12 @@ public NextcloudConnector(String serviceUrl, AuthenticationConfig authentication
      */
     public NextcloudConnector(String serverName, boolean useHTTPS, int port,
             AuthenticationConfig authenticationConfig) {
-        _serverConfig = new ServerConfig(serverName, useHTTPS, port, authenticationConfig);
-        pc = new ProvisionConnector(_serverConfig);
-        fc = new FilesharingConnector(_serverConfig);
-        cc = new ConfigConnector(_serverConfig);
-        fd = new Folders(_serverConfig);
-        fl = new Files(_serverConfig);
+        this.serverConfig = new ServerConfig(serverName, useHTTPS, port, authenticationConfig);
+        pc = new ProvisionConnector(this.serverConfig);
+        fc = new FilesharingConnector(this.serverConfig);
+        cc = new ConfigConnector(this.serverConfig);
+        fd = new Folders(this.serverConfig);
+        fl = new Files(this.serverConfig);
     }
 
     /**
@@ -171,8 +171,8 @@ public void setWebDavPathResolverAsType(final WebDavPathResolverBuilder.TYPE typ
         WebDavPathResolver resolver = WebDavPathResolverBuilder.get(type)
                 .ofVersion(NextcloudVersion.get(getServerVersion()))
                 .withUserName(getCurrentUser().getId())
-//                .withUserName(_serverConfig.getUserName())
-                .withBasePathPrefix(_serverConfig.getSubPathPrefix()).build();
+//                .withUserName(this.serverConfig.getUserName())
+                .withBasePathPrefix(this.serverConfig.getSubPathPrefix()).build();
 
         this.fd.setWebDavPathResolver(resolver);
         this.fl.setWebDavPathResolver(resolver);
@@ -192,10 +192,10 @@ public void shutdown() throws IOException {
      * Trust all HTTPS certificates presented by the server. This is e.g. used
      * to work against a Nextcloud instance with a self-signed certificate.
      *
-     * @param trustAllCertificates Do we accep self signed certificates or not
+     * @param trustAllCertificates Do we accept self-signed certificates or not
      */
     public void trustAllCertificates(boolean trustAllCertificates) {
-        _serverConfig.setTrustAllCertificates(trustAllCertificates);
+        this.serverConfig.setTrustAllCertificates(trustAllCertificates);
     }
 
     /**
@@ -206,7 +206,7 @@ public void trustAllCertificates(boolean trustAllCertificates) {
      * installed in root
      */
     public void setSubpathPrefix(String subpathPrefix) {
-        _serverConfig.setSubPathPrefix(subpathPrefix);
+        this.serverConfig.setSubPathPrefix(subpathPrefix);
     }
 
     /**
@@ -662,7 +662,7 @@ public CompletableFuture<UserDetailsAnswer> getUserAsync(String userId) {
     }
 
     /**
-     * Gets user details of currently logged in user
+     * Gets user details of currently logged-in user
      *
      * @return all user details
      */
@@ -968,7 +968,7 @@ public void uploadFile(File srcFile, String remotePath) {
      * @deprecated Since some nextcloud installations use fpm or fastcgi to
      * connect to php, here the uploads might get zero empty on the server Use a
      * (temp) file to upload the data, so the content length is known in advance
-     * https://github.com/a-schild/nextcloud-java-api/issues/20
+     * <a href="https://github.com/a-schild/nextcloud-java-api/issues/20">...</a>
      */
     public void uploadFile(InputStream inputStream, String remotePath) {
         fl.uploadFile(inputStream, remotePath);
@@ -987,7 +987,7 @@ public void uploadFile(InputStream inputStream, String remotePath) {
      * @deprecated Since some nextcloud installations use fpm or fastcgi to
      * connect to php, here the uploads might get zero empty on the server Use a
      * (temp) file to upload the data, so the content length is known in advance
-     * https://github.com/a-schild/nextcloud-java-api/issues/20
+     * <a href="https://github.com/a-schild/nextcloud-java-api/issues/20">...</a>
      */
     public void uploadFile(InputStream inputStream, String remotePath, boolean continueHeader) {
         fl.uploadFile(inputStream, remotePath, continueHeader);
@@ -1136,7 +1136,7 @@ public CompletableFuture<XMLAnswer> editShareAsync(int shareId, Map<ShareData, S
      * nextcloud server
      * @param downloadpath Local path where the file has to be downloaded in the
      * local machine
-     * @return boolean true if sucessfull
+     * @return boolean true if successful
      * @throws java.io.IOException In case of IO errors
      */
     public boolean downloadFile(String remotepath, String downloadpath) throws IOException {
@@ -1219,7 +1219,7 @@ public String getAppConfigAppKeyValue(String appConfigAppKeyPath) {
      * @param appConfigAppKey a key name as returned by
      * {@link #getAppConfigAppKeys(String)}
      * @param value the value to set
-     * @return true if sucessfully set
+     * @return true if successfully set
      */
     public boolean setAppConfigAppKeyValue(String appConfigApp, String appConfigAppKey, Object value) {
         return cc.setAppConfigAppKeyValue(appConfigApp, appConfigAppKey, value);
@@ -1230,7 +1230,7 @@ public boolean setAppConfigAppKeyValue(String appConfigApp, String appConfigAppK
      * @param appConfigAppKeyPath the full appConfigAppKeyPath combining
      * appConfigApp and appConfigAppKey with "/"
      * @param value the value to set
-     * @return Operation sucessfull
+     * @return Operation successful
      */
     public boolean setAppConfigAppKeyValue(String appConfigAppKeyPath, Object value) {
         return cc.setAppConfigAppKeyValue(appConfigAppKeyPath, value);
@@ -1243,7 +1243,7 @@ public boolean setAppConfigAppKeyValue(String appConfigAppKeyPath, Object value)
      * {@link #getAppConfigApps()}
      * @param appConfigAppkey a key name as returned by
      * {@link #getAppConfigAppKeys(String)}
-     * @return Operation sucessfull
+     * @return Operation successful
      */
     public boolean deleteAppConfigAppKeyEntry(String appConfigApp, String appConfigAppkey) {
         return cc.deleteAppConfigAppKeyEntry(appConfigApp, appConfigAppkey);