Update Smart Group form unsets the existing Group Types

[jusfreeman]

Overview

Update Smart Group form unsets the existing Group Types, which will cause the Group Type values to be lost when Smart Group is updated.

The screenshot below, Group has the Mailing List group type set.

Before

When using the Update Smart Group form, it unsets the Mailing List group type.

After

With this PR, Mailing List group type remains set on the Update Smart Group form.

Agileware Ref: CIVICRM-1391

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

[seamuslee001]

(CiviCRM Review Template WORD-1.2)

• General standards
  • (r-explain) Pass
  • (r-user) Pass
  • (r-doc) Pass
  • (r-run) Pass replicated the issue on my local dev and confirmed that for non mailing smart groups that the mailing list box remains unchecked
• Developer standards
  • (r-tech) Pass
  • (r-code) Pass
  • (r-maint) Pass
  • (r-test) Pass