Skip to content

Commit

Permalink
use cache instead of database call
Browse files Browse the repository at this point in the history
  • Loading branch information
auumgn committed Sep 17, 2024
1 parent f8949b8 commit 1b96db4
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
package org.orcid.core.manager.v3.impl;


import org.orcid.core.manager.ProfileEntityCacheManager;
import org.orcid.core.manager.v3.ProfileEmailDomainManager;
import org.orcid.core.manager.v3.read_only.impl.ProfileEmailDomainManagerReadOnlyImpl;
import org.orcid.jaxb.model.v3.release.common.Visibility;
Expand Down Expand Up @@ -36,8 +37,8 @@ public class ProfileEmailDomainManagerImpl extends ProfileEmailDomainManagerRead
@Resource(name = "emailDaoReadOnly")
protected EmailDao emailDaoReadOnly;

@Resource(name = "profileDaoReadOnly")
protected ProfileDao profileDaoReadOnly;
@Resource
private ProfileEntityCacheManager profileEntityCacheManager;

@Transactional
public void updateEmailDomains(String orcid, org.orcid.pojo.ajaxForm.Emails newEmails) {
Expand Down Expand Up @@ -80,7 +81,7 @@ public void processDomain(String orcid, String email) {
// ADD NEW DOMAIN IF ONE DOESN'T EXIST
if (existingDomain == null) {
// Verify the user doesn't have more emails with that domain
ProfileEntity profile = profileDaoReadOnly.find(orcid);
ProfileEntity profile = profileEntityCacheManager.retrieve(orcid);
String domainVisibility = profile.getActivitiesVisibilityDefault();
profileEmailDomainDao.addEmailDomain(orcid, domain, domainVisibility);
}
Expand Down

0 comments on commit 1b96db4

Please sign in to comment.