MOBILE-3320 assign: Fix leave confirm displayed when shouldn't

main
Dani Palou 2021-06-28 13:22:29 +02:00
parent f1d6014304
commit 331fd9fef9
2 changed files with 7 additions and 6 deletions

View File

@ -93,7 +93,7 @@ export class AddonModAssignFeedbackCommentsComponent extends AddonModAssignFeedb
// Update the text and save it as draft.
this.isSent = false;
this.text = this.replacePluginfileUrls(text);
this.text = this.replacePluginfileUrls(text || '');
AddonModAssignFeedbackDelegate.saveFeedbackDraft(this.assign.id, this.userId, this.plugin, {
text: text,
format: 1,

View File

@ -49,13 +49,14 @@ export class AddonModAssignFeedbackCommentsHandlerService implements AddonModAss
* @param inputData Data entered in the feedback edit form.
* @return Text to submit.
*/
getTextFromInputData(plugin: AddonModAssignPlugin, inputData: AddonModAssignFeedbackCommentsTextData): string {
getTextFromInputData(plugin: AddonModAssignPlugin, inputData: AddonModAssignFeedbackCommentsTextData): string | undefined {
if (inputData.assignfeedbackcomments_editor === undefined) {
return undefined;
}
const files = plugin.fileareas && plugin.fileareas[0] ? plugin.fileareas[0].files : [];
// The input data can have a string or an object with text and format. Get the text.
const text = inputData.assignfeedbackcomments_editor || '';
return CoreTextUtils.restorePluginfileUrls(text, files || []);
return CoreTextUtils.restorePluginfileUrls(inputData.assignfeedbackcomments_editor, files || []);
}
/**