Changeset 2577

Timestamp:

06/14/08 00:46:04 (20 months ago)

Author:

bchoate

Message:

Fixed javascript error for removing a role from a member. Fixed table listing highlight. BugId:80110

Location:

branches/release-40/tmpl/cms

Files:

• include/member_table.tmpl (modified) (1 diff)
• list_member.tmpl (modified) (3 diffs)

branches/release-40/tmpl/cms/include/member_table.tmpl

@@ -43 +43 @@
                 <mt:if name="filter_key" ne="commenter">
                 <td>
-                <mt:loop name="role_loop" glue=", "><span title="<mt:var name="role_perms" escape="html">" onmouseover="DOM.addClassName(this, 'focus')" onmouseout="DOM.removeClassName(this, 'focus')"><mt:var name="role_name"><mt:if name="can_administer_blog"> <a href="javascript:void(0)" onclick="removeMemberRole(<mt:var name="id">,<mt:var name="role_id">)" class="delete" title="<__trans phrase="Remove this role">">&nbsp;</a></mt:if></span></mt:loop></td>
+                <mt:loop name="role_loop" glue=", "><span title="<mt:var name="role_perms" escape="html">" onmouseover="DOM.addClassName(this, 'focus')" onmouseout="DOM.removeClassName(this, 'focus')"><mt:var name="role_name"><mt:if name="can_administer_blog"> <a href="javascript:void(0)" onclick="return removeMemberRole(<mt:var name="id">,<mt:var name="role_id">)" class="delete" title="<__trans phrase="Remove this role">">&nbsp;</a></mt:if></span></mt:loop></td>
                 </mt:if><mt:loop name="more_columns"><mt:var name="__value__"></mt:loop>
             </tr>

branches/release-40/tmpl/cms/list_member.tmpl

@@ -66 +66 @@
     {
         // setup
-        tableSelect = new TC.TableSelect( "author-listing-table" );
+        tableSelect = new TC.TableSelect( "user-listing-table" );
         tableSelect.rowSelect = true;
         setFilterCol(<mt:if name="filter">'<mt:var name="filter">'<mt:else>'none'</mt:if>);<mt:if name="filter">
@@ -75 +75 @@
     function removeMemberRole(author_id,role_id) {
         if (confirm("<__trans phrase="Are you sure you want to remove this role?" escape="js">")) {
-            var f = document.forms['author-listing-form'];
+            var f = document.forms['user-listing-form'];
             f['__mode'].value = "revoke_role";
             f['author_id'].value = author_id;
@@ -81 +81 @@
             f.submit();
         }
+        return false;
     }
     TC.attachLoadEvent( init );