try usng a promise

Former-commit-id: b39126fdd9

_embed/public/js/common.js

@@ -123,20 +123,21 @@ function getCSSRule(rules) {
 // We must create functions that do the requests to the webdav backend.
 // this functions will contain a 'callback' to be used withing the other function. 
 
-webdav.rename = function(oldLink, newLink, callback) {
-    let request = new XMLHttpRequest();
-
-    request.open('MOVE', toWebDavURL(oldLink));
-    request.setRequestHeader('Destination', toWebDavURL(newLink));
-    request.send();
-    request.onreadystatechange = function() {
-        if (request.readyState == 4) {
-            if (typeof callback == 'function') {
-                // This callback argument is a 'success'
-                callback(request.status == 201 || request.status == 204);
+webdav.move = function(oldLink, newLink) {
+    return new Promise((resolve, reject) => {
+        let request = new XMLHttpRequest();
+        request.open('MOVE', toWebDavURL(oldLink), true);
+        request.setRequestHeader('Destination', toWebDavURL(newLink));
+        request.onload = () => {
+            if (request.status == 201 || request.status == 204) {
+                resolve(request.response);
+            } else {
+                reject(request.statusText);
             }
         }
-    }
+        request.onerror = () => reject(request.statusText);
+        request.send();
+    });
 }
 
 

_embed/public/js/listing.js

@@ -130,22 +130,23 @@ listing.rename = function(event) {
         let newName = event.currentTarget.querySelector('input').value,
             newLink = removeLastDirectoryPartOf(link) + "/" + newName,
             html = buttons.rename.querySelector('i').changeToLoading();
+        closePrompt(event);
 
-        webdav.rename(link, newLink, success => {
-            if (success) {
+        webdav.move(link, newLink)
+            .then(data => {
                 listing.reload(() => {
                     newName = btoa(newName);
                     selectedItems = [newName];
                     document.getElementById(newName).setAttribute("aria-selected", true);
                     listing.handleSelectionChange();
                 });
-            } else {
-                item.querySelector('.name').innerHTML = name;
-            }
 
-            closePrompt(event);
-            buttons.rename.querySelector('i').changeToDone(!success, html);
-        });
+                buttons.rename.querySelector('i').changeToDone(false, html);
+            })
+            .catch(error => {
+                item.querySelector('.name').innerHTML = name;
+                buttons.rename.querySelector('i').changeToDone(true, html);
+            });
             
         return false;
     }