Skip to content

Commit df664ed

Browse files
authored
Merge pull request #36564 from nextcloud/artonge/fix/restore_copy_button_text_on_error
Restore copy button text on directory fetch error
2 parents 6236235 + a547aff commit df664ed

File tree

5 files changed

+13
-12
lines changed

5 files changed

+13
-12
lines changed

core/src/OC/dialogs.js

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -427,7 +427,7 @@ const Dialogs = {
427427
if (checkInput()) {
428428
var newname = $input.val()
429429
self.filepicker.filesClient.createDirectory(self.$filePicker.data('path') + "/" + newname).always(function (status) {
430-
self._fillFilePicker(self.$filePicker.data('path') + "/" + newname)
430+
self._fillFilePicker(self.$filePicker.data('path') + "/" + newname, type)
431431
})
432432
OC.hideMenus()
433433
self.$filePicker.ocdialog('unsetEnterCallback')
@@ -459,10 +459,10 @@ const Dialogs = {
459459
var dir = self.$filePicker.data('path')
460460
self.filepicker.sortField = $(event.currentTarget).data('sort')
461461
self.filepicker.sortOrder = self.filepicker.sortOrder === 'asc' ? 'desc' : 'asc'
462-
self._fillFilePicker(dir)
462+
self._fillFilePicker(dir, type)
463463
}
464464
})
465-
self._fillFilePicker(path)
465+
self._fillFilePicker(path, type)
466466
})
467467

468468
// build buttons
@@ -1120,7 +1120,7 @@ const Dialogs = {
11201120
/**
11211121
* fills the filepicker with files
11221122
*/
1123-
_fillFilePicker: async function(dir) {
1123+
_fillFilePicker: async function(dir, type) {
11241124
var self = this
11251125
this.$filelist.empty()
11261126
this.$filePicker.find('.emptycontent').hide()
@@ -1155,6 +1155,7 @@ const Dialogs = {
11551155
console.error('Requested path does not exists, falling back to root')
11561156
var files = await getFolderContents('/')
11571157
this.$filePicker.data('path', '/')
1158+
this._changeButtonsText(type, '')
11581159
}
11591160

11601161
self.filelist = files
@@ -1320,7 +1321,7 @@ const Dialogs = {
13201321
_handleTreeListSelect: function(event, type) {
13211322
var self = event.data
13221323
var dir = $(event.target).closest('.crumb').data('dir')
1323-
self._fillFilePicker(dir)
1324+
self._fillFilePicker(dir, type)
13241325
var getOcDialog = (event.target).closest('.oc-dialog')
13251326
var buttonEnableDisable = $('.primary', getOcDialog)
13261327
this._changeButtonsText(type, dir.split(/[/]+/).pop())
@@ -1343,7 +1344,7 @@ const Dialogs = {
13431344
$element.toggleClass('filepicker_element_selected')
13441345
buttonEnableDisable.prop('disabled', false)
13451346
} else if ($element.data('type') === 'dir') {
1346-
this._fillFilePicker(this.$filePicker.data('path') + '/' + $element.data('entryname'))
1347+
this._fillFilePicker(this.$filePicker.data('path') + '/' + $element.data('entryname'), type)
13471348
this._changeButtonsText(type, $element.data('entryname'))
13481349
if (this.$filePicker.data('mimetype').indexOf('httpd/unix-directory') !== -1 || this.$filePicker.data('allowDirectoryChooser')) {
13491350
buttonEnableDisable.prop('disabled', false)

dist/core-login.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/core-login.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/core-main.js

Lines changed: 2 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

dist/core-main.js.map

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)