Merge pull request #32 from PolyHx/fix/upload-options

BrandonRbg · web-flow · commit 3dd865e74c05 · 2019-03-22T01:12:42.000-04:00
Fix to check upload options only if there are options
diff --git a/src/modules/database/questions/questions.service.ts b/src/modules/database/questions/questions.service.ts
@@ -121,7 +121,8 @@ export class QuestionsService {
     }
 
     public async uploadFile(file: Express.Multer.File, question: Questions, teamId: string): Promise<boolean> {
-        if (!question.option.contentTypes.some(x => x.toLowerCase() === file.mimetype.toLowerCase())) {
+        if (question.option && question.option.contentTypes &&
+            !question.option.contentTypes.some(x => x.toLowerCase() === file.mimetype.toLowerCase())) {
             return false;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,8 @@ export class QuestionsService {`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`public async uploadFile(file: Express.Multer.File, question: Questions, teamId: string): Promise<boolean> {`
`124`		`- if (!question.option.contentTypes.some(x => x.toLowerCase() === file.mimetype.toLowerCase())) {`
	`124`	`+ if (question.option && question.option.contentTypes &&`
	`125`	`+ !question.option.contentTypes.some(x => x.toLowerCase() === file.mimetype.toLowerCase())) {`
`125`	`126`	`return false;`
`126`	`127`	`}`
`127`	`128`