diff --git a/core/ajax/share.php b/core/ajax/share.php index 11fee4a000..3582f8596b 100644 --- a/core/ajax/share.php +++ b/core/ajax/share.php @@ -18,37 +18,38 @@ * You should have received a copy of the GNU Affero General Public * License along with this library. If not, see . */ +require_once '../../lib/base.php'; -OCP\JSON::checkLoggedIn(); +OC_JSON::checkLoggedIn(); if (isset($_POST['action'])) { switch ($_POST['action']) { case 'share': $return = OCP\Share::share($_POST['itemType'], $_POST['item'], $_POST['shareType'], $_POST['shareWith'], $_POST['permissions']); // TODO May need to return private link - ($return) ? OCP\JSON::success() : OCP\JSON::error(); + ($return) ? OC_JSON::success() : OC_JSON::error(); break; case 'unshare': $return = OCP\Share::unshare($_POST['itemType'], $_POST['item'], $_POST['shareType'], $_POST['shareWith']); - ($return) ? OCP\JSON::success() : OCP\JSON::error(); + ($return) ? OC_JSON::success() : OC_JSON::error(); break; case 'setTarget': $return = OCP\Share::setTarget($_POST['itemType'], $_POST['item'], $_POST['newTarget']); - ($return) ? OCP\JSON::success() : OCP\JSON::error(); + ($return) ? OC_JSON::success() : OC_JSON::error(); break; case 'setPermissions': $return = OCP\Share::setPermissions($_POST['itemType'], $_POST['item'], $_POST['shareType'], $_POST['shareWith'], $_POST['permissions']); - ($return) ? OCP\JSON::success() : OCP\JSON::error(); + ($return) ? OC_JSON::success() : OC_JSON::error(); break; } } else if (isset($_GET['fetch'])) { switch ($_GET['fetch']) { case 'getItemsSharedStatuses': $return = OCP\Share::getItemsSharedStatuses($_POST['itemType']); - ($return) ? OCP\JSON::success(array('data' => $return)) : OCP\JSON::error(); + ($return) ? OC_JSON::success(array('data' => $return)) : OC_JSON::error(); break; case 'getItemShared': $return = OCP\Share::getItemShared($_POST['itemType'], $_POST['item']); - ($return) ? OCP\JSON::success(array('data' => $return)) : OCP\JSON::error(); + ($return) ? OC_JSON::success(array('data' => $return)) : OC_JSON::error(); break; case 'getShareWith': // TODO Autocomplete for all users, groups, etc. diff --git a/core/js/share.js b/core/js/share.js index b7821ce14b..73a7ff86c2 100644 --- a/core/js/share.js +++ b/core/js/share.js @@ -21,20 +21,20 @@ OC.Share={ }); }, loadItem:function(itemType, item) { - $.get(OC.filePath('core', 'ajax', 'share.php'), { fetch: 'getItemShared', itemType: itemType, item: item }, async: false, function(result) { + $.get(OC.filePath('core', 'ajax', 'share.php'), { fetch: 'getItemShared', itemType: itemType, item: item }, function(result) { if (result && result.status === 'success') { OC.Share.item = result.data; } }); }, - share:function(itemType, shareType, shareWith, permissions, callback) { - $.post(OC.filePath('core', 'ajax', 'share.php'), { action: 'share', itemType: itemType, shareType: shareType, shareWith: shareWith, permissions: permissions }, function(result) { + share:function(itemType, item, shareType, shareWith, permissions, callback) { + $.post(OC.filePath('core', 'ajax', 'share.php'), { action: 'share', itemType: itemType, item: item, shareType: shareType, shareWith: shareWith, permissions: permissions }, function(result) { if (result && result.status === 'success') { if (callback) { callback(result.data); } } else { - OC.dialogs.alert(result.data.message, 'Error while sharing'); + OC.dialogs.alert('Error', 'Error while sharing'); } }); }, @@ -58,11 +58,9 @@ OC.Share={ }, showDropDown:function(itemType, item, appendTo) { OC.Share.loadItem(item); - var html = '