r1580

openlp/core/lib/__init__.py

@@ -36,58 +36,6 @@ from PyQt4 import QtCore, QtGui
 
 log = logging.getLogger(__name__)
 
-base_html_expands = []
-
-# Hex Color tags from http://www.w3schools.com/html/html_colornames.asp
-base_html_expands.append({u'desc': u'Red', u'start tag': u'{r}',
-    u'start html': u'<span style="-webkit-text-fill-color:red">',
-    u'end tag': u'{/r}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Black', u'start tag': u'{b}',
-    u'start html': u'<span style="-webkit-text-fill-color:black">',
-    u'end tag': u'{/b}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Blue', u'start tag': u'{bl}',
-    u'start html': u'<span style="-webkit-text-fill-color:blue">',
-    u'end tag': u'{/bl}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Yellow', u'start tag': u'{y}',
-    u'start html': u'<span style="-webkit-text-fill-color:yellow">',
-    u'end tag': u'{/y}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Green', u'start tag': u'{g}',
-    u'start html': u'<span style="-webkit-text-fill-color:green">',
-    u'end tag': u'{/g}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Pink', u'start tag': u'{pk}',
-    u'start html': u'<span style="-webkit-text-fill-color:#FFC0CB">',
-    u'end tag': u'{/pk}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Orange', u'start tag': u'{o}',
-    u'start html': u'<span style="-webkit-text-fill-color:#FFA500">',
-    u'end tag': u'{/o}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Purple', u'start tag': u'{pp}',
-    u'start html': u'<span style="-webkit-text-fill-color:#800080">',
-    u'end tag': u'{/pp}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'White', u'start tag': u'{w}',
-    u'start html': u'<span style="-webkit-text-fill-color:white">',
-    u'end tag': u'{/w}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Superscript', u'start tag': u'{su}',
-    u'start html': u'<sup>', u'end tag': u'{/su}', u'end html': u'</sup>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Subscript', u'start tag': u'{sb}',
-    u'start html': u'<sub>', u'end tag': u'{/sb}', u'end html': u'</sub>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Paragraph', u'start tag': u'{p}',
-    u'start html': u'<p>', u'end tag': u'{/p}', u'end html': u'</p>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Bold', u'start tag': u'{st}',
-    u'start html': u'<strong>', u'end tag': u'{/st}', u'end html': u'</strong>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Italics', u'start tag': u'{it}',
-    u'start html': u'<em>', u'end tag': u'{/it}', u'end html': u'</em>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Underline', u'start tag': u'{u}',
-    u'start html': u'<span style="text-decoration: underline;">',
-    u'end tag': u'{/u}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Break', u'start tag': u'{br}',
-    u'start html': u'<br>', u'end tag': u'', u'end html': u'',
-    u'protected': True})
-
 def translate(context, text, comment=None,
     encoding=QtCore.QCoreApplication.CodecForTr, n=-1,
     translate=QtCore.QCoreApplication.translate):

openlp/core/lib/displaytags.py

@@ -28,7 +28,7 @@
 Provide Html Tag management and Display Tag access class
 """
 
-from openlp.core.lib import base_html_expands
+from openlp.core.lib import translate
 
 class DisplayTags(object):
     """
