BMGS hard coding Removal

Question Set Editor

IEditorConfig is not generalised and it contains BMGS fields

board, medium, grade Level, subject, topic present in editor.ts file as:

Which defines the IEditorConfig interface

export interface IEditorConfig {
    context: Context;
    config: any;
    metadata?: any;
    data?: any;
}

export interface Context {
    ...
    ...
    board?: any;
    medium?: any;
    gradeLevel?: any;
    subject?: any;
    topic?: any;
    ...
    ...
}

This interface defines the editorConfig.
These value of board, medium, grade Level, subject, topic are getting attached as default value to question if question set does not have BMGS fields.
If question set contains BMGS fields and then BMGS present in editor config get replaced with selected BMGS in question set.

Solution to Generalise interface

export interface IEditorConfig {
    context: Context;
    config: any;
    metadata?: any;
    data?: any;
}

export interface Context {
    ...
    ...
    frameworkDefaultValues?: any
    ...
    ...
}

This frameworkDefaultValues can store board, medium, grade Level, subject, topic, etc as:

frameworkDefaultValues: {
  board: 'CBSE',
  medium: 'English',
  gradeLevel: 'Class 6'
  subject: 'Mathematics',
  ...
}

or it can also contains any values such as hospital, department, etc

frameworkDefaultValues: {
  hospital: 'Apollo',
  department: 'Cardiology',
  ...
  ...
}

Place where above passed BMGS is currently getting fetched and getting added to question metadata

File - question.component.ts

_.pick(_.get(this.editorService.editorConfig, 'context'), ['board', 'medium',
'gradeLevel', 'subject', 'topic'])

If we pass this values in context.frameworkDefaultValues

then this line can also be generalised to

_.get(this.editorService.editorConfig, 'context.frameworkDefaultValues')

BMGS and other fields hard coding in ConfigService

File - config.service.ts

sessionContext contains hard coded fields name as

public sessionContext: Array<string> = ['board', 'medium', 'gradeLevel', 'subject',
'topic', 'author', 'channel', 'framework', 'copyright', 'attributions', 'audience', 'license']

To remove these hardcoded fields name we can pass it as a config to editor as sessionContextFields

editorConfig: {
  config: {
    sessionContextFields: ['board', 'medium', 'gradeLevel', 'subject',
    'topic', 'author', 'channel', 'framework', 'copyright', 'attributions', 'audience', 'license']
  }
}

Another File which contains BMGS hard coded is File - SessionContext.ts

This file is not getting used anywhere we can remove this file.

Other files which contains BMGS are mock data for test cases in these files:

components/assign-page-number/assign-page-number.component.spec.data.ts
components/editor/editor.component.spec.data.ts
components/fancy-tree/fancy-tree.component.spec.data.ts
components/meta-form/meta-form.component.spec.data.ts
components/question/question.component.spec.data.ts
components/quml-player/quml-player.component.spec.data.ts
components/qumlplayer-page/qumlplayer-page.component.spec.data.ts
services/editor/editor.service.spec.data.ts
services/tree/tree.service.spec.data.ts

We can remove BMGS fields from these files.

Resource Library

BMGS and various other fields are shown in this page and it’s not configurable

Content-player-page

Solution 1 - we can use the form config which is being used while question creation to the field code and label and display in this same above format without bringing any new UI change.

Solution 2-We can define a new form config in which these fields can be set as config. (Not Recommended)

It will add additional config to the editor.

Board value is getting displayed to the contents

Suggestion1 - We can remove showing “By CBSE”

Suggestion2 - we can make it configurable if showing a field is needed

BMGS is present in SessionContext.ts file

This file is not getting used anywhere so can be removed.

BMGS is present in config.service.ts

It defines sessionContext but it is not getting used so can be removed.

So we can remove sessionContext from this file.