Fix batch summary to use the api

[eileenmcnaughton]

Overview

Fix batch summary to use the api & fix retrieve

Before

Old batch retrieve but when we try to update per #22543 we get a test error - this softens the update - but also avoids the function in the specific test fail
https://test.civicrm.org/job/CiviCRM-Core-PR/46718/testReport/junit/(root)/CRM_Financial_Page_AjaxBatchSummaryTest/testMakeBatchSummary/

After

No change

Technical Details

In looking at @colemanw's work to clean up the retrieve function it
turned out that in at least one place NULL is being passed to Batch::retrieve

Since I dug into that 1 place I fixed it to use v4 api (it is
the screen accessed after creating a new accounting batch
from contribution->accounting batches->new batch)

However, it seems safer to continue to accept NULL

Comments

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