Remove reference to undefined variable in CRM_Contact_Form_Inline_CommunicationPreferences

[braders]

Overview

In #20214 a change was introduced which means the inline communication form references a missing variable. The biggest impact of that is that your PHP log is likely to end up with something like this:

PHP Notice:  Trying to get property '_action' of non-object in ..

The if statement could never be true, and setting a default on an edit form (as opposed to a create form) doesn't make much sense. Therefore I propose this block of code is removed.

Before

PHP notice.

After

No PHP notice.

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

[demeritcowboy]

This seems ok. Noting also that getContactDefaultLanguage never returns false so that inner if wouldn't quite work as intended if it even got there.

Separate: If on a new contact you clear the field intending to leave it blank, it seems to get filled in with a default anyway, but that happens before the patch too. I'm not really sure what blank would mean - any later code that sees blank would likely just use the site default.