[whimsy] branch master updated: Oops, applied escape to the wrong thing

[se...@apache.org]

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 77e9556  Oops, applied escape to the wrong thing
77e9556 is described below

commit 77e9556672115c0879072f8f184ecae485c22cfb
Author: Sebb <se...@apache.org>
AuthorDate: Mon Feb 22 12:09:50 2021 +0000

    Oops, applied escape to the wrong thing
---
 www/members/nominations.cgi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/www/members/nominations.cgi b/www/members/nominations.cgi
index 1d13603..1ec1b3f 100755
--- a/www/members/nominations.cgi
+++ b/www/members/nominations.cgi
@@ -91,7 +91,7 @@ _html do
           _ul nominations.sort_by {|nominee| nominee[:name]} do |nominee|
             _li! do
               person = ASF::Person.find(nominee[:id])
-              match = /\b(#{Regexp.escape(nominee[:name])}|#{Regexp.escape(person.public_name)})\b/i
+              match = /\b(#{Regexp.escape(nominee[:name]|person.public_name)})\b/i
 
               if emails.any? {|mail| mail.subject.downcase =~ match}
                 _a.present person.public_name, href: "#{ROSTER}/#{nominee[:id]}"