Remove unreachable code

[eileenmcnaughton]

Overview

Remove unreachable code

Before

The last 2 conditions would only be true if the first is & it's an ELSE construct....

After

Technical Details

Since it's broken I'm just removing it - the code it DOES hit is kinda doing the same thing - since it does compare against option values valid for the field hence no-one noticed it....

Comments

[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).

[eileenmcnaughton]

@monishdeb are you able to merge this - I think it's a no brainer since the first if means it will never get to the last 2 elseifs

[monishdeb]

Agree, just to ensure I double-checked the code, and seems ok to remove the unreachable code. Hence merging.

[eileenmcnaughton]

thanks @monishdeb !