roll back google changes

Author: jonocodes

release/remotestorage.js

@@ -108,6 +108,36 @@ class FileIdCache {
   }
 }
 
+/**
+ * Overwrite BaseClient's getItemURL with our own implementation
+ *
+ * TODO: Still needs to be implemented. At the moment it just throws
+ * and error saying that it's not implemented yet.
+ *
+ * @param {object} rs - RemoteStorage instance
+ *
+ * @private
+ */
+function hookGetItemURL (rs): void {
+  if (rs._origBaseClientGetItemURL) { return; }
+  rs._origBaseClientGetItemURL = BaseClient.prototype.getItemURL;
+  BaseClient.prototype.getItemURL = function (/* path */): never {
+    throw new Error('getItemURL is not implemented for Google Drive yet');
+  };
+}
+
+/**
+ * Restore BaseClient's getItemURL original implementation
+ *
+ * @param {object} rs - RemoteStorage instance
+ *
+ * @private
+ */
+function unHookGetItemURL (rs): void {
+  if (!rs._origBaseClientGetItemURL) { return; }
+  BaseClient.prototype.getItemURL = rs._origBaseClientGetItemURL;
+  delete rs._origBaseClientGetItemURL;
+}
 
 /**
  * @class GoogleDrive
@@ -702,18 +732,6 @@ class GoogleDrive extends RemoteBase implements Remote {
    *
    * @protected
    */
-  /**
-   * Google Drive does not support unauthenticated public file access, so
-   * this always resolves to ``undefined``. See GitHub issue #1051 for the
-   * full discussion.
-   *
-   * Implements {@link Remote.getItemURL}.
-   */
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  async getItemURL (_path: string): Promise<string | undefined> {
-    return undefined;
-  }
-
   static _rs_init (remoteStorage): void {
     const config = remoteStorage.apiKeys.googledrive;
     if (config) {
@@ -722,6 +740,7 @@ class GoogleDrive extends RemoteBase implements Remote {
         remoteStorage._origRemote = remoteStorage.remote;
         remoteStorage.remote = remoteStorage.googledrive;
 
+        hookGetItemURL(remoteStorage);
       }
     }
   }
@@ -750,6 +769,7 @@ class GoogleDrive extends RemoteBase implements Remote {
       remoteStorage.remote = remoteStorage._origRemote;
       delete remoteStorage._origRemote;
     }
+    unHookGetItemURL(remoteStorage);
   }
 }