Get rid of SaveReportJsonDataInInternalDir option

TODO.txt

@@ -5,11 +5,12 @@
 -duplicate existing report with new name
 -cleanup empty uuid folders in case user gets an internal folder created but never saves
 
--always save report data (file locations, title, etc.) to internal dir (might already be done?)
 -add dropdown to Add Items button to have add items from folder and remove choose working folder option (data always saved internally)
--no more option to save data internally -> ALWAYS save report_data.json internally
+-always save report data (file locations, title, etc.) to internal dir (might already be done?)
+    -no more option to save data internally -> ALWAYS save report_data.json internally
 -update project title -> should update recently used data
     -this sort of works, something is wrong with the upgrade process where the UUID is not brought over properly; need to test and fix (maybe fixed already and my dataset is wrong?)
+-add option to backup added files to internal data directory
 -make backup of last generated PDF
 -iOS-specific (MAUI essentials?)
     -Take picture

src/MayShow.Shared/Models/Settings.cs

@@ -17,7 +17,6 @@ class Settings : ChangeNotifier
     private bool _saveOutputPdfInWorkingDir;
     private string _outputPdfDir;
     private decimal _imageResizeThreshold;
-    private bool _saveReportJsonDataInInternalDir;
     private Dictionary<string, string> _workingFolderToInternalFolderName; // obsolete
     private List<PDFReportInfo> _allReportInfo;
     public string _dataGridDateFormat;
@@ -31,7 +30,6 @@ class Settings : ChangeNotifier
         _saveOutputPdfInWorkingDir = true;
         _outputPdfDir = "";
         _imageResizeThreshold = 1.5m;
-        _saveReportJsonDataInInternalDir = false;
         _workingFolderToInternalFolderName = [];
         _allReportInfo = [];
         _settingsVersion = 2;
@@ -46,7 +44,6 @@ class Settings : ChangeNotifier
         _saveOutputPdfInWorkingDir = other.SaveOutputPdfInWorkingDir;
         _outputPdfDir = other.OutputPdfDir;
         _imageResizeThreshold = other.ImageResizeThreshold;
-        _saveReportJsonDataInInternalDir = other.SaveReportJsonDataInInternalDir;
         _workingFolderToInternalFolderName = other.WorkingFolderToInternalFolderName;
         _settingsVersion = other.SettingsVersion;
         _allReportInfo = other.AllReportInfo;
@@ -90,13 +87,6 @@ class Settings : ChangeNotifier
         set { _imageResizeThreshold = value; NotifyPropertyChanged(); }
     }
 
-    [JsonInclude]
-    public bool SaveReportJsonDataInInternalDir
-    {
-        get => _saveReportJsonDataInInternalDir;
-        set { _saveReportJsonDataInInternalDir = value; NotifyPropertyChanged(); }
-    }
-
     [JsonInclude]
     public Dictionary<string, string> WorkingFolderToInternalFolderName
     {
@@ -217,7 +207,6 @@ class Settings : ChangeNotifier
             }
             settings.AllReportInfo = list.OrderBy(x => x.Title).ToList();
             settings.WorkingFolderToInternalFolderName = []; // clear this list; it is no longer going to be used
-            settings.SaveReportJsonDataInInternalDir = true; // this is now always true
             settings.SettingsVersion = 2;
             settings.SaveSettingsNotAsync(); // saves all data; UUIDs should be in sync if user has toggled settings
         }

src/MayShow.Shared/ViewModels/CreatePDFReportViewModel.cs

@@ -271,20 +271,13 @@ class CreatePDFReportViewModel : BaseViewModel, ICanCheckShutdown, ILogger
 
     private string GetReportSavedDataPath(string workingFolder)
     {
-        if (_settings.SaveReportJsonDataInInternalDir)
+        var internalPath = Utilities.GetInternalDataPath();
+        var internalReportDataDir = Path.Combine(internalPath, _pdfReport.UUID);
+        if (!Directory.Exists(internalReportDataDir))
         {
-            var internalPath = Utilities.GetInternalDataPath();
+            Directory.CreateDirectory(internalReportDataDir);
-            var internalReportDataDir = Path.Combine(internalPath, _pdfReport.UUID);
-            if (!Directory.Exists(internalReportDataDir))
-            {
-                Directory.CreateDirectory(internalReportDataDir);
-            }
-            return Path.Combine(internalReportDataDir, Constants.ReportSavedDataFileName);
-        }
-        else
-        {
-            return Path.Combine(workingFolder, Constants.ReportSavedDataFileName);
         }
+        return Path.Combine(internalReportDataDir, Constants.ReportSavedDataFileName);
     }
 
     private void ScanFolder(string path)

src/MayShow.Shared/ViewModels/SettingsViewModel.cs

@@ -113,16 +113,6 @@ class SettingsViewModel: ChangeNotifier
         }
     }
 
-    public bool SaveReportJsonDataInInternalDir
-    {
-        get => true; // _settings.SaveReportJsonDataInInternalDir;
-        //set
-        //{
-        //    _settings.SaveReportJsonDataInInternalDir = value;
-        //    NotifyPropertyChanged();
-        //}
-    }
-
     public List<DateDisplayFormat> DateFormats
     {
         get => _dateFormats;

src/MayShow.Shared/Views/SettingsView.axaml

@@ -72,7 +72,6 @@
                     </DataTemplate>
                 </ComboBox.ItemTemplate>
             </ComboBox>
-            <!-- <CheckBox IsChecked="{Binding SaveReportJsonDataInInternalDir}">Save report data (names, notes, etc.) in MayShow settings directory (saves in working directory by default)</CheckBox> -->
             <Button Command="{Binding OpenSettingsDir}">
                 <TextBlock>
                     <Run Text="&#xf07c;"