← Back to team overview

openlp-core team mailing list archive

[Merge] lp:~elderp/openlp/advancedtab into lp:openlp

 

Stevan Pettit has proposed merging lp:~elderp/openlp/advancedtab into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)

For more details, see:
https://code.launchpad.net/~elderp/openlp/advancedtab/+merge/115025

Added code to hide the Data Location wizard on the Advanced tab if OpenLP is running portable.

I intended this when coding the portable but left out this piece of code by mistake.
-- 
https://code.launchpad.net/~elderp/openlp/advancedtab/+merge/115025
Your team OpenLP Core is requested to review the proposed merge of lp:~elderp/openlp/advancedtab into lp:openlp.
=== modified file 'openlp/core/ui/advancedtab.py'
--- openlp/core/ui/advancedtab.py	2012-06-22 14:14:53 +0000
+++ openlp/core/ui/advancedtab.py	2012-07-15 12:09:18 +0000
@@ -545,6 +545,10 @@
             self.currentDataPath))
         self.defaultColorButton.setStyleSheet(
             u'background-color: %s' % self.defaultColor)
+        # Don't allow data directory move if running portable.
+        if Settings().value(u'advanced/is portable',
+            QtCore.QVariant(False)).toBool():
+            self.dataDirectoryGroupBox.hide()
 
     def save(self):
         """


Follow ups