Skip to content

Commit

Permalink
refactor(NodeService): remove dialog parameter from constructors in N…
Browse files Browse the repository at this point in the history
…odeService and TeacherNodeService. It's only used in StudentNodeService.
  • Loading branch information
hirokiterashima committed Dec 20, 2024
1 parent 9c57795 commit b43059e
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 7 deletions.
2 changes: 0 additions & 2 deletions src/assets/wise5/services/nodeService.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
import { Injectable } from '@angular/core';
import { MatDialog } from '@angular/material/dialog';
import { ConfigService } from './configService';
import { ProjectService } from './projectService';
import { DataService } from '../../../app/services/data.service';
Expand All @@ -17,7 +16,6 @@ export abstract class NodeService {
protected configService: ConfigService,
protected constraintService: ConstraintService,
protected dataService: DataService,
protected dialog: MatDialog,
protected projectService: ProjectService
) {}

Expand Down
4 changes: 2 additions & 2 deletions src/assets/wise5/services/studentNodeService.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@ export class StudentNodeService extends NodeService {
protected configService: ConfigService,
protected constraintService: ConstraintService,
protected dataService: StudentDataService,
protected dialog: MatDialog,
private dialog: MatDialog,
private nodeStatusService: NodeStatusService,
protected projectService: ProjectService
) {
super(configService, constraintService, dataService, dialog, projectService);
super(configService, constraintService, dataService, projectService);
}

setCurrentNode(nodeId: string): void {
Expand Down
4 changes: 1 addition & 3 deletions src/assets/wise5/services/teacherNodeService.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import { NodeService } from './nodeService';
import { Subject, Observable } from 'rxjs';
import { TeacherDataService } from './teacherDataService';
import { TeacherProjectService } from './teacherProjectService';
import { MatDialog } from '@angular/material/dialog';
import { ConfigService } from './configService';
import { ConstraintService } from './constraintService';

Expand All @@ -21,10 +20,9 @@ export class TeacherNodeService extends NodeService {
protected configService: ConfigService,
protected constraintService: ConstraintService,
protected dataService: TeacherDataService,
protected dialog: MatDialog,
protected projectService: TeacherProjectService
) {
super(configService, constraintService, dataService, dialog, projectService);
super(configService, constraintService, dataService, projectService);
}

broadcastComponentShowSubmitButtonValueChanged(args: any): void {
Expand Down

0 comments on commit b43059e

Please sign in to comment.