use defined expiration var cacheExpiration when setting transient
#2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes
Currently, there is a protected class variable
cacheExpiration. This value is not used anywhere. It should be used when saving the release payload as transient. Instead the transient is set with the repeated value of 6 hours.This PR uses the value that is in place already as it appears to be intended for. So, now if someone sets the expiration it will actually be used.
Type of Change
Checklist
Further comments