Hotfix: Fix inconsistent feedback query

repositories/FeedbackRepository.ts

@@ -15,8 +15,16 @@ export class FeedbackRepository extends BaseRepository<FeedbackModel> {
     return this.getBaseFindQuery({}).where({ uuid }).getOne();
   }
 
-  public async getAllFeedbackForUser(user: UserModel): Promise<FeedbackModel[]> {
-    return this.getBaseFindQuery({}).where({ user }).getMany();
+  // temporary fix for getting feedback for a user for an event
+  public async getStandardUserFeedback(user: UserModel, options: FeedbackSearchOptions): Promise<FeedbackModel[]> {
+    let query = this.getBaseFindQuery(options);
+    query = query.andWhere('feedback.user = :user', { user: user.uuid });
+
+    return query.getMany();
+  }
+
+  public async getAllFeedbackForUser(user: UserModel, options: FeedbackSearchOptions): Promise<FeedbackModel[]> {
+    return this.getBaseFindQuery(options).where({ user }).getMany();
   }
 
   public async upsertFeedback(feedback: FeedbackModel,

services/FeedbackService.ts

@@ -25,7 +25,7 @@ export default class FeedbackService {
           .map((fb) => fb.getPublicFeedback());
       }
 
-      const userFeedback = await feedbackRepository.getAllFeedbackForUser(user);
+      const userFeedback = await feedbackRepository.getStandardUserFeedback(user, options);
       return userFeedback.map((fb) => fb.getPublicFeedback());
     });
   }

tests/feedback.test.ts

@@ -164,7 +164,7 @@ describe('feedback submission', () => {
     const feedbackController = ControllerFactory.feedback(conn);
     await feedbackController.submitFeedback(buildFeedbackRequest(feedback1), member1);
     await feedbackController.submitFeedback(buildFeedbackRequest(feedback2), member2);
-    const user1Feedback = await feedbackController.getFeedback({}, member1);
+    const user1Feedback = await feedbackController.getFeedback({ user: member1.uuid }, member1);
 
     expect(user1Feedback.feedback).toHaveLength(1);