CDR Tickets

Issue Number 3649
Summary [Summaries] PDQ Board Invitation History Report - Problem with exclude current members options
Created 2013-08-21 14:32:15
Issue Type Bug
Submitted By Juthe, Robin (NIH/NCI) [E]
Assigned To Englisch, Volker (NIH/NCI) [C]
Status Closed
Resolved 2013-09-12 19:08:29
Resolution Fixed
Path /home/bkline/backups/jira/ocecdr/issue.112248
Description

The PDQ Board Invitation History Report has options to exclude current Editorial Board members and/or exclude current Advisory Board members from the report output. These options are working some of the time and not working some of the time. I think the problem is coming in when someone is a former member of one Board and a current member of another Board.

For example, Karen Lu recently left the Genetics EDITORIAL Board and joined the Genetics ADVISORY Board. If I run a report to see everyone who has been invited to the Editorial Board and I exclude both current Editorial Board members AND current Advisory Board members, then Karen should not display. But she does. Mark Robson is another example of a Board member who appears on this report despite being a current Advisory Board member (and former Ed Board member).

Comment entered 2013-09-03 16:26:49 by Englisch, Volker (NIH/NCI) [C]

I see that the code assumes that a board member is only in one board, either the editorial or the advisory board.
Is that correct or could a member be active on both boards?

Comment entered 2013-09-03 17:24:34 by Juthe, Robin (NIH/NCI) [E]

Yes and no. To explain...a Board member cannot be an active member of the Editorial and Advisory Genetic Boards simultaneously, but they can be a member of the Genetics Editorial Board and the Screening and Prevention Advisory Board.

Comment entered 2013-09-03 17:44:22 by Englisch, Volker (NIH/NCI) [C]

Oh boy, did you have to make it complicated? :-)

Comment entered 2013-09-04 10:10:16 by Englisch, Volker (NIH/NCI) [C]

With this additional wrinkle I have an additional question:
What exactly should the checkbox for 'Exclude current Ed board members' mean?

Lets say we have a member X who is active Genetics Adv. member, inactive Genetics Ed board member, and active treatment board member and we're running a report listing advisory board members.
Do we exclude this member X because he/she is active on any editorial board or do we list the member because he/she is not an active Genetics editorial board member?

Comment entered 2013-09-12 19:08:00 by Englisch, Volker (NIH/NCI) [C]

I think I am done with this issue. Could you please have a look on DEV (remember it has pretty old data) or give me a few of the special cases? I would like to do a little more testing but it's taking me some time to find the specific persons that would be affected by the changes and I'm guessing you already know the persons to look at.

If it would help I could move the report to QA (with has much more recent data).

Comment entered 2013-09-12 19:08:29 by Englisch, Volker (NIH/NCI) [C]

Ready for review on DEV.

Comment entered 2013-09-30 10:45:40 by Juthe, Robin (NIH/NCI) [E]

The option to Exclude Current Adv Board members seems to be excluding current Editorial and Advisory Board members.

For example,
I ran the report for the Genetics Editorial Board with neither "exclude" option selected. I saw some of my current Editorial Board members on the report (as expected). Example: Ilana Cass. I also saw some current Adv Board members (as expected). Example: Kathleen Blazer

I then ran the report for the Genetics Editorial Board with only the "exclude current Adv Board members" option selected. I no longer saw current Adv Board members (such as Kathleen Blazer) but I also no longer saw current Editorial Board members (such as Ilana Cass).

Comment entered 2013-09-30 17:05:30 by Englisch, Volker (NIH/NCI) [C]

It was meant as a feature so that you don't get overwhelmed with too many board members. :-)

Could you please take another look. Ilana Cass is listed again even with 'exclude advisory board' selected. I believe the other three options where already working properly.

Comment entered 2013-10-24 09:40:33 by Beckwith, Margaret (NIH/NCI) [E]

Robin, I tested this using the Adult Treatment Board, and I think it is working properly. If you have time, could you give it one more try so we can move this ahead? Thanks.

Comment entered 2013-10-25 07:55:58 by Juthe, Robin (NIH/NCI) [E]

I ran this for the Genetics Ed and Adv Boards and it looked good. Marking this QA verified.

Comment entered 2013-10-30 16:32:54 by Englisch, Volker (NIH/NCI) [C]

I've saved the changes to the report in subversion:

  • R12095: BoardInvitationHistory.py

Comment entered 2013-11-07 13:21:53 by Juthe, Robin (NIH/NCI) [E]

Verified on QA.

Comment entered 2013-12-03 13:25:11 by Juthe, Robin (NIH/NCI) [E]

Verified in production.

Elapsed: 0:00:00.001681