Small Bug Fixes from Merge

Bugs:
 - Save as
 - RightCheckBoxAction

Additional key changes:
- Checking for dirty before prompting save

Author: jtle00

src/pymapmanager/interface/mainMenus.py

@@ -444,7 +444,7 @@ def _refreshFileMenu(self):
         saveFileAction.setShortcut("Ctrl+S")
         # saveFileAction.setEnabled(enableUndo and isDirty)
         saveFileAction.setEnabled(enableSave)
-        saveFileAction.triggered.connect(self.getApp().saveAs)
+        saveFileAction.triggered.connect(self.getApp().save)
         self.fileMenu.addAction(saveFileAction)
 
         # TODO: add export mmap to zip (a bit complicated)

src/pymapmanager/interface/pyMapManagerApp.py

@@ -618,7 +618,16 @@ def saveAs(self):
         if _frontWidget is None:
             return
         # self._openWidgetList.saveAs(_frontWidget)
-        _frontWidget.saveAs(_frontWidget)
+        # _frontWidget.saveAs(_frontWidget)
+        _frontWidget.saveAs()
+
+    def save(self):
+        """ Save to file
+        """
+        _frontWidget = self.getFrontStackWindow()
+        if _frontWidget is None:
+            return
+        _frontWidget.save()
 
     #abj
     def _showAnalysisParameters(self):

src/pymapmanager/interface/stackWidgets/imagePlotWidget2.py

@@ -25,6 +25,7 @@
 from .base.annotationPlotWidget2 import pointPlotWidget, linePlotWidget
 
 from pymapmanager._logger import logger
+from pymapmanager.interface.stackWidgets.base.rightCheckBoxAction import RightCheckBoxAction
 
 class ImagePlotWidget(mmWidget2):
     """A plot widget (pg.PlotWidget) to plot

src/pymapmanager/interface/stackWidgets/stackWidget.py

@@ -133,13 +133,18 @@ def closeEvent(self, event):
         Parameters
         ----------
         event : PyQt5.QtGui.QCloseEvent
-        """
-        logger.warning('NEED TO CHECK IF spine/segment IS DIRTY AND PROMPT TO SAVE')
-        
+        """ 
         # logger.info(self.geometry())
         # temp = len(self._openPluginDict)
         # logger.info(f"temp {temp}")
 
+        # check dirty
+        if not self.getDirty():
+            pluginObjList = list(self._openPluginDict.values())
+            for pluginObj in pluginObjList:
+                self.closePluginInDict(pluginObj)
+            return
+
         # check if openPluginDict is not empty
         if len(self._openPluginDict) > 0:
             # open warning before closing
@@ -154,10 +159,6 @@ def closeEvent(self, event):
             msg_box.setIcon(QtWidgets.QMessageBox.Warning)  # or QMessageBox.Information, QMessageBox.Critical, etc.
 
             # Set the buttons
-
-            # Save Don't Save Cancel
-            discard = "Don't Save"
-       
             msg_box.setStandardButtons(QtWidgets.QMessageBox.Save| QtWidgets.QMessageBox.Discard | QtWidgets.QMessageBox.Cancel)
             msg_box.setDefaultButton(QtWidgets.QMessageBox.Cancel)  # Set the default button
 

src/pymapmanager/tests/interface/runInterfaceJohnson.py

@@ -54,7 +54,9 @@ def run():
     logger.error(f"C:\\Users\\johns\\Documents\\GitHub\\MapManagerCore-Data\\data\\202504\\single_timepoint_202504.mmap")
 
     # abb 202504, this is what you were loading
-    path = 'C:\\Users\\johns\\Documents\\GitHub\\MapManagerCore-Data\\data\\single_timepoint.mmap'
+    # path = 'C:\\Users\\johns\\Documents\\GitHub\\MapManagerCore-Data\\data\\single_timepoint.mmap'
+    # path = "C:\\Users\\johns\\Documents\\GitHub\\MapManagerCore-Data\\data\\202504\\single_timepoint_202504.mmap"
+    path = "C:\\Users\\johns\\Documents\\GitHub\\MapManagerCore-Data\\data\\202504\\jcopy_single_timepoint.mmap"
 
     # path = '/Users/johns/Documents/GitHub/MapManagerCore/data/rr30a_s0u.mmap'
 
@@ -320,11 +322,11 @@ def testingGetValues():
     print("testGetValues: ", test)
 
 if __name__ == '__main__':
-    # run()
+    run()
     # testingGetValues()
 
     # TODO: fix merging for just tif, only works for zarr
-    run_tif()
+    # run_tif()
     # run_2_tifs()
 
     # runThenLoad()