From 5e4ac548bb8eb752e8c05f260991b1dfb1fff9ab Mon Sep 17 00:00:00 2001 From: Joas Schilling <nickvergessen@gmx.de> Date: Mon, 30 Jun 2014 13:20:38 +0200 Subject: [PATCH] Use attr() instead of data() so the item ids work as intended Fix #8841 --- apps/files_sharing/js/share.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/files_sharing/js/share.js b/apps/files_sharing/js/share.js index 0999bfd6be..2efed5310b 100644 --- a/apps/files_sharing/js/share.js +++ b/apps/files_sharing/js/share.js @@ -107,7 +107,7 @@ var appendTo = $tr.find('td.filename'); // Check if drop down is already visible for a different file if (OC.Share.droppedDown) { - if ($tr.data('id') !== $('#dropdown').attr('data-item-source')) { + if ($tr.attr('data-id') !== $('#dropdown').attr('data-item-source')) { OC.Share.hideDropDown(function () { $tr.addClass('mouseOver'); OC.Share.showDropDown(itemType, $tr.data('id'), appendTo, true, possiblePermissions, filename); -- GitLab