dev/core#183 Ensure that having a rule where each field is the same w…

[seamuslee001]

…eight with the threshold being the total sum of the weights does not cause a fatal error when searching for dupes

Overview

This fixes a fatal error that Richard @magnolia61 found as per comment on #15826. I think the main difference here is that no dupes will have been found with the first 2 rules i think but not sure however when I put Richard's rule into a unit test it failed and this fixed the unit test

Before

DB error already exist can occur with the given rule

After

No DB error and dupes can be found

ping @eileenmcnaughton @totten @mattwire @magnolia61

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[magnolia61]

Just tested this patch and indeed I do not run into fatals anymore. Thanks!

[seamuslee001]

I'm going to merge this based on @magnolia61 's review and given it fixes a possible fatal in the RC which is due out in only a couple of days