From 61cb834c9124fc6a43752a3a06b704ede98baaa5 Mon Sep 17 00:00:00 2001
From: Bijoy Singh Kochar <bijoysingh693@gmail.com>
Date: Thu, 21 Feb 2019 21:45:16 +0000
Subject: [PATCH] Fixing potential crash issues if data gets corrupted

---
 .../maubis/scarlet/base/core/note/NoteSortingUtils.kt |  2 +-
 .../com/maubis/scarlet/base/database/NotesProvider.kt |  2 ++
 .../com/maubis/scarlet/base/note/NoteExtensions.kt    | 11 +++++++----
 3 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/base/src/main/java/com/maubis/scarlet/base/core/note/NoteSortingUtils.kt b/base/src/main/java/com/maubis/scarlet/base/core/note/NoteSortingUtils.kt
index 77eefac5..1113cc22 100644
--- a/base/src/main/java/com/maubis/scarlet/base/core/note/NoteSortingUtils.kt
+++ b/base/src/main/java/com/maubis/scarlet/base/core/note/NoteSortingUtils.kt
@@ -26,7 +26,7 @@ fun sort(notes: List<Note>, sortingTechnique: SortingTechnique): List<Note> {
     SortingTechnique.ALPHABETICAL -> notes.sortedBy { note ->
       val content = note.getAlphabets()
       if (note.pinned || content.isBlank()) 0
-      else content[0].toInt()
+      else content[0].toUpperCase().toInt()
     }
     else -> notes.sortedByDescending { note ->
       if (note.pinned) Long.MAX_VALUE
diff --git a/base/src/main/java/com/maubis/scarlet/base/database/NotesProvider.kt b/base/src/main/java/com/maubis/scarlet/base/database/NotesProvider.kt
index d7e59a93..3857de91 100644
--- a/base/src/main/java/com/maubis/scarlet/base/database/NotesProvider.kt
+++ b/base/src/main/java/com/maubis/scarlet/base/database/NotesProvider.kt
@@ -5,6 +5,7 @@ import com.maubis.scarlet.base.core.note.INoteContainer
 import com.maubis.scarlet.base.database.room.folder.Folder
 import com.maubis.scarlet.base.database.room.note.Note
 import com.maubis.scarlet.base.database.room.note.NoteDao
+import com.maubis.scarlet.base.note.applySanityChecks
 import java.util.concurrent.ConcurrentHashMap
 
 class NotesProvider {
@@ -96,6 +97,7 @@ class NotesProvider {
       return
     }
     database().all.forEach {
+      it.applySanityChecks()
       notes[it.uuid] = it
     }
   }
diff --git a/base/src/main/java/com/maubis/scarlet/base/note/NoteExtensions.kt b/base/src/main/java/com/maubis/scarlet/base/note/NoteExtensions.kt
index 763842b4..f5ca2348 100644
--- a/base/src/main/java/com/maubis/scarlet/base/note/NoteExtensions.kt
+++ b/base/src/main/java/com/maubis/scarlet/base/note/NoteExtensions.kt
@@ -2,6 +2,7 @@ package com.maubis.scarlet.base.note
 
 import android.content.Context
 import android.content.Intent
+import android.util.Log
 import com.github.bijoysingh.starter.util.DateFormatter
 import com.google.gson.Gson
 import com.maubis.markdown.Markdown
@@ -11,6 +12,7 @@ import com.maubis.scarlet.base.config.CoreConfig.Companion.tagsDb
 import com.maubis.scarlet.base.core.format.Format
 import com.maubis.scarlet.base.core.format.FormatType
 import com.maubis.scarlet.base.core.note.NoteState
+import com.maubis.scarlet.base.core.note.generateUUID
 import com.maubis.scarlet.base.core.note.getFormats
 import com.maubis.scarlet.base.core.note.getTagUUIDs
 import com.maubis.scarlet.base.database.room.note.Note
@@ -20,6 +22,7 @@ import com.maubis.scarlet.base.note.creation.activity.CreateNoteActivity
 import com.maubis.scarlet.base.note.creation.activity.INTENT_KEY_DISTRACTION_FREE
 import com.maubis.scarlet.base.note.creation.activity.INTENT_KEY_NOTE_ID
 import com.maubis.scarlet.base.note.creation.activity.ViewAdvancedNoteActivity
+import com.maubis.scarlet.base.settings.sheet.sNoteDefaultColor
 import com.maubis.scarlet.base.support.ui.ThemedActivity
 import com.maubis.scarlet.base.support.utils.removeMarkdownHeaders
 import java.util.*
@@ -276,11 +279,11 @@ fun Note.copy(context: Context) {
 fun Note.applySanityChecks() {
   folder = folder ?: ""
   description = description ?: ""
-  timestamp = timestamp ?: 0L
-  color = color ?: 0
-  state = state ?: ""
+  timestamp = timestamp ?: System.currentTimeMillis()
+  color = color ?: sNoteDefaultColor
+  state = state ?: NoteState.DEFAULT.name
   tags = tags ?: ""
-  uuid = uuid ?: ""
+  uuid = uuid ?: generateUUID()
 }
 
 fun Note.save(context: Context) {