Updated documentation in WorkOrder and the Project Fixes #5113 (e9eaafbb) · Commits · aflab / astrogeology / ISIS3

isis/src/qisis/objs/Directory/Directory.cpp

+14 −5

Original line number	Diff line number	Diff line
		@@ -948,6 +948,7 @@ namespace Isis {
		return;
		}
		m_bundleObservationViews.removeAll(bundleObservationView);
		m_project->setClean(false);
		}


		@@ -969,6 +970,7 @@ namespace Isis {
		}

		m_cnetEditorViewWidgets.removeAll(cnetEditorWidget);
		m_project->setClean(false);
		}


		@@ -982,6 +984,7 @@ namespace Isis {
		return;
		}
		m_cubeDnViewWidgets.removeAll(cubeDnView);
		m_project->setClean(false);
		}


		@@ -995,6 +998,7 @@ namespace Isis {
		return;
		}
		m_fileListWidgets.removeAll(imageFileListWidget);
		m_project->setClean(false);
		}


		@@ -1008,6 +1012,7 @@ namespace Isis {
		return;
		}
		m_footprint2DViewWidgets.removeAll(footprintView);
		m_project->setClean(false);
		}


		@@ -1020,6 +1025,7 @@ namespace Isis {
		if (!controlPointEditView) {
		return;
		}
		m_project->setClean(false);
		// For now delete the ChipViewportsWidget also, which must be done first since it is a child
		// of ControlPointEditView
		// delete m_chipViewports;
		@@ -1037,6 +1043,7 @@ namespace Isis {
		return;
		}
		m_matrixViewWidgets.removeAll(matrixWidget);
		m_project->setClean(false);
		}


		@@ -1050,6 +1057,7 @@ namespace Isis {
		return;
		}
		m_sensorInfoWidgets.removeAll(sensorInfoWidget);
		m_project->setClean(false);
		}


		@@ -1063,6 +1071,7 @@ namespace Isis {
		return;
		}
		m_targetInfoWidgets.removeAll(targetInfoWidget);
		m_project->setClean(false);
		}

+3 −0

Original line number	Diff line number	Diff line
		@@ -174,6 +174,9 @@ namespace Isis {
		* needed at the moment Fixes #5026
		* @history 2017-08-11 Christopher Combs - Added serialization of CnetEditorWidgets to save()
		* and startElement(). Fixes #4989.
		* @history 2017-08-11 Cole Neubauer - Added project setClean(false) call to all views cleanup
		* slot. This will make a a view closing be treated as a project change
		* Fixes #5113
		*/
		class Directory : public QObject {
		Q_OBJECT

+5 −2

Original line number	Diff line number	Diff line
		@@ -1236,7 +1236,8 @@ namespace Isis {


		/**
		* Accessor to determine whether the curren project is Unsaved
		* Accessor to determine whether the current project is Unsaved. This is used to determine how the
		* program should react with things like opening a new project or closing he window.
		*/
		bool Project::isClean() {
		return m_isClean;
		@@ -1244,7 +1245,9 @@ namespace Isis {


		/**
		* Function to change the clean state of the project
		* Function to change the clean state of the project. This is needed because not every action that
		* changes the project is a workorder. This needs to be called everytime a user would consider an
		* action changing the project. This also explicitely sets the undoStack to the same value passed.
		* @param the boolean value to set the clean state to
		*/
		void Project::setClean(bool value) {

+1 −0

Original line number	Diff line number	Diff line
		@@ -169,6 +169,7 @@ namespace Isis {
		* ImageList Fixes #4567
		* @history 2017-08-11 Cole Neubauer - Removed unnecessary code in controlClosed that was
		* a segfault causing. Fixes #5064
		* @history 2017-08-11 Cole Neubauer - Updated documentation for setClean and isClean #5113
		*/
		class Project : public QObject {
		Q_OBJECT

+8 −7

Original line number	Diff line number	Diff line
		@@ -341,6 +341,7 @@ namespace Isis {
		projectItem->setText(newName);
		}
		}
		project->setClean(false);
		}


		@@ -729,8 +730,8 @@ namespace Isis {
		item->setText(name);
		item->shapeList()->setName(name);
		}
		return true;

		return true;
		}