Merge pull request #1704 from Infomaniak/subject-max-length

KevinBoulongne · web-flow · commit 97819a405917 · 2024-02-16T11:15:16.000+01:00
Truncate Message' Subject if it's longer than 998 characters
diff --git a/app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageViewModel.kt b/app/src/main/java/com/infomaniak/mail/ui/newMessage/NewMessageViewModel.kt
@@ -633,6 +633,7 @@ class NewMessageViewModel @Inject constructor(
     private fun saveDraftToLocal(action: DraftAction) {
         SentryLog.d("Draft", "Save Draft to local")
 
+        draft.subject = draft.subject?.take(SUBJECT_MAX_LENGTH)
         draft.body = draft.uiBody.textToHtml() + (draft.uiSignature ?: "") + (draft.uiQuote ?: "")
         draft.action = action
 
@@ -733,5 +734,6 @@ class NewMessageViewModel @Inject constructor(
     companion object {
         private const val DELAY_BEFORE_AUTO_SAVING_DRAFT = 1_000L
         private const val FILE_SIZE_25_MB = 25L * 1_024L * 1_024L
+        private const val SUBJECT_MAX_LENGTH = 998
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -633,6 +633,7 @@ class NewMessageViewModel @Inject constructor(`
`633`	`633`	`private fun saveDraftToLocal(action: DraftAction) {`
`634`	`634`	`SentryLog.d("Draft", "Save Draft to local")`
`635`	`635`
	`636`	`+ draft.subject = draft.subject?.take(SUBJECT_MAX_LENGTH)`
`636`	`637`	`draft.body = draft.uiBody.textToHtml() + (draft.uiSignature ?: "") + (draft.uiQuote ?: "")`
`637`	`638`	`draft.action = action`
`638`	`639`
`@@ -733,5 +734,6 @@ class NewMessageViewModel @Inject constructor(`
`733`	`734`	`companion object {`
`734`	`735`	`private const val DELAY_BEFORE_AUTO_SAVING_DRAFT = 1_000L`
`735`	`736`	`private const val FILE_SIZE_25_MB = 25L * 1_024L * 1_024L`
	`737`	`+ private const val SUBJECT_MAX_LENGTH = 998`
`736`	`738`	`}`
`737`	`739`	`}`