r15803 by jghali - #9476 : not appearing guide issues when applying masterpages with guides

[scribus-commit]

Author: jghali
Date: Sun Nov  7 16:38:34 2010
New Revision: 15803

URL: http://scribus.info/websvn/listing.php?repname=Scribus&sc=1&rev=15803
Log:
#9476 : not appearing guide issues when applying masterpages with guides

Modified:
    trunk/Scribus/scribus/scribus.cpp
    trunk/Scribus/scribus/scribusdoc.cpp
    trunk/Scribus/scribus/ui/guidemanager.cpp
    trunk/Scribus/scribus/ui/guidemanager.h

Modified: trunk/Scribus/scribus/scribus.cpp
URL: http://scribus.info/websvn/diff.php?repname=Scribus&rev=15803&path=/trunk/Scribus/scribus/scribus.cpp
==============================================================================
--- trunk/Scribus/scribus/scribus.cpp (original)
+++ trunk/Scribus/scribus/scribus.cpp Sun Nov  7 16:38:34 2010
@@ -8578,7 +8578,10 @@
 	//CB done by doc::reformpages
 	//slotDocCh();
 	pagePalette->Rebuild();
-	guidePalette->setupPage();
+	// #9476 : call setupPage with false arg to setup only guidePalette GUI
+	// Otherwise setupPage() will apply guides to current page, doesn't need that, 
+	// Apply_MasterPage() has already done it
+	guidePalette->setupPage(false);
 	delete dia;
 }
 

Modified: trunk/Scribus/scribus/scribusdoc.cpp
URL: http://scribus.info/websvn/diff.php?repname=Scribus&rev=15803&path=/trunk/Scribus/scribus/scribusdoc.cpp
==============================================================================
--- trunk/Scribus/scribus/scribusdoc.cpp (original)
+++ trunk/Scribus/scribus/scribusdoc.cpp Sun Nov  7 16:38:34 2010
@@ -10369,7 +10369,9 @@
 		m_ScMW->guidePalette->setDoc(this);
 		if (!isLoading())
 		{
-			m_ScMW->guidePalette->setupPage();
+			// #9476 : call setupPage with false arg to setup only guidePalette GUI
+			// Otherwise expect problems when applying masterpages with guides
+			m_ScMW->guidePalette->setupPage(false);
 			m_ScMW->pagePalette->markPage(newPage->pageNr());
 		}
 	}

Modified: trunk/Scribus/scribus/ui/guidemanager.cpp
URL: http://scribus.info/websvn/diff.php?repname=Scribus&rev=15803&path=/trunk/Scribus/scribus/ui/guidemanager.cpp
==============================================================================
--- trunk/Scribus/scribus/ui/guidemanager.cpp (original)
+++ trunk/Scribus/scribus/ui/guidemanager.cpp Sun Nov  7 16:38:34 2010
@@ -124,14 +124,14 @@
 	tabWidget->setEnabled(doc ? true : false);
 }
 
-void GuideManager::setupPage()
+void GuideManager::setupPage(bool storeValues)
 {
 	if (!m_Doc)
 		return;
 	m_drawGuides = false;
 	setEnabled(true);
 	// store old values for current page (=page to leave)
-	if (currentPage)
+	if (currentPage && storeValues)
 		storePageValues(currentPage);
 
 	currentPage = m_Doc->currentPage();

Modified: trunk/Scribus/scribus/ui/guidemanager.h
URL: http://scribus.info/websvn/diff.php?repname=Scribus&rev=15803&path=/trunk/Scribus/scribus/ui/guidemanager.h
==============================================================================
--- trunk/Scribus/scribus/ui/guidemanager.h (original)
+++ trunk/Scribus/scribus/ui/guidemanager.h Sun Nov  7 16:38:34 2010
@@ -42,7 +42,7 @@
 
 	/*! \brief Set the widgets on the page change.
 	It has to be called on every page to page transition */
-	void setupPage();
+	void setupPage(bool storeValues = true);
 	void setupGui();
 
 	/*! \brief Clear and reset the GUI horizontal list widget. */