#826 in progress

lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Main/Display.pm

@@ -471,7 +471,7 @@ sub mkOidcConsent {
     # TODO: add JS to remove consent
     if ( $session->{_oidcConnectedRP} ) {
         $tmp .=
-qq'<tr><td>$_ <a class="oidcConsent" partner="$_">remove</a></td></tr>'
+qq'<tr partner="$_"><td>$_ <a class="oidcConsent" partner="$_">remove</a></td></tr>'
           foreach ( split /,/, $session->{_oidcConnectedRP} );
         $tmp .= qq'</tbody></table><script type="application/init">
 {

lemonldap-ng-portal/site/coffee/portal.coffee

@@ -53,9 +53,9 @@ setOrder = ->
 
 removeOidcConsent = (partner) ->
 	r = new RegExp partner, 'g'
-	datas['oidcConsents'] = datas['oidcConsents'].replace(r,'').replace(',+',',')
+	datas['oidcConsents'] = datas['oidcConsents'].replace(r,'').replace(/,+/,',').replace(/^,/,'').replace(/,$/,'')
 	setKey '_oidcConnectedRP', datas['oidcConsents']
-	$(this).text()
+	$("tr[partner='#{partner}']").hide()
 
 setKey = (key,val) ->
 	$.ajax

lemonldap-ng-portal/site/htdocs/static/common/js/portal.js

@@ -77,9 +77,9 @@ LemonLDAP::NG Portal jQuery scripts
   removeOidcConsent = function(partner) {
     var r;
     r = new RegExp(partner, 'g');
-    datas['oidcConsents'] = datas['oidcConsents'].replace(r, '').replace(',+', ',');
+    datas['oidcConsents'] = datas['oidcConsents'].replace(r, '').replace(/,+/, ',').replace(/^,/, '').replace(/,$/, '');
     setKey('_oidcConnectedRP', datas['oidcConsents']);
-    return $(this).text();
+    return $("tr[partner='" + partner + "']").hide();
   };
 
   setKey = function(key, val) {