Replace Orika mapping with MapStruct

pom.xml

@@ -33,6 +33,7 @@
         <ossindex.version>3.2.0</ossindex.version>
         <swagger-core.version>2.2.34</swagger-core.version>
         <jgit.version>6.10.1.202505221210-r</jgit.version>
+        <mapstruct.version>1.6.3</mapstruct.version>
         <surefire.jacoco.args />
         <failsafe.jacoco.args />
     </properties>
@@ -139,12 +140,6 @@
             <version>${jgit.version}</version>
         </dependency>
 
-        <dependency>
-            <groupId>ma.glasnost.orika</groupId>
-            <artifactId>orika-core</artifactId>
-            <version>1.5.4</version>
-        </dependency>
-
         <dependency>
             <groupId>com.google.guava</groupId>
             <artifactId>guava</artifactId>
@@ -480,6 +475,12 @@
             <version>4.3.1</version>
         </dependency>
 
+        <dependency>
+            <groupId>org.mapstruct</groupId>
+            <artifactId>mapstruct</artifactId>
+            <version>${mapstruct.version}</version>
+        </dependency>
+
         <!-- These two don't need to be included in the war package so their scope is "provided" which means we
              promise to make them available in the deployment environment, but we aren't going to because they are
              not needed at runtime anyway. -->
@@ -536,6 +537,13 @@
                 <version>3.10.1</version>
                 <configuration>
                     <release>11</release>
+                    <annotationProcessorPaths>
+                        <path>
+                            <groupId>org.mapstruct</groupId>
+                            <artifactId>mapstruct-processor</artifactId>
+                            <version>${mapstruct.version}</version>
+                        </path>
+                    </annotationProcessorPaths>
                 </configuration>
             </plugin>
             <plugin>

src/main/java/uk/ac/cam/cl/dtg/isaac/api/EventsFacade.java

@@ -23,7 +23,6 @@
 import com.opencsv.CSVWriter;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.tags.Tag;
-import ma.glasnost.orika.MapperFacade;
 import org.jboss.resteasy.annotations.GZIP;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -66,6 +65,7 @@
 import uk.ac.cam.cl.dtg.segue.search.AbstractFilterInstruction;
 import uk.ac.cam.cl.dtg.segue.search.DateRangeFilterInstruction;
 import uk.ac.cam.cl.dtg.util.AbstractConfigLoader;
