Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

oxauth code should use new cacheService.getWithPut() method #1150

Closed
yuriyz opened this issue Sep 11, 2019 · 1 comment
Closed

oxauth code should use new cacheService.getWithPut() method #1150

yuriyz opened this issue Sep 11, 2019 · 1 comment
Assignees
@yuriyz
Labels
enhancement libs update, re-factroring, etc.
Projects
CE 4.2.0
Milestone
4.2

Comments

@yuriyz
Copy link
Contributor

yuriyz commented Sep 11, 2019 •
edited

Describe the issue

oxauth code should use new cacheService.getWithPut() method to avoid redundant requests to persistence.
@yuriyz yuriyz added the enhancement libs update, re-factroring, etc. label Sep 11, 2019
@yuriyz yuriyz added this to the 4.0 milestone Sep 11, 2019
@yuriyz yuriyz self-assigned this Sep 11, 2019
@yuriyz yuriyz modified the milestones: 4.0, 4.1 Sep 11, 2019
@yuriyz yuriyz mentioned this issue Sep 11, 2019
Closed
@yuriyz yuriyz modified the milestones: 4.1, 4.2 Dec 12, 2019
@yurem yurem added this to In progress in CE 4.2.0 Jan 3, 2020
@yurem yurem moved this from In progress to To do in CE 4.2.0 Jan 3, 2020
yuriyz added a commit that referenced this issue May 5, 2020
@yuriyz
Used new cacheService.getWithPut() method where possible. Removed r…
0b00f6b 
…edundant region specification and removed redundant string->integer convertion.

#1150
@yuriyz
Copy link
Contributor Author

yuriyz commented May 12, 2020

Done in 4.2

@yuriyz yuriyz closed this as completed May 12, 2020
CE 4.2.0 automation moved this from To do to Done May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yuriyz yuriyz

Labels
enhancement libs update, re-factroring, etc.
Projects
No open projects
CE 4.2.0
  
Done
Milestone
4.2
Development

No branches or pull requests
1 participant
@yuriyz