Fix compilation issues

ORCID · Nov 16, 2023 · 963a781 · 963a781
1 parent 60faa27
commit 963a781
Show file tree

Hide file tree

Showing 6 changed files with 6 additions and 6 deletions.
diff --git a/orcid-web/src/main/java/org/orcid/frontend/oauth2/OauthController.java b/orcid-web/src/main/java/org/orcid/frontend/oauth2/OauthController.java
@@ -97,7 +97,7 @@ public class OauthController {
             if (responseParam != null && !responseParam.isEmpty() && !PojoUtil.isEmpty(responseParam.get(0))) {
                 isResponseSet = true;
                 if (Features.EVENTS.isActive()) {
-                    eventManager.createEvent(requestInfoForm.getUserOrcid(), EventType.REAUTHORIZE, request);
+                    eventManager.createEvent(EventType.REAUTHORIZE, request);
                 }
             }
         }

diff --git a/orcid-web/src/main/java/org/orcid/frontend/spring/AjaxAuthenticationSuccessHandler.java b/orcid-web/src/main/java/org/orcid/frontend/spring/AjaxAuthenticationSuccessHandler.java
@@ -26,7 +26,7 @@ public class AjaxAuthenticationSuccessHandler extends AjaxAuthenticationSuccessH
     public void onAuthenticationSuccess(HttpServletRequest request, HttpServletResponse response, Authentication authentication) throws IOException, ServletException {
     	String targetUrl = getTargetUrl(request, response, authentication);
         if (Features.EVENTS.isActive()) {
-            eventManager.createEvent(authentication.getPrincipal().toString(), EventType.SIGN_IN, request);
+            eventManager.createEvent(EventType.SIGN_IN, request);
         }
         response.setContentType("application/json");
         response.getWriter().println("{\"success\": true, \"url\": \"" + targetUrl.replaceAll("^/", "") + "\"}");        

diff --git a/orcid-web/src/main/java/org/orcid/frontend/web/controllers/OauthAuthorizeController.java b/orcid-web/src/main/java/org/orcid/frontend/web/controllers/OauthAuthorizeController.java
@@ -276,7 +276,7 @@ public ModelAndView loginGetHandler(HttpServletRequest request, HttpServletRespo
             } else {
                 orcid = auth.getPrincipal().toString();
             }
-            eventManager.createEvent(orcid, eventType, request);
+            eventManager.createEvent(eventType, request);
         }
         if(new HttpSessionRequestCache().getRequest(request, response) != null)
             new HttpSessionRequestCache().removeRequest(request, response);

diff --git a/orcid-web/src/main/java/org/orcid/frontend/web/controllers/PublicRecordController.java b/orcid-web/src/main/java/org/orcid/frontend/web/controllers/PublicRecordController.java
@@ -185,7 +185,7 @@ PublicRecord getPublicRecord(@PathVariable("orcid") String orcid) {
 
         try {
             if (Features.EVENTS.isActive()) {
-                eventManager.createEvent(orcid, EventType.PUBLIC_PAGE, null);
+                eventManager.createEvent(EventType.PUBLIC_PAGE, null);
             }
             // Check if the profile is deprecated or locked
             orcidSecurityManager.checkProfile(orcid);

diff --git a/orcid-web/src/main/java/org/orcid/frontend/web/controllers/RegistrationController.java b/orcid-web/src/main/java/org/orcid/frontend/web/controllers/RegistrationController.java
@@ -282,7 +282,7 @@ public void validateGrcaptcha(HttpServletRequest request, @RequestBody Registrat
             // Ip
             String ip = OrcidRequestUtil.getIpAddress(request);
             if (Features.EVENTS.isActive()) {
-                eventManager.createEvent(getCurrentUserOrcid(), EventType.NEW_REGISTRATION, request);
+                eventManager.createEvent(EventType.NEW_REGISTRATION, request);
             }
             createMinimalRegistrationAndLogUserIn(request, response, reg, usedCaptcha, locale, ip);
         } catch (Exception e) {

diff --git a/orcid-web/src/main/java/org/orcid/frontend/web/controllers/ShibbolethController.java b/orcid-web/src/main/java/org/orcid/frontend/web/controllers/ShibbolethController.java
@@ -173,7 +173,7 @@ public ModelAndView signinHandler(HttpServletRequest request, HttpServletRespons
                 processAuthentication(remoteUser, userConnectionEntity);
                 if (Features.EVENTS.isActive()) {
                     OrcidProfileUserDetails orcidProfileUserDetails = getOrcidProfileUserDetails(userConnectionEntity.getOrcid());
-                    eventManager.createEvent(orcidProfileUserDetails.getOrcid(), EventType.SIGN_IN, request);
+                    eventManager.createEvent(EventType.SIGN_IN, request);
                 }
             } catch (AuthenticationException e) {
                 // this should never happen