APIv4 - Set 'activity_type_id' to required

[colemanw]

Overview

In APIv4 GetFields, sets activity_type_id because it is in fact needed to create an activity.

Before

Field not required, but should be.

After

Required.

Technical Details

The database default '1' is problematic as the option list is user-configurable, so activity type '1' doesn't necessarily exist.

[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]

Why on earth would THIS be the field where we use a db default rather than required. Could make the case for a schema change here!