FINERACT-2652: Decouple SavingsAccount from injected collaborators

[alberto-art3ch]

Description

We are removing @transient service fields from SavingsAccount.

The transient SavingsAccountTransactionSummaryWrapper, SavingsHelper and ConfigurationDomainService (previously injected post-hydration via setHelpers) and the accountNumberRequiresAutoGeneration flag are removed. The three stateless collaborators are bundled into a new singleton SavingsContext bean that is passed as a method parameter; the account-number flag is derived at the service layer.

FINERACT-2652

Checklist

Please make sure these boxes are checked before submitting your pull request - thanks!

• Write the commit message as per our guidelines
• Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.
• Create/update unit or integration tests for verifying the changes made.
• Follow our coding conventions.
• Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/legacy-docs/apiLive.htm with details of any API changes
• This PR must not be a "code dump". Large changes can be made in a branch, with assistance. Ask for help on the developer mailing list.

Your assigned reviewer(s) will follow our guidelines for code reviews.

[adamsaghy]

Not much better if we are still injecting (not directly, but via arguments) services into the JPA...

[alberto-art3ch]

It has already been updated and a more robust change has been made for the extraction of business methods from Savings and Deposit entities.

[adamsaghy]

I am having concerns. It's better than directly injecting services and helpers into the JPA, but still not clean solution:

• Injecting context which contains helpers / services which are called inside the JPA entity still

Would it be possible to truly decouple and extract all these logic out of the JPA entity?

[adamsaghy]

> Task :fineract-command-async:licenseJmh NO-SOURCE
        switch (this.accountTermAndPreClosure.depositPeriodFrequencyType()) {
        ^
    (see https://errorprone.info/bugpattern/StatementSwitchToExpressionSwitch)
  Did you mean 'switch (this.accountTermAndPreClosure.depositPeriodFrequencyType()) {'?
/home/runner/work/fineract/fineract/fineract-savings/src/main/java/org/apache/fineract/portfolio/savings/domain/FixedDepositAccount.java:869: error: [MissingOverride] setClosedOnDate overrides method in SavingsAccount; expected @Override
    public void setClosedOnDate(final LocalDate closedOnDate) {
                ^
        ^
    (see https://errorprone.info/bugpattern/StatementSwitchToExpressionSwitch)
Unknown file extension: fineract-command-async/src/main/resources/META-INF/spring/org.springframework.boot.autoconfigure.AutoConfiguration.imports
  Did you mean 'switch (this.accountTermAndPreClosure.depositPeriodFrequencyType()) {'?
> Task :fineract-command-async:licenseMain UP-TO-DATE
> Task :fineract-command-async:licenseTest UP-TO-DATE
> Task :fineract-command-async:license UP-TO-DATE
/home/runner/work/fineract/fineract/fineract-savings/src/main/java/org/apache/fineract/portfolio/savings/domain/RecurringDepositAccount.java:1116: error: [MissingOverride] setClosedOnDate overrides method in SavingsAccount; expected @Override
    public void setClosedOnDate(final LocalDate closedOnDate) {