@@ -50,15 +50,77 @@ class DisplayTags(object):
         Resets the html_expands list.
         """
         DisplayTags.html_expands = []
-        for html in base_html_expands:
+        base_tags = []
-            DisplayTags.html_expands.append(html)
+        # Append the base tags.
+        # Hex Color tags from http://www.w3schools.com/html/html_colornames.asp
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Red'),
+            u'start tag': u'{r}',
+            u'start html': u'<span style="-webkit-text-fill-color:red">',
+            u'end tag': u'{/r}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc':  translate('OpenLP.DisplayTags', 'Black'),
+            u'start tag': u'{b}',
+            u'start html': u'<span style="-webkit-text-fill-color:black">',
+            u'end tag': u'{/b}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Blue'),
+            u'start tag': u'{bl}',
+            u'start html': u'<span style="-webkit-text-fill-color:blue">',
+            u'end tag': u'{/bl}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Yellow'),
+            u'start tag': u'{y}',
+            u'start html': u'<span style="-webkit-text-fill-color:yellow">',
+            u'end tag': u'{/y}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Green'),
+            u'start tag': u'{g}',
+            u'start html': u'<span style="-webkit-text-fill-color:green">',
+            u'end tag': u'{/g}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Pink'),
+            u'start tag': u'{pk}',
+            u'start html': u'<span style="-webkit-text-fill-color:#FFC0CB">',
+            u'end tag': u'{/pk}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Orange'),
+            u'start tag': u'{o}',
+            u'start html': u'<span style="-webkit-text-fill-color:#FFA500">',
+            u'end tag': u'{/o}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Purple'),
+            u'start tag': u'{pp}',
+            u'start html': u'<span style="-webkit-text-fill-color:#800080">',
+            u'end tag': u'{/pp}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'White'),
+            u'start tag': u'{w}',
+            u'start html': u'<span style="-webkit-text-fill-color:white">',
+            u'end tag': u'{/w}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({
+            u'desc': translate('OpenLP.DisplayTags', 'Superscript'),
+            u'start tag': u'{su}', u'start html': u'<sup>',
+            u'end tag': u'{/su}', u'end html': u'</sup>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Subscript'),
+            u'start tag': u'{sb}', u'start html': u'<sub>',
+            u'end tag': u'{/sb}', u'end html': u'</sub>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Paragraph'),
+            u'start tag': u'{p}', u'start html': u'<p>', u'end tag': u'{/p}',
+            u'end html': u'</p>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Bold'),
+            u'start tag': u'{st}', u'start html': u'<strong>',
+            u'end tag': u'{/st}', u'end html': u'</strong>',
+            u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Italics'),
+            u'start tag': u'{it}', u'start html': u'<em>', u'end tag': u'{/it}',
+            u'end html': u'</em>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Underline'),
+            u'start tag': u'{u}',
+            u'start html': u'<span style="text-decoration: underline;">',
+            u'end tag': u'{/u}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Break'),
+            u'start tag': u'{br}', u'start html': u'<br>', u'end tag': u'',
+            u'end html': u'', u'protected': True})
+        DisplayTags.add_html_tags(base_tags)
 
     @staticmethod
-    def add_html_tag(tag):
+    def add_html_tags(tags):
         """
-        Add a new tag to the list
+        Add a list of tags to the list
         """
-        DisplayTags.html_expands.append(tag)
+        DisplayTags.html_expands.extend(tags)
 
     @staticmethod
     def remove_html_tag(tag_id):

openlp/core/lib/serviceitem.py

@@ -177,10 +177,11 @@ class ServiceItem(object):
                     .format_slide(slide[u'raw_slide'], line_break, self)
                 for page in formatted:
                     page = page.replace(u'<br>', u'{br}')
+                    html = expand_tags(cgi.escape(page.rstrip()))
                     self._display_frames.append({
                         u'title': clean_tags(page),
                         u'text': clean_tags(page.rstrip()),
-                        u'html': expand_tags(cgi.escape(page.rstrip())),
+                        u'html': html.replace(u'&amp;nbsp;', u'&nbsp;'),
                         u'verseTag': slide[u'verseTag']
                     })
         elif self.service_item_type == ServiceItemType.Image or \

openlp/core/ui/displaytagdialog.py

@@ -69,9 +69,6 @@ class Ui_DisplayTagDialog(object):
         spacerItem = QtGui.QSpacerItem(40, 20,
             QtGui.QSizePolicy.Expanding, QtGui.QSizePolicy.Minimum)
         self.horizontalLayout.addItem(spacerItem)
-        self.defaultPushButton = QtGui.QPushButton(self.widget)
-        self.defaultPushButton.setObjectName(u'defaultPushButton')
-        self.horizontalLayout.addWidget(self.defaultPushButton)
         self.deletePushButton = QtGui.QPushButton(self.widget)
         self.deletePushButton.setObjectName(u'deletePushButton')
         self.horizontalLayout.addWidget(self.deletePushButton)
@@ -141,8 +138,6 @@ class Ui_DisplayTagDialog(object):
         self.endTagLabel.setText(
             translate('OpenLP.DisplayTagDialog', 'End tag'))
         self.deletePushButton.setText(UiStrings().Delete)
-        self.defaultPushButton.setText(
-            translate('OpenLP.DisplayTagDialog', 'Default'))
         self.newPushButton.setText(UiStrings().New)
         self.tagTableWidget.horizontalHeaderItem(0).setText(
             translate('OpenLP.DisplayTagDialog', 'Description'))

openlp/core/ui/displaytagform.py

@@ -51,8 +51,6 @@ class DisplayTagForm(QtGui.QDialog, Ui_DisplayTagDialog):
         self._loadDisplayTags()
         QtCore.QObject.connect(self.tagTableWidget,
             QtCore.SIGNAL(u'clicked(QModelIndex)'), self.onRowSelected)
-        QtCore.QObject.connect(self.defaultPushButton,
-            QtCore.SIGNAL(u'pressed()'), self.onDefaultPushed)
         QtCore.QObject.connect(self.newPushButton,
             QtCore.SIGNAL(u'pressed()'), self.onNewPushed)
         QtCore.QObject.connect(self.savePushButton,
@@ -87,8 +85,7 @@ class DisplayTagForm(QtGui.QDialog, Ui_DisplayTagDialog):
         if user_expands_string:
             user_tags = cPickle.loads(user_expands_string)
             # If we have some user ones added them as well
-            for t in user_tags:
+            DisplayTags.add_html_tags(user_tags)
-                DisplayTags.add_html_tag(t)
 
     def onRowSelected(self):
         """
