diff options
| author | Vincent Petry <pvince81@owncloud.com> | 2013-10-21 11:29:40 +0200 |
|---|---|---|
| committer | Vincent Petry <pvince81@owncloud.com> | 2013-10-21 11:29:40 +0200 |
| commit | f8717bc26005912f14fb0da52b7dd66f86f3b929 (patch) | |
| tree | 60180a1d55f28eb05f76a65eb8e8f96083a12a1c /core | |
| parent | f5a6d6b43e6a91d0e9ef0940e1a83530f5a5cff6 (diff) | |
| download | nextcloud-server-f8717bc26005912f14fb0da52b7dd66f86f3b929.tar.gz nextcloud-server-f8717bc26005912f14fb0da52b7dd66f86f3b929.zip | |
Fixed share with link checkbox missing as regular user
Instead of loading the app config setting "shareapi_allow_links" using a
synchronous ajax call that fails when the user is not an admin, this fix
puts the flag directly in the template so it doesn't need to be loaded
afterwards.
Fixes #5440
Diffstat (limited to 'core')
| -rw-r--r-- | core/js/share.js | 13 |
1 files changed, 1 insertions, 12 deletions
diff --git a/core/js/share.js b/core/js/share.js index 6ec15f01c1e..9c606b9de62 100644 --- a/core/js/share.js +++ b/core/js/share.js @@ -203,18 +203,7 @@ OC.Share={ html += '<input id="shareWith" type="text" placeholder="'+t('core', 'Share with')+'" />'; html += '<ul id="shareWithList">'; html += '</ul>'; - var linksAllowed = false; - $.ajax({ - type: 'GET', - url: OC.filePath('core', 'ajax', 'appconfig.php'), - data: { action:'getValue', app:'core', key:'shareapi_allow_links', defaultValue:'yes' }, - async: false, - success: function(result) { - if (result && result.status === 'success' && result.data === 'yes') { - linksAllowed = true; - } - } - }); + var linksAllowed = $('#allowShareWithLink').val() === 'yes'; if (link && linksAllowed) { html += '<div id="link">'; html += '<input type="checkbox" name="linkCheckbox" id="linkCheckbox" value="1" /><label for="linkCheckbox">'+t('core', 'Share with link')+'</label>'; |