[whimsy] branch master updated: Doh! Need to use different param name

[sebb]

This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new 8e92411b Doh! Need to use different param name
8e92411b is described below

commit 8e92411b8b9d1d8c2dc1c56d42bad3820b2e8cd3
Author: Sebb <s...@apache.org>
AuthorDate: Sat Apr 15 22:40:12 2023 +0100

    Doh! Need to use different param name
---
 www/roster/models/group.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/www/roster/models/group.rb b/www/roster/models/group.rb
index b4205a2b..8f5e21c9 100644
--- a/www/roster/models/group.rb
+++ b/www/roster/models/group.rb
@@ -31,13 +31,13 @@ class Group
   # The ids 'treasurer' and 'svnadmins' currently both have two definitions
   # treasurer: LDAP Auth Group and Pit Auth; svnadmins: LDAP Auth Group and 
LDAP service
   # So the type is now provided as a work-round
-  def self.serialize(id, type=nil)
+  def self.serialize(id, itype=nil)
     response = {}
 
     type = 'LDAP group'
     group = ASF::Group.find(id)
 
-    unless group.hasLDAP? or (%w{treasurer svnadmins}.include? id and type !~ 
%r{Auth Group}i)
+    unless group.hasLDAP? or (%w{treasurer svnadmins}.include?(id) and (itype 
!~ %r{Auth Group}i))
       type = 'LDAP auth group'
       group = ASF::AuthGroup.find(id)
     end