+import uk.ac.cam.cl.dtg.util.mappers.MainMapper;
 
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.ws.rs.DELETE;
@@ -117,7 +117,7 @@ public class EventsFacade extends AbstractIsaacFacade {
     private final UserAccountManager userAccountManager;
     private final SchoolListReader schoolListReader;
 
-    private final MapperFacade mapper;
+    private final MainMapper mapper;
 
     /**
      * EventsFacade.
@@ -136,7 +136,7 @@ public EventsFacade(final AbstractConfigLoader properties, final ILogManager log
                         final UserAssociationManager userAssociationManager,
                         final GroupManager groupManager,
                         final UserAccountManager userAccountManager, final SchoolListReader schoolListReader,
-                        final MapperFacade mapper) {
+                        final MainMapper mapper) {
         super(properties, logManager);
         this.bookingManager = bookingManager;
         this.userManager = userManager;
@@ -327,7 +327,7 @@ private ResultsWrapper<ContentDTO> getEventsReservedByUser(final HttpServletRequ
             throws SegueDatabaseException, ContentManagerException {
         List<ContentDTO> filteredResults = Lists.newArrayList();
 
-        List<EventBookingDTO> userReservationList = this.mapper.mapAsList(bookingManager.getAllEventReservationsForUser(currentUser.getId()), EventBookingDTO.class);
+        List<EventBookingDTO> userReservationList = this.mapper.map(bookingManager.getAllEventReservationsForUser(currentUser.getId()));
 
         for (EventBookingDTO booking : userReservationList) {
 
@@ -565,7 +565,7 @@ public final Response getEventBookingForGivenGroup(@Context final HttpServletReq
             eventBookings = userAssociationManager.filterUnassociatedRecords(currentUser, eventBookings,
                     booking -> booking.getUserBooked().getId());
 
-            return Response.ok(this.mapper.mapAsList(eventBookings, EventBookingDTO.class)).build();
+            return Response.ok(this.mapper.copy(eventBookings)).build();
         } catch (SegueDatabaseException e) {
             String errorMsg = String.format(
                     "Database error occurred while trying retrieve bookings for group (%s) on event (%s).",
@@ -598,12 +598,15 @@ public final Response getEventBookingForAllGroups(@Context final HttpServletRequ
                 return new SegueErrorResponse(Status.FORBIDDEN, "You do not have permission to use this endpoint.").toResponse();
             }
 
-            List<EventBookingDTO> eventBookings = this.mapper.mapAsList(bookingManager.getBookingsByEventId(eventId), EventBookingDTO.class);
+            List<EventBookingDTO> eventBookings = this.mapper.map(bookingManager.getBookingsByEventId(eventId));
 
             // Only allowed to see the bookings of connected users
             eventBookings = userAssociationManager.filterUnassociatedRecords(
                     currentUser, eventBookings, booking -> booking.getUserBooked().getId());
 
+            eventBookings.forEach(booking -> booking.setUserBooked(mapper.map(booking.getUserBooked(),
+                    UserSummaryDTO.class)));
+
             return Response.ok(eventBookings).build();
         } catch (NoUserLoggedInException e) {
             return SegueErrorResponse.getNotLoggedInResponse();
@@ -778,7 +781,7 @@ public final Response createBookingForGivenUser(@Context final HttpServletReques
                             ADMIN_BOOKING_REASON_FIELDNAME, additionalInformation.get("authorisation") == null ? "NOT_PROVIDED" : additionalInformation.get("authorisation")
                     ));
 
-            return Response.ok(this.mapper.map(booking, EventBookingDTO.class)).build();
+            return Response.ok(this.mapper.copy(booking)).build();
         } catch (NoUserLoggedInException e) {
             return SegueErrorResponse.getNotLoggedInResponse();
         } catch (SegueDatabaseException e) {
@@ -859,7 +862,7 @@ public final Response createReservationsForGivenUsers(@Context final HttpServlet
                             USER_ID_LIST_FKEY_FIELDNAME, userIds.toArray(),
                             BOOKING_STATUS_FIELDNAME, BookingStatus.RESERVED.toString()
                     ));
-            return Response.ok(this.mapper.mapAsList(bookings, EventBookingDTO.class)).build();
+            return Response.ok(this.mapper.copy(bookings)).build();
 
         } catch (NoUserLoggedInException e) {
             return SegueErrorResponse.getNotLoggedInResponse();
@@ -1017,7 +1020,7 @@ public final Response createBookingForMe(@Context final HttpServletRequest reque
             this.getLogManager().logEvent(userManager.getCurrentUser(request), request,
                     SegueServerLogType.EVENT_BOOKING, ImmutableMap.of(EVENT_ID_FKEY_FIELDNAME, event.getId()));
 
-            return Response.ok(this.mapper.map(eventBookingDTO, EventBookingDTO.class)).build();
+            return Response.ok(this.mapper.copy(eventBookingDTO)).build();
         } catch (NoUserLoggedInException e) {
             return SegueErrorResponse.getNotLoggedInResponse();
         } catch (SegueDatabaseException e) {
@@ -1072,7 +1075,7 @@ public final Response addMeToWaitingList(@Context final HttpServletRequest reque
             this.getLogManager().logEvent(userManager.getCurrentUser(request), request,
                     SegueServerLogType.EVENT_WAITING_LIST_BOOKING, ImmutableMap.of(EVENT_ID_FKEY_FIELDNAME, event.getId()));
 
-            return Response.ok(this.mapper.map(eventBookingDTO, EventBookingDTO.class)).build();
+            return Response.ok(this.mapper.copy(eventBookingDTO)).build();
         } catch (NoUserLoggedInException e) {
             return SegueErrorResponse.getNotLoggedInResponse();
         } catch (SegueDatabaseException e) {
@@ -1608,7 +1611,7 @@ private IsaacEventPageDTO getRawEventDTOById(final String eventId)
             // The Events Facade *mutates* the EventDTO returned by this method; we must return a copy of
             // the original object else we will poison the contentManager's cache!
             // TODO: might it be better to get the DO from the cache and map it to DTO here to reduce overhead?
-            return mapper.map(possibleEvent, IsaacEventPageDTO.class);
+            return (IsaacEventPageDTO) mapper.copy(possibleEvent);
         }
         return null;
     }

src/main/java/uk/ac/cam/cl/dtg/isaac/api/PagesFacade.java

@@ -20,7 +20,6 @@
 import com.google.inject.Inject;
 import io.swagger.v3.oas.annotations.Operation;
 import io.swagger.v3.oas.annotations.tags.Tag;
-import ma.glasnost.orika.MapperFacade;
 import org.jboss.resteasy.annotations.GZIP;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -60,6 +59,7 @@
 import uk.ac.cam.cl.dtg.segue.dao.content.ContentManagerException;
 import uk.ac.cam.cl.dtg.segue.dao.content.GitContentManager;
 import uk.ac.cam.cl.dtg.util.AbstractConfigLoader;
+import uk.ac.cam.cl.dtg.util.mappers.MainMapper;
 
 import jakarta.annotation.Nullable;
 import jakarta.servlet.http.HttpServletRequest;
@@ -102,7 +102,7 @@ public class PagesFacade extends AbstractIsaacFacade {
     private static final Logger log = LoggerFactory.getLogger(PagesFacade.class);
 
     private final ContentService api;
-    private final MapperFacade mapper;
+    private final MainMapper mapper;
     private final UserAccountManager userManager;
     private final URIManager uriManager;
     private final QuestionManager questionManager;
@@ -134,7 +134,7 @@ public class PagesFacade extends AbstractIsaacFacade {
      */
     @Inject
     public PagesFacade(final ContentService api, final AbstractConfigLoader propertiesLoader,
-                       final ILogManager logManager, final MapperFacade mapper, final GitContentManager contentManager,
+                       final ILogManager logManager, final MainMapper mapper, final GitContentManager contentManager,
                        final UserAccountManager userManager, final URIManager uriManager,
                        final QuestionManager questionManager, final GameManager gameManager,
                        final UserAttemptManager userAttemptManager) {

src/main/java/uk/ac/cam/cl/dtg/isaac/api/managers/GameManager.java

@@ -19,7 +19,6 @@
 import com.google.api.client.util.Maps;
 import com.google.common.collect.Sets;
 import com.google.inject.Inject;
-import ma.glasnost.orika.MapperFacade;
 import org.apache.commons.collections4.comparators.ComparatorChain;
 import org.apache.commons.lang3.Validate;
 import org.apache.commons.lang3.tuple.ImmutablePair;
@@ -56,10 +55,11 @@
 import uk.ac.cam.cl.dtg.segue.dao.SegueDatabaseException;
 import uk.ac.cam.cl.dtg.segue.dao.content.ContentManagerException;
 import uk.ac.cam.cl.dtg.segue.dao.content.GitContentManager;
+import uk.ac.cam.cl.dtg.util.AbstractConfigLoader;
+import uk.ac.cam.cl.dtg.util.mappers.MainMapper;
 
 import jakarta.annotation.Nullable;
 import jakarta.validation.constraints.NotNull;
-import uk.ac.cam.cl.dtg.util.AbstractConfigLoader;
 
 import java.util.ArrayList;
 import java.util.Collection;
@@ -93,7 +93,7 @@ public class GameManager {
 
     private final GameboardPersistenceManager gameboardPersistenceManager;
     private final Random randomGenerator;
-    private final MapperFacade mapper;
+    private final MainMapper mapper;
     private final GitContentManager contentManager;
     private final QuestionManager questionManager;
 
@@ -111,7 +111,7 @@ public class GameManager {
      */
     @Inject
     public GameManager(final GitContentManager contentManager,
-                       final GameboardPersistenceManager gameboardPersistenceManager, final MapperFacade mapper,
+                       final GameboardPersistenceManager gameboardPersistenceManager, final MainMapper mapper,
                        final QuestionManager questionManager,
                        final AbstractConfigLoader properties) {
         this.contentManager = contentManager;

src/main/java/uk/ac/cam/cl/dtg/isaac/api/managers/QuizQuestionManager.java

@@ -32,16 +32,14 @@
 import uk.ac.cam.cl.dtg.segue.api.managers.QuestionManager;
 import uk.ac.cam.cl.dtg.segue.dao.SegueDatabaseException;
 import uk.ac.cam.cl.dtg.segue.dao.content.ContentManagerException;
-import uk.ac.cam.cl.dtg.segue.dao.content.ContentMapper;
 import uk.ac.cam.cl.dtg.isaac.dos.QuestionValidationResponse;
-import uk.ac.cam.cl.dtg.isaac.dos.content.Choice;
-import uk.ac.cam.cl.dtg.isaac.dos.content.DTOMapping;
 import uk.ac.cam.cl.dtg.isaac.dos.content.Question;
 import uk.ac.cam.cl.dtg.isaac.dto.QuestionValidationResponseDTO;
 import uk.ac.cam.cl.dtg.isaac.dto.SegueErrorResponse;
 import uk.ac.cam.cl.dtg.isaac.dto.content.ChoiceDTO;
 import uk.ac.cam.cl.dtg.isaac.dto.content.QuestionDTO;
 import uk.ac.cam.cl.dtg.isaac.dto.users.RegisteredUserDTO;
+import uk.ac.cam.cl.dtg.util.mappers.MainMapper;
 
 import jakarta.annotation.Nullable;
 import jakarta.ws.rs.core.Response;
@@ -58,7 +56,7 @@
 
 public class QuizQuestionManager {
     private final QuestionManager questionManager;
-    private final ContentMapper mapper;
+    private final MainMapper mapper;
     private final IQuizQuestionAttemptPersistenceManager quizQuestionAttemptManager;
     private final QuizManager quizManager;
     private final QuizAttemptManager quizAttemptManager;
@@ -83,7 +81,7 @@ public class QuizQuestionManager {
      *            - for attempts, particularly checking attempts are completed before revealing feedback.
      */
     @Inject
-    public QuizQuestionManager(final QuestionManager questionManager, final ContentMapper mapper,
+    public QuizQuestionManager(final QuestionManager questionManager, final MainMapper mapper,
                                final IQuizQuestionAttemptPersistenceManager quizQuestionAttemptManager,
                                final QuizManager quizManager, final QuizAttemptManager quizAttemptManager) {
         this.questionManager = questionManager;
@@ -109,7 +107,7 @@ public QuestionValidationResponseDTO validateAnswer(Question question, ChoiceDTO
     }
 
     public void recordQuestionAttempt(QuizAttemptDTO quizAttempt, QuestionValidationResponseDTO questionResponse) throws SegueDatabaseException {
-        QuestionValidationResponse questionResponseDO = this.mapper.getAutoMapper().map(questionResponse, QuestionValidationResponse.class);
+        QuestionValidationResponse questionResponseDO = this.mapper.map(questionResponse);
 
         this.quizQuestionAttemptManager.registerQuestionAttempt(quizAttempt.getId(), questionResponseDO);
     }
@@ -301,11 +299,8 @@ void augmentQuestionObjectWithAttemptInformation(Map<QuestionDTO, QuestionValida
                     lastAttempt = questionManager.convertQuestionValidationResponseToDTO(lastResponse);
                 } else {
                     // Manual extract only the safe details (questionId, answer).
-                    Choice answer = lastResponse.getAnswer();
                     lastAttempt = new QuestionValidationResponseDTO();
-                    DTOMapping dtoMapping = answer.getClass().getAnnotation(DTOMapping.class);
-                    lastAttempt.setAnswer(mapper.getAutoMapper().map(lastResponse.getAnswer(),
-                        (Class<? extends ChoiceDTO>) dtoMapping.value()));
+                    lastAttempt.setAnswer(mapper.map(lastResponse.getAnswer()));
                     lastAttempt.setQuestionId(lastResponse.getQuestionId());
                 }
                 lastAttempt.setDateAttempted(null);  // Strip timestamps, since quiz responses may be seen by other users.

src/main/java/uk/ac/cam/cl/dtg/isaac/api/services/ContentSummarizerService.java

@@ -16,20 +16,20 @@
 package uk.ac.cam.cl.dtg.isaac.api.services;
 
 import com.google.inject.Inject;
-import ma.glasnost.orika.MapperFacade;
 import uk.ac.cam.cl.dtg.isaac.api.managers.URIManager;
 import uk.ac.cam.cl.dtg.isaac.dto.ResultsWrapper;
 import uk.ac.cam.cl.dtg.isaac.dto.content.ContentDTO;
 import uk.ac.cam.cl.dtg.isaac.dto.content.ContentSummaryDTO;
+import uk.ac.cam.cl.dtg.util.mappers.ContentMapper;
 
 import java.util.ArrayList;
 
 public class ContentSummarizerService {
-    protected final MapperFacade mapper;
+    protected final ContentMapper mapper;
     private final URIManager uriManager;
 
     @Inject
-    public ContentSummarizerService(final MapperFacade mapper, final URIManager uriManager) {
+    public ContentSummarizerService(final ContentMapper mapper, final URIManager uriManager) {
         this.mapper = mapper;
         this.uriManager = uriManager;
     }

src/main/java/uk/ac/cam/cl/dtg/isaac/dao/GameboardPersistenceManager.java

@@ -24,7 +24,6 @@
 import com.google.common.collect.Lists;
 import com.google.common.collect.Maps;
 import com.google.inject.Inject;
-import ma.glasnost.orika.MapperFacade;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import uk.ac.cam.cl.dtg.isaac.dos.GameboardContentDescriptor;
@@ -40,9 +39,10 @@
 import uk.ac.cam.cl.dtg.segue.api.Constants;
 import uk.ac.cam.cl.dtg.segue.dao.SegueDatabaseException;
 import uk.ac.cam.cl.dtg.segue.dao.content.ContentManagerException;
-import uk.ac.cam.cl.dtg.segue.dao.content.ContentMapper;
+import uk.ac.cam.cl.dtg.segue.dao.content.ContentSubclassMapper;
 import uk.ac.cam.cl.dtg.segue.dao.content.GitContentManager;
 import uk.ac.cam.cl.dtg.segue.database.PostgresSqlDb;
+import uk.ac.cam.cl.dtg.util.mappers.MainMapper;
 
 import jakarta.annotation.Nullable;
 import java.io.IOException;
@@ -81,7 +81,7 @@ public class GameboardPersistenceManager {
     private final PostgresSqlDb database;
     private final Cache<String, GameboardDO> gameboardNonPersistentStorage;
 
-    private final MapperFacade mapper; // used for content object mapping.
+    private final MainMapper mapper; // used for content object mapping.
     private final ObjectMapper objectMapper; // used for json serialisation
 
     private final GitContentManager contentManager;
@@ -100,7 +100,7 @@ public class GameboardPersistenceManager {
      */
     @Inject
     public GameboardPersistenceManager(final PostgresSqlDb database, final GitContentManager contentManager,
-                                       final MapperFacade mapper, final ContentMapper objectMapper) {
+                                       final MainMapper mapper, final ContentSubclassMapper objectMapper) {
         this.database = database;
         this.mapper = mapper;
         this.contentManager = contentManager;
@@ -189,7 +189,7 @@ public String temporarilyStoreGameboard(final GameboardDTO gameboard) {
      */
     public String saveGameboardToPermanentStorage(final GameboardDTO gameboard)
         throws SegueDatabaseException {
-        GameboardDO gameboardToSave = mapper.map(gameboard, GameboardDO.class);
+        GameboardDO gameboardToSave = mapper.map(gameboard);
         // the mapping operation won't work for the list so we should just
         // create a new one.
         gameboardToSave.setContents(Lists.newArrayList());
@@ -601,7 +601,7 @@ private List<GameboardDTO> convertToGameboardDTOs(final List<GameboardDO> gamebo
         List<GameboardDTO> gameboardDTOs = Lists.newArrayList();
 
         for (GameboardDO gameboardDO : gameboardDOs) {
-            GameboardDTO gameboardDTO = mapper.map(gameboardDO, GameboardDTO.class);
+            GameboardDTO gameboardDTO = mapper.map(gameboardDO);
             List<GameboardItem> sparseGameboardItems = gameboardDO.getContents().stream()
                     .map(GameboardItem::buildLightweightItemFromContentDescriptor)
                     .collect(Collectors.toList());
@@ -640,7 +640,7 @@ private GameboardDTO convertToGameboardDTO(final GameboardDO gameboardDO, final
      * @return GameboardDO.
      */
     private GameboardDO convertToGameboardDO(final GameboardDTO gameboardDTO) {
-        GameboardDO gameboardDO = mapper.map(gameboardDTO, GameboardDO.class);
+        GameboardDO gameboardDO = mapper.map(gameboardDTO);
         // the mapping operation won't work for the list so we should just
         // create a new one.
         gameboardDO.setContents(Lists.newArrayList());