@@ -128,22 +125,20 @@ class DisplayTagForm(QtGui.QDialog, Ui_DisplayTagDialog):
                     'Tag "n" already defined.'))
                 return
         # Add new tag to list
-        tag = {u'desc': u'New Item', u'start tag': u'{n}',
+        tag = {
-            u'start html': u'<Html_here>', u'end tag': u'{/n}',
+            u'desc': translate('OpenLP.DisplayTagTab', 'New Tag'),
-            u'end html': u'</and here>', u'protected': False}
+            u'start tag': u'{n}',
-        DisplayTags.add_html_tag(tag)
+            u'start html': translate('OpenLP.DisplayTagTab', '<Html_here>'),
+            u'end tag': u'{/n}',
+            u'end html': translate('OpenLP.DisplayTagTab', '</and here>'),
+            u'protected': False
+        }
+        DisplayTags.add_html_tags([tag])
         self._resetTable()
         # Highlight new row
         self.tagTableWidget.selectRow(self.tagTableWidget.rowCount() - 1)
         self.onRowSelected()
 
-    def onDefaultPushed(self):
-        """
-        Remove all Custom Tags and reset to base set only.
-        """
-        DisplayTags.reset_html_tags()
-        self._resetTable()
-
     def onDeletePushed(self):
         """
         Delete selected custom tag.

openlp/plugins/bibles/lib/mediaitem.py

@@ -775,6 +775,7 @@ class BibleMediaItem(MediaManagerItem):
             # We have to be 'Continuous'.
             else:
                 bible_text = u'%s %s %s\n' % (bible_text, verse_text, text)
+            bible_text = bible_text.strip(u' ')
             if not old_item:
                 start_item = bitem
             elif self.checkTitle(bitem, old_item):

openlp/plugins/custom/forms/editcustomform.py

@@ -116,7 +116,7 @@ class EditCustomForm(QtGui.QDialog, Ui_CustomEditDialog):
         log.debug(u'accept')
         if self.saveCustom():
             Receiver.send_message(u'custom_set_autoselect_item',
-                self.customSlide.title)
+                self.customSlide.id)
             Receiver.send_message(u'custom_load_list')
             QtGui.QDialog.accept(self)
 

openlp/plugins/custom/lib/mediaitem.py

@@ -147,7 +147,7 @@ class CustomMediaItem(MediaManagerItem):
                 QtCore.Qt.UserRole, QtCore.QVariant(custom_slide.id))
             self.listView.addItem(custom_name)
             # Auto-select the item if name has been set
-            if custom_slide.title == self.autoSelectItem:
+            if custom_slide.id == self.autoSelectItem:
                 self.listView.setCurrentItem(custom_name)
 
     def onNewClick(self):

openlp/plugins/songs/forms/editsongform.py

@@ -696,9 +696,9 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
         self.clearCaches()
         if self._validate_song():
             self.saveSong()
-            Receiver.send_message(u'songs_set_autoselect_item',
+            Receiver.send_message(u'songs_set_autoselect_item',self.song.id)
-                unicode(self.titleEdit.text()))
             Receiver.send_message(u'songs_load_list')
+            self.song = None
             QtGui.QDialog.accept(self)
 
     def saveSong(self, preview=False):
@@ -756,8 +756,6 @@ class EditSongForm(QtGui.QDialog, Ui_EditSongDialog):
             self.song.topics.append(self.manager.get_object(Topic, topicId))
         clean_song(self.manager, self.song)
         self.manager.save_object(self.song)
-        if not preview:
-            self.song = None
 
     def _processLyrics(self):
         """

openlp/plugins/songs/lib/mediaitem.py

@@ -245,7 +245,7 @@ class SongMediaItem(MediaManagerItem):
             song_name.setData(QtCore.Qt.UserRole, QtCore.QVariant(song.id))
             self.listView.addItem(song_name)
             # Auto-select the item if name has been set
-            if song.title == self.autoSelectItem :
+            if song.id == self.autoSelectItem :
                 self.listView.setCurrentItem(song_name)
 
     def displayResultsAuthor(self, searchresults):