refactor: Use ReadView instead of ApplyView in authorizedDepositPreauth()

[tequdev]

High Level Overview of Change

Change the parameter type of credentials::authorizedDepositPreauth from ApplyView const& to ReadView const&, since the function only performs read operations on the view.

Context of Change

authorizedDepositPreauth only calls view.read() and view.exists(), which are ReadView methods. Using ApplyView was unnecessarily restrictive and misleading about the function's intent. This
change narrows the interface to the minimal required type.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.8%. Comparing base (013c2d6) to head (e9486d3).
⚠️ Report is 3 commits behind head on develop.

Additional details and impacted files

@@            Coverage Diff            @@
##           develop   #6560     +/-   ##
=========================================
- Coverage     79.8%   79.8%   -0.0%     
=========================================
  Files          878     878             
  Lines        68026   68026             
  Branches      7553    7555      +2     
=========================================
- Hits         54306   54300      -6     
- Misses       13720   13726      +6     

Files with missing lines	Coverage Δ
src/libxrpl/ledger/CredentialHelpers.cpp	97.6% <100.0%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[mvadari]

@tequdev is this PR ready to merge?

[tequdev]

@tequdev is this PR ready to merge?

yes