diff options
-rw-r--r-- | apps/files/js/fileactions.js | 29 | ||||
-rw-r--r-- | apps/files/js/filelist.js | 20 | ||||
-rw-r--r-- | apps/files/templates/list.php | 4 | ||||
-rw-r--r-- | core/css/jquery.ocdialog.css | 3 | ||||
-rw-r--r-- | core/js/jquery.ocdialog.js | 5 | ||||
-rw-r--r-- | core/js/oc-dialogs.js | 101 |
6 files changed, 119 insertions, 43 deletions
diff --git a/apps/files/js/fileactions.js b/apps/files/js/fileactions.js index 527cbc0bd34..668960d5937 100644 --- a/apps/files/js/fileactions.js +++ b/apps/files/js/fileactions.js @@ -618,30 +618,21 @@ }); this.registerAction({ - name: 'Move', - displayName: t('files', 'Move'), + name: 'CopyMove', + displayName: t('files', 'Copy or Move'), mime: 'all', order: -25, permissions: OC.PERMISSION_UPDATE, iconClass: 'icon-external', actionHandler: function (filename, context) { - OC.dialogs.filepicker(t('files', 'Target folder'), function(targetPath) { - context.fileList.move(filename, targetPath); - }, false, "httpd/unix-directory", true); - } - }); - - this.registerAction({ - name: 'Copy', - displayName: t('files', 'Copy'), - mime: 'all', - order: -25, - permissions: OC.PERMISSION_READ, - iconClass: 'icon-external', - actionHandler: function (filename, context) { - OC.dialogs.filepicker(t('files', 'Target folder'), function(targetPath) { - context.fileList.copy(filename, targetPath); - }, false, "httpd/unix-directory", true); + OC.dialogs.filepicker(t('files', 'Target folder'), function(targetPath, type) { + if (type === OC.dialogs.FILEPICKER_TYPE_COPY) { + context.fileList.copy(filename, targetPath); + } + if (type === OC.dialogs.FILEPICKER_TYPE_MOVE) { + context.fileList.move(filename, targetPath); + } + }, false, "httpd/unix-directory", true, OC.dialogs.FILEPICKER_TYPE_COPY_MOVE); } }); diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js index 946ac125484..d67cb409b1e 100644 --- a/apps/files/js/filelist.js +++ b/apps/files/js/filelist.js @@ -337,11 +337,7 @@ this.$el.on('urlChanged', _.bind(this._onUrlChanged, this)); this.$el.find('.select-all').click(_.bind(this._onClickSelectAll, this)); this.$el.find('.download').click(_.bind(this._onClickDownloadSelected, this)); -<<<<<<< HEAD - this.$el.find('.move').click(_.bind(this._onClickMoveSelected, this)); -======= - this.$el.find('.copy').click(_.bind(this._onClickCopySelected, this)); ->>>>>>> Allow files to be copied through action menu & multiple files actions + this.$el.find('.copy-move').click(_.bind(this._onClickCopyMoveSelected, this)); this.$el.find('.delete-selected').click(_.bind(this._onClickDeleteSelected, this)); this.$el.find('.selectedActions a').tooltip({placement:'top'}); @@ -765,7 +761,7 @@ /** * Event handler for when clicking on "Move" for the selected files */ - _onClickMoveSelected: function(event) { + _onClickCopyMoveSelected: function(event) { var files; var self = this; @@ -783,10 +779,14 @@ OCA.Files.FileActions.updateFileActionSpinner(moveFileAction, false); }; - OCA.Files.FileActions.updateFileActionSpinner(moveFileAction, true); - OC.dialogs.filepicker(t('files', 'Target folder'), function(targetPath) { - self.move(files, targetPath, disableLoadingState); - }, false, "httpd/unix-directory", true); + OC.dialogs.filepicker(t('files', 'Target folder'), function(targetPath, type) { + if (type === OC.dialogs.FILEPICKER_TYPE_COPY) { + self.copy(files, targetPath, disableLoadingState); + } + if (type === OC.dialogs.FILEPICKER_TYPE_MOVE) { + self.move(files, targetPath, disableLoadingState); + } + }, false, "httpd/unix-directory", true, OC.dialogs.FILEPICKER_TYPE_COPY_MOVE); return false; }, diff --git a/apps/files/templates/list.php b/apps/files/templates/list.php index 67c330c38c7..3857ad3badd 100644 --- a/apps/files/templates/list.php +++ b/apps/files/templates/list.php @@ -47,9 +47,9 @@ </label> <a class="name sort columntitle" data-sort="name"><span><?php p($l->t( 'Name' )); ?></span><span class="sort-indicator"></span></a> <span id="selectedActionsList" class="selectedActions"> - <a href="" class="move"> + <a href="" class="copy-move"> <span class="icon icon-external"></span> - <span><?php p($l->t('Move'))?></span> + <span><?php p($l->t('Copy or Move'))?></span> </a> <a href="" class="download"> <span class="icon icon-download"></span> diff --git a/core/css/jquery.ocdialog.css b/core/css/jquery.ocdialog.css index 487bc1c4f69..2100a3db7a6 100644 --- a/core/css/jquery.ocdialog.css +++ b/core/css/jquery.ocdialog.css @@ -38,6 +38,9 @@ .oc-dialog-buttonrow.twobuttons button:nth-child(1) { float: left; } +.oc-dialog-buttonrow.twobuttons.aside button:nth-child(1) { + float: none; +} .oc-dialog-buttonrow.twobuttons button:nth-child(2) { float: right; } diff --git a/core/js/jquery.ocdialog.js b/core/js/jquery.ocdialog.js index b54cce2c0ca..555b35e59ff 100644 --- a/core/js/jquery.ocdialog.js +++ b/core/js/jquery.ocdialog.js @@ -130,6 +130,11 @@ }); this._setSizes(); break; + case 'style': + if (value.buttons !== undefined) { + this.$buttonrow.addClass(value.buttons); + } + break; case 'closeButton': if(value) { var $closeButton = $('<a class="oc-dialog-close"></a>'); diff --git a/core/js/oc-dialogs.js b/core/js/oc-dialogs.js index 5fc224e38bf..f81ead06216 100644 --- a/core/js/oc-dialogs.js +++ b/core/js/oc-dialogs.js @@ -29,6 +29,12 @@ var OCdialogs = { // dialog button types YES_NO_BUTTONS: 70, OK_BUTTONS: 71, + + FILEPICKER_TYPE_CHOOSE: 1, + FILEPICKER_TYPE_MOVE: 2, + FILEPICKER_TYPE_COPY: 3, + FILEPICKER_TYPE_COPY_MOVE: 4, + // used to name each dialog dialogsCounter: 0, /** @@ -174,13 +180,19 @@ var OCdialogs = { * @param multiselect whether it should be possible to select multiple files * @param mimetypeFilter mimetype to filter by - directories will always be included * @param modal make the dialog modal + * @param type Type of file picker : Choose, copy, move, copy and move */ - filepicker:function(title, callback, multiselect, mimetypeFilter, modal) { + filepicker:function(title, callback, multiselect, mimetypeFilter, modal, type) { var self = this; // avoid opening the picker twice if (this.filepicker.loading) { return; } + + if (type === undefined) { + type = this.FILEPICKER_TYPE_CHOOSE; + } + this.filepicker.loading = true; this.filepicker.filesClient = (OCA.Sharing && OCA.Sharing.PublicApp && OCA.Sharing.PublicApp.fileList)? OCA.Sharing.PublicApp.fileList.filesClient: OC.Files.getClient(); $.when(this._getFilePickerTemplate()).then(function($tmpl) { @@ -210,15 +222,17 @@ var OCdialogs = { self.$filePicker.ready(function() { self.$filelist = self.$filePicker.find('.filelist tbody'); self.$dirTree = self.$filePicker.find('.dirtree'); - self.$dirTree.on('click', 'div:not(:last-child)', self, self._handleTreeListSelect.bind(self)); + self.$dirTree.on('click', 'div:not(:last-child)', self, function (event) { + self._handleTreeListSelect(event, type); + }); self.$filelist.on('click', 'tr', function(event) { - self._handlePickerClick(event, $(this)); + self._handlePickerClick(event, $(this), type); }); self._fillFilePicker(''); }); // build buttons - var functionToCall = function() { + var functionToCall = function(returnType) { if (callback !== undefined) { var datapath; if (multiselect === true) { @@ -233,15 +247,47 @@ var OCdialogs = { datapath += '/' + selectedName; } } - callback(datapath); + callback(datapath, returnType); self.$filePicker.ocdialog('close'); } }; - var buttonlist = [{ - text: t('core', 'Choose'), - click: functionToCall, - defaultButton: true - }]; + + var chooseCallback = function () { + functionToCall(OCdialogs.FILEPICKER_TYPE_CHOOSE); + }; + + var copyCallback = function () { + console.log('copy callback'); + functionToCall(OCdialogs.FILEPICKER_TYPE_COPY); + }; + + var moveCallback = function () { + functionToCall(OCdialogs.FILEPICKER_TYPE_MOVE); + }; + + var buttonlist = []; + if (type === OCdialogs.FILEPICKER_TYPE_CHOOSE) { + buttonlist.push({ + text: t('core', 'Choose'), + click: chooseCallback, + defaultButton: true + }); + } else { + if (type === OCdialogs.FILEPICKER_TYPE_COPY || type === OCdialogs.FILEPICKER_TYPE_COPY_MOVE) { + buttonlist.push({ + text: t('core', 'Copy'), + click: copyCallback, + defaultButton: false + }); + } + if (type === OCdialogs.FILEPICKER_TYPE_MOVE || type === OCdialogs.FILEPICKER_TYPE_COPY_MOVE) { + buttonlist.push({ + text: t('core', 'Move'), + click: moveCallback, + defaultButton: true + }); + } + } self.$filePicker.ocdialog({ closeOnEscape: true, @@ -250,6 +296,9 @@ var OCdialogs = { height: 500, modal: modal, buttons: buttonlist, + style: { + buttons: 'aside', + }, close: function() { try { $(this).ocdialog('destroy').remove(); @@ -879,12 +928,13 @@ var OCdialogs = { /** * handle selection made in the tree list */ - _handleTreeListSelect:function(event) { + _handleTreeListSelect:function(event, type) { var self = event.data; var dir = $(event.target).parent().data('dir'); self._fillFilePicker(dir); var getOcDialog = (event.target).closest('.oc-dialog'); var buttonEnableDisable = $('.primary', getOcDialog); + this._changeButtonsText(type, dir.split(/[/]+/).pop()); if (this.$filePicker.data('mimetype') === "httpd/unix-directory") { buttonEnableDisable.prop("disabled", false); } else { @@ -894,7 +944,7 @@ var OCdialogs = { /** * handle clicks made in the filepicker */ - _handlePickerClick:function(event, $element) { + _handlePickerClick:function(event, $element, type) { var getOcDialog = this.$filePicker.closest('.oc-dialog'); var buttonEnableDisable = getOcDialog.find('.primary'); if ($element.data('type') === 'file') { @@ -905,11 +955,38 @@ var OCdialogs = { buttonEnableDisable.prop("disabled", false); } else if ( $element.data('type') === 'dir' ) { this._fillFilePicker(this.$filePicker.data('path') + '/' + $element.data('entryname')); + this._changeButtonsText(type, $element.data('entryname')); if (this.$filePicker.data('mimetype') === "httpd/unix-directory") { buttonEnableDisable.prop("disabled", false); } else { buttonEnableDisable.prop("disabled", true); } } + }, + + /** + * Handle + * @param type of action + * @param dir on which to change buttons text + * @private + */ + _changeButtonsText(type, dir) { + var copyText = dir === '' ? t('core', 'Copy') : t('core', 'Copy to {folder}', {folder: dir}); + var moveText = dir === '' ? t('core', 'Move') : t('core', 'Move to {folder}', {folder: dir}); + var buttons = $('.oc-dialog-buttonrow button'); + switch (type) { + case this.FILEPICKER_TYPE_CHOOSE: + break; + case this.FILEPICKER_TYPE_COPY: + buttons.text(copyText); + break; + case this.FILEPICKER_TYPE_MOVE: + buttons.text(moveText); + break; + case this.FILEPICKER_TYPE_COPY_MOVE: + buttons.eq(0).text(copyText); + buttons.eq(1).text(moveText); + break; + } } }; |