fix(Question Bank): Only mark questions the individual workgroup used

Previously it would mark questions that any of the workgroups used.

#1413

src/assets/wise5/components/peerChat/peer-chat-question-bank/question-bank-helper.ts

@@ -0,0 +1,6 @@
+export function getQuestionIdsUsed(componentStates: any[], workgroupId: number): string[] {
+  return componentStates
+    .filter((componentState) => componentState.workgroupId === workgroupId)
+    .filter((componentState) => componentState.studentData.questionId != null)
+    .map((componentState) => componentState.studentData.questionId);
+}

src/assets/wise5/components/peerChat/peer-chat-show-work/peer-chat-show-work.component.ts

@@ -12,6 +12,7 @@ import { NodeService } from '../../../services/nodeService';
 import { FeedbackRule } from '../../common/feedbackRule/FeedbackRule';
 import { QuestionBankRule } from '../peer-chat-question-bank/QuestionBankRule';
 import { forkJoin, Observable } from 'rxjs';
+import { getQuestionIdsUsed } from '../peer-chat-question-bank/question-bank-helper';
 
 @Component({
   selector: 'peer-chat-show-work',
@@ -102,7 +103,7 @@ export class PeerChatShowWorkComponent extends ComponentShowWorkDirective {
     this.peerChatService.setPeerChatMessages(this.peerChatMessages, componentStates);
     this.dynamicPrompt = this.getDynamicPrompt(componentStates, this.workgroupId);
     this.questionBankRules = this.getQuestionBankRule(componentStates, this.workgroupId);
-    this.questionIdsUsed = this.getQuestionIdsUsed(componentStates, this.workgroupId);
+    this.questionIdsUsed = getQuestionIdsUsed(componentStates, this.workgroupId);
   }
 
   private getDynamicPrompt(componentStates: any[], workgroupId: number): FeedbackRule {
@@ -134,13 +135,6 @@ export class PeerChatShowWorkComponent extends ComponentShowWorkDirective {
     return null;
   }
 
-  private getQuestionIdsUsed(componentStates: any[], workgroupId: number): string[] {
-    return componentStates
-      .filter((componentState) => componentState.workgroupId === workgroupId)
-      .filter((componentState) => componentState.studentData.questionId != null)
-      .map((componentState) => componentState.studentData.questionId);
-  }
-
   private addWorkgroupIdsFromPeerChatMessages(
     workgroupIds: Set<number>,
     componentStates: any[]

src/assets/wise5/components/peerChat/peer-chat-student/peer-chat-student.component.ts

@@ -21,8 +21,8 @@ import { PeerChatMessage } from '../PeerChatMessage';
 import { PeerChatService } from '../peerChatService';
 import { PeerGroup } from '../PeerGroup';
 import { Question } from '../peer-chat-question-bank/Question';
-import { ComponentState } from '../../../../../app/domain/componentState';
 import { QuestionBankService } from '../peer-chat-question-bank/questionBank.service';
+import { getQuestionIdsUsed } from '../peer-chat-question-bank/question-bank-helper';
 
 @Component({
   selector: 'peer-chat-student',
@@ -124,7 +124,7 @@ export class PeerChatStudentComponent extends ComponentStudent {
             ]).subscribe(([componentStates, annotations]) => {
               this.setPeerChatMessages(componentStates);
               this.peerChatService.processIsDeletedAnnotations(annotations, this.peerChatMessages);
-              this.questionIdsUsed = this.getQuestionIdsUsed(componentStates);
+              this.questionIdsUsed = getQuestionIdsUsed(componentStates, this.myWorkgroupId);
             });
           }
         },
@@ -134,12 +134,6 @@ export class PeerChatStudentComponent extends ComponentStudent {
       );
   }
 
-  private getQuestionIdsUsed(componentStates: ComponentState[]): string[] {
-    return componentStates
-      .filter((componentState) => componentState.studentData.questionId != null)
-      .map((componentState) => componentState.studentData.questionId);
-  }
-
   private addTeacherWorkgroupIds(workgroupIds: number[]): void {
     workgroupIds.push(...this.configService.getTeacherWorkgroupIds());
   }