From 92b5517229e4d56fc64b91898c9950bd670f893e Mon Sep 17 00:00:00 2001
From: Lukas Reschke <lukas@owncloud.com>
Date: Fri, 27 Feb 2015 17:38:59 +0100
Subject: Use custom attribute instead of the div identifier

Otherwise problems arrive when a username contains characters such as an `@`

Fixes https://github.com/owncloud/user_shibboleth/issues/38
---
 core/js/share.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'core/js/share.js')

diff --git a/core/js/share.js b/core/js/share.js
index 692ce0b0ba0..2370916e8be 100644
--- a/core/js/share.js
+++ b/core/js/share.js
@@ -666,7 +666,7 @@ OC.Share={
 			html += '<a href="#" class="unshare"><img class="svg" alt="'+t('core', 'Unshare')+'" title="'+t('core', 'Unshare')+'" src="'+OC.imagePath('core', 'actions/delete')+'"/></a>';
 			if (oc_config.enable_avatars === true) {
 				if (shareType === OC.Share.SHARE_TYPE_USER) {
-					html += '<div id="avatar-' + escapeHTML(shareWith) + '" class="avatar"></div>';
+					html += '<div data-user="' + escapeHTML(shareWith) + '" class="avatar"></div>';
 				} else {
 					html += '<div class="avatar" style="padding-right: 32px"></div>';
 				}
@@ -703,7 +703,7 @@ OC.Share={
 			html += '</li>';
 			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');
-- 
cgit v1.2.3