Fixing some conflicts
multiple comma support
Added support for writing the "to Field" or "from Field" information to the issue messages, in advance of possibly making this a single report. Also going to eventually make this so it itterates over a list and not doing these one by one.
This version adds logic for instances where there is a split role attny but there is no date value in the priv log, which I didnt initially think would ever happen but found instances where it does. Also adds support for situations where the name cant parse because there are multiple commas in the metadata name value.
in author, when resorting to a name search when the email doesnt match, because in this field the email address is not always in a paren, the email address was surviving and not removed. added an extra line to make sure it's removed before trying to parse the name.
Fixed a bug where it was writing more than one non match attempt to the issues. This was because of that extra else. This was incorrect and the else belonged at the very end of cycling through the different variations, not during. Putting this after all the variations have run but still no match fixed the issue.
Updated the variations function to force add AMAZON.COM if there is an @amazon. in the domain name because Eli said I could be confident that the same email name will show up as the amazon.co.uk, amazon.it, etc. This should increase my matches.
Fixed an annoying bug where, although I was checking to make sure the first name and last name both exist in the person I'm about to test when I need to match first name and last name only, I was looking at the full list of matching last names, so the report was wrong.
Copied, almost word for word, the first name last name matching piece under the email, when you cant match email, to the part where you werent even able to extract an email, which also needed to be first name last name matched.
Added the same (except for the text where it says low priority instead of high) split out for split role people to the email failed so lets match names section. this was copied from the email did match section. Really need to refactor this code since it has a lot of repeating code.
I had to place the put the for loop into the "if allPossibleVariationsList" because I was getting an issue where it was returning some none from the MAL but still using older variation variables. this way, and not referring to the variation variables in the matchFlag check works better. that was a terrible bug.
This version finishes the evaluation on the email address matches split role and reports on the results by adding an issue. Also added part at end of the email address match section to catch where I can match an email address in the metadata, the person is an attorney, but there is no matching value in the formatted field. This is a potential missing attorney and it a high confidence flag because I had the email address.
I'm still just printing the results of the split role date check but it's working correctly now. Next step will be to action on these decisions.
I'm adding some code to work out if a split role attorney was an attorney at the time of the doc date. I'm just getting it to print out here, so this is not yet complete.
With the additions to the issues list now a function, I simply removed the commented out old lines. Again tested it to be sure I didnt mess something up.
Consolidated all of the additions into the issuesMatrix into a fuction. commented out all the individual adds and tested using the function. results were identical except for a period in the text that was a typo anyway.
This program is similar to Amazon_PerformNamesNormQC but it will perform a deeper level of names norm QC. I may just replace Amazon_PerformNamesNormQC with this file but for now i'd like to keep both. This is the first working version of this program but with a lot still missing.
This form allows you to request diffs between any two revisions of this file. For each of the two "sides" of the diff, enter a numeric revision.