Following PR-25994 and PR-26074

[samuelsov]

Overview

In searchkit, deleting a group parents inline doesn't work.

Finishing the work started in #25994 and #26074

Before

After

It works.

Technical Details

Move the deletion logic from the group form edit to the CRM_Contact_BAO_Group::create function

Not sure if we can assume that if we don't remove a parent, there is no need for cache refresh (assuming no for now) :

      // this is always required, since we don't know when a
      // parent group is removed
      CRM_Contact_BAO_GroupNestingCache::update();

[civibot]

(Standard links)

• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[colemanw]

Ok, this looks pretty good now. The BAO is still weird and nonstandard but this is certainly an improvement.

Not sure if we can assume that if we don't remove a parent, there is no need for cache refresh (assuming no for now)

I think the answer is yes, now that both adding and removing are controlled by the same function, we can safely skip cache refreshing if neither operation was done. Feel free to add that to this PR or as a followup after this is merged.