From d351bb6c8c6ea27a659f02ebf8ea65617c643ed9 Mon Sep 17 00:00:00 2001
From: flaviuvsp <flaviu.silaghi@vspartners.us>
Date: Tue, 1 Oct 2024 14:00:24 +0300
Subject: [PATCH 1/2] Fix description and change texts

---
 .../compose/EditMetadataScreen.kt                | 16 +++++++++-------
 .../viewmodels/EditMetadataViewModel.kt          |  2 +-
 app/src/main/res/values/strings.xml              |  4 ++--
 3 files changed, 12 insertions(+), 10 deletions(-)

diff --git a/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt b/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt
index 6c4b9100..5bf97148 100644
--- a/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt
+++ b/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt
@@ -122,7 +122,10 @@ fun EditMetadataScreen(
             lightGreyColor,
             someFilesHaveDescription,
             redColor,
-            smallTextSize
+            smallTextSize,
+            onTextChange = {
+                inputDescription = it
+            }
         )
 
         Divider(modifier = Modifier.padding(vertical = 16.dp))
@@ -282,9 +285,10 @@ private fun DescriptionView(
     lightGreyColor: Color,
     someFilesHaveDescription: Boolean?,
     redColor: Color,
-    smallTextSize: TextUnit
+    smallTextSize: TextUnit,
+    onTextChange: (String) -> Unit
 ) {
-    var description = inputDescription
+//    var description = inputDescription
     Row(
         verticalAlignment = Alignment.Top, horizontalArrangement = Arrangement.spacedBy(16.dp)
     ) {
@@ -304,10 +308,8 @@ private fun DescriptionView(
     Spacer(modifier = Modifier.height(10.dp))
 
     TextField(
-        value = description,
-        onValueChange = { value ->
-            description = value
-        },
+        value = inputDescription,
+        onValueChange = onTextChange,
         label = { Text(text = stringResource(id = R.string.edit_files_metadata_description_hint)) },
         singleLine = true,
         modifier = Modifier
diff --git a/app/src/main/java/org/permanent/permanent/viewmodels/EditMetadataViewModel.kt b/app/src/main/java/org/permanent/permanent/viewmodels/EditMetadataViewModel.kt
index 4862dce4..d8081cc4 100644
--- a/app/src/main/java/org/permanent/permanent/viewmodels/EditMetadataViewModel.kt
+++ b/app/src/main/java/org/permanent/permanent/viewmodels/EditMetadataViewModel.kt
@@ -26,7 +26,6 @@ class EditMetadataViewModel(application: Application) : ObservableAndroidViewMod
     private val tagsOfSelectedRecords =
         MutableLiveData<SnapshotStateList<Tag>>(mutableStateListOf())
     private var commonTags: MutableList<Tag> = mutableListOf()
-    private var initialDescription: String = ""
     private var commonDescription: String = ""
     private var showWarningSomeFilesHaveDescription = MutableLiveData(false)
     val showError = MutableLiveData<String>()
@@ -96,6 +95,7 @@ class EditMetadataViewModel(application: Application) : ObservableAndroidViewMod
     }
 
     private fun checkForCommonDescription() {
+        var initialDescription = ""
         for (record in records) {
             val recordDescription = record.fileData?.description
             if (!recordDescription.isNullOrBlank()) {
diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml
index 8de8d90c..5a944707 100644
--- a/app/src/main/res/values/strings.xml
+++ b/app/src/main/res/values/strings.xml
@@ -723,12 +723,12 @@
     <string name="add_location">Add Location</string>
     <string name="various_locations">Various locations</string>
     <string name="location_confirmation_title">New location</string>
-    <string name="location_confirmation_substring">Are you sure you want set a new location for selected items?</string>
+    <string name="location_confirmation_substring">Are you sure you want to set a new date &amp; time for the selected items?</string>
     <string name="add_date_time">Add date and time</string>
     <string name="add_date">Add date</string>
     <string name="various_date_times">Various dates and times</string>
     <string name="date_confirmation_title">New Date &amp; Time</string>
-    <string name="date_confirmation_substring">Are you sure you want set a new date &amp; time for selected items?</string>
+    <string name="date_confirmation_substring">Are you sure you want to set a new date &amp; time for the selected items?</string>
     <string name="set_date_and_time">Set date and time</string>
 
 

From aee12eec29ca654a9ef9f0fce7d260e8de303f63 Mon Sep 17 00:00:00 2001
From: flaviuvsp <flaviu.silaghi@vspartners.us>
Date: Thu, 3 Oct 2024 11:35:36 +0300
Subject: [PATCH 2/2] remove commented code

---
 .../permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt  | 1 -
 1 file changed, 1 deletion(-)

diff --git a/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt b/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt
index 5bf97148..7614ae25 100644
--- a/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt
+++ b/app/src/main/java/org/permanent/permanent/ui/bulkEditMetadata/compose/EditMetadataScreen.kt
@@ -288,7 +288,6 @@ private fun DescriptionView(
     smallTextSize: TextUnit,
     onTextChange: (String) -> Unit
 ) {
-//    var description = inputDescription
     Row(
         verticalAlignment = Alignment.Top, horizontalArrangement = Arrangement.spacedBy(16.dp)
     ) {