logging method

methods call from constructor should be final Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
nextcloud · Dec 2, 2019 · 0e59023 · 0e59023
1 parent 44ba1e6
commit 0e59023
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/main/java/com/owncloud/android/lib/common/operations/RemoteOperation.java b/src/main/java/com/owncloud/android/lib/common/operations/RemoteOperation.java
@@ -91,11 +91,13 @@ public abstract class RemoteOperation implements Runnable {
 	 */
 	@Deprecated
     protected RemoteOperationResult run(OwnCloudClient client) {
+        Log_OC.d(this, "Not used anymore");
         throw new UnsupportedOperationException("Not used anymore");
     }
 
     protected RemoteOperationResult run(NextcloudClient client) {
         // Once all RemoteOperation implement it, this should be abstract 
+        Log_OC.d(this, "Not yet implemented");
         throw new UnsupportedOperationException("Not yet implemented");
     }
 

diff --git a/src/main/java/com/owncloud/android/lib/common/operations/RemoteOperationResult.java b/src/main/java/com/owncloud/android/lib/common/operations/RemoteOperationResult.java
@@ -664,7 +664,7 @@ public String getRedirectedLocation() {
         return mRedirectedLocation;
     }
 
-    public boolean isIdPRedirection() {
+    public final boolean isIdPRedirection() {
         return (mRedirectedLocation != null &&
                 (mRedirectedLocation.toUpperCase(Locale.US).contains("SAML") ||
                         mRedirectedLocation.toLowerCase(Locale.US).contains("wayf")));