Clean up another cache usage to use metadata

[eileenmcnaughton]

Overview

Clean up another cache usage to use metadata

Before

Function uses its own special version of a cache with a php static front end

After

Cache uses 'metadata' cache which has the same config & it otherwise used for option values

Technical Details

This is just standardisation - I didn't identify it during performance testing - but ideally we would have a consistent pattern. I think I might hit some of those fresh param calls in the first test run though

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

[seamuslee001]

This seems ok to me