[jira] [Commented] (WHIMSY-190) Agenda.parse of older attendee lists inconsistent

["Shane Curcuru (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/WHIMSY-190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16424111#comment-16424111 ] 

Shane Curcuru commented on WHIMSY-190:
--------------------------------------

Note that in any case, we need to remove all () parens anywhere from names before calling LDAP, since that will throw an error:

 
ASF::Person.list("cn=#\{name}")

> Agenda.parse of older attendee lists inconsistent
> -------------------------------------------------
>
>                 Key: WHIMSY-190
>                 URL: https://issues.apache.org/jira/browse/WHIMSY-190
>             Project: Whimsy
>          Issue Type: Bug
>          Components: BoardAgenda
>            Reporter: Shane Curcuru
>            Priority: Minor
>
> Parsing older board meeting minutes 2007-2009 gives inconsistent results for some ill-formatted attendee/regrets names.  In particular, one month throws an LDAP error instead of parsing at all using tools/agenda_summary.rb:
>  
> "board_agenda_2009_08_19": {
>  "errors": "ERROR(board_agenda_2009_08_19) Agenda parse error: no result returned by search /Users/curcuru/src/g/whimsy/lib/whimsy/asf/ldap.rb:337:in `search2'"
>  },
>  
> Proposed improvements to lib/whimsy/asf/agenda/front.rb:
>  
> {code:java}
> # attempt to identify the people mentioned in the Roll Call
> people = attr['text'].scan(/ {8}(\w.*)/).flatten.each do |name|
>   next if name == 'none'
>   name.gsub! /\s*\(.*?\)/, ''
> # Replace above to remove everything following just an open parens, inclusive
> # Add removal of [ square brackets inclusive as well
> # Add .strip to remove extraneous whitespace
> {code}
> Problematic lines in past attendance (examples):
> J. Aaron Farr  [will join the call late] 
> Geir Magnusson Jr.  (starting a small
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)