silverstripe / silverstripe-assets

Silverstripe Assets component
BSD 3-Clause "New" or "Revised" License
9 stars 65 forks source link

Rescue Master Branch PR: ENH Stop using old deprecated `allVersions` call #507

Closed GuySartorelli closed 1 year ago

GuySartorelli commented 1 year ago

Rescues https://github.com/silverstripe/silverstripe-assets/commit/f4259a148fa3f6139f85853d7af6d5f06e766e23

Targetting 1 because allVersions() is already deprecated and we don't want to be throwing deprecation warnings for core usage.

I only kept the actual change of method used - the other changes from the original commit caused failures in tests.

Parent issue

GuySartorelli commented 1 year ago

Please read the PR description which explains the difference between this PR and the original which is rescued.