diff --git a/core/ajax/share.php b/core/ajax/share.php index 86ee018e38..abcb4b5c22 100644 --- a/core/ajax/share.php +++ b/core/ajax/share.php @@ -80,6 +80,14 @@ if (isset($_POST['action']) && isset($_POST['itemType']) && isset($_POST['itemSo break; case 'setExpirationDate': if (isset($_POST['date'])) { + $l = OC_L10N::get('core'); + $date = new \DateTime($_POST['date']); + $today = new \DateTime('now'); + + if ($date < $today) { + OC_JSON::error(array('data' => array('message' => $l->t('Expiration date is in the past.')))); + return; + } $return = OCP\Share::setExpirationDate($_POST['itemType'], $_POST['itemSource'], $_POST['date']); ($return) ? OC_JSON::success() : OC_JSON::error(); } diff --git a/core/js/share.js b/core/js/share.js index 0939259b7d..0b65e15309 100644 --- a/core/js/share.js +++ b/core/js/share.js @@ -720,7 +720,11 @@ $(document).ready(function() { var itemSource = $('#dropdown').data('item-source'); $.post(OC.filePath('core', 'ajax', 'share.php'), { action: 'setExpirationDate', itemType: itemType, itemSource: itemSource, date: $(this).val() }, function(result) { if (!result || result.status !== 'success') { - OC.dialogs.alert(t('core', 'Error setting expiration date'), t('core', 'Error')); + if (!result.data.message) { + OC.dialogs.alert(t('core', 'Error setting expiration date'), t('core', 'Error')); + } else { + OC.dialogs.alert(result.data.message, t('core', 'Error')); + } } }); });