Manual direct debit is using the core method "transitionComponentWithReturnMessage" that is in the Contribution BAO class, though this method is now removed in core (see civicrm/civicrm-core#18961 for more info), and thus it result in failure when submitting the payment batch.

Here I replaced the calls for this method with "transitionComponents",
given that the former is basicly calling it but just return some extra
feedback messages, and that these returned feedback messages are not
even used or showed back to user in this extension, so it is safe to
just call "transitionComponents" instead.

PHP does not allow different method signature when override method from the parent class. As the sendEmail method in the child class requires different arguments, the method name has been changed to address the warning.

This abstract common class to decouple functions that duplicate in Contribution and Membership Email classes

The methods in parents class have been moved from the parent class to the traits. In order to use the functions in the traits which is not static, we need to change the context of method that calls the function in traits to have the same context.

Since the common code have been moved from common class to the traits in CiviCRM core, we need to refactor these classes to align with CiviCRM core.

MAE-493: Fix search button markup on manage screen

…ectdebit-notification

MAE-501: Fix unable to send direct debit notifications.

When using the API v3 `getvalue` action on any entity, it will return
the return value directly, but some places in direct debit importer
assumed that it would return the value indexed by the `result` key.

Here I am changing that so results from this API action are used directly.

MAE-494: Fix payment batch submission after CiviCRM 5.35 upgrade