diff --git a/core/js/share.js b/core/js/share.js index 692ce0b0ba..2370916e8b 100644 --- a/core/js/share.js +++ b/core/js/share.js @@ -666,7 +666,7 @@ OC.Share={ html += ''; if (oc_config.enable_avatars === true) { if (shareType === OC.Share.SHARE_TYPE_USER) { - html += '
'; + html += ''; } else { html += ''; } @@ -703,7 +703,7 @@ OC.Share={ html += ''; html = $(html).appendTo('#shareWithList'); if (oc_config.enable_avatars === true && shareType === OC.Share.SHARE_TYPE_USER) { - $('#avatar-' + escapeHTML(shareWith)).avatar(escapeHTML(shareWith), 32); + $('.avatar[data-user="' + escapeHTML(shareWith) + '"]').avatar(escapeHTML(shareWith), 32); } // insert cruds button into last label element var lastLabel = html.find('>label:last'); diff --git a/core/js/tests/specs/shareSpec.js b/core/js/tests/specs/shareSpec.js index 1856fc27bc..bcdc1df3d3 100644 --- a/core/js/tests/specs/shareSpec.js +++ b/core/js/tests/specs/shareSpec.js @@ -459,7 +459,7 @@ describe('OC.Share tests', function() { expect($('#shareWithList').children().length).toEqual(2); - expect($('#avatar-user1').length).toEqual(1); + expect($('.avatar[data-user="user1"]').length).toEqual(1); expect(args.length).toEqual(2); expect(args[0]).toEqual('user1'); });