Merge branch 'fix-414-merge-issue' into 'master'

Fix the CI error after merging !414 and !417

See merge request openlp/openlp!418

openlp/core/display/render.py

@@ -157,7 +157,7 @@ def remove_tags(text, can_remove_chords=False):
     text = text.replace('<em>', '')
     text = text.replace('</em>', '')
     for tag in FormattingTags.get_html_tags():
-        if tag['hidden']:
+        if tag.get('hidden'):
             text = re.sub(r'' + tag['start tag'] + ".*?" + tag['end tag'], '', text)
         else:
             text = text.replace(tag['start tag'], '')

openlp/core/lib/formattingtags.py

@@ -63,99 +63,173 @@ class FormattingTags(object):
                 'desc': translate('OpenLP.FormattingTags', 'Red'),
                 'start tag': '{r}',
                 'start html': '<span style="-webkit-text-fill-color:red">',
-                'end tag': '{/r}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/r}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Black'),
                 'start tag': '{b}',
                 'start html': '<span style="-webkit-text-fill-color:black">',
-                'end tag': '{/b}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/b}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Blue'),
                 'start tag': '{bl}',
                 'start html': '<span style="-webkit-text-fill-color:blue">',
-                'end tag': '{/bl}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/bl}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Yellow'),
                 'start tag': '{y}',
                 'start html': '<span style="-webkit-text-fill-color:yellow">',
-                'end tag': '{/y}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/y}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Green'),
                 'start tag': '{g}',
                 'start html': '<span style="-webkit-text-fill-color:green">',
-                'end tag': '{/g}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/g}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Pink'),
                 'start tag': '{pk}',
                 'start html': '<span style="-webkit-text-fill-color:#FFC0CB">',
-                'end tag': '{/pk}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/pk}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Orange'),
                 'start tag': '{o}',
                 'start html': '<span style="-webkit-text-fill-color:#FFA500">',
-                'end tag': '{/o}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/o}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Purple'),
                 'start tag': '{pp}',
                 'start html': '<span style="-webkit-text-fill-color:#800080">',
-                'end tag': '{/pp}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/pp}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'White'),
                 'start tag': '{w}',
                 'start html': '<span style="-webkit-text-fill-color:white">',
-                'end tag': '{/w}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/w}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
-                'desc': translate('OpenLP.FormattingTags', 'Superscript'),
+                'temporary': False,
-                'start tag': '{su}', 'start html': '<sup>',
-                'end tag': '{/su}', 'end html': '</sup>', 'protected': True,
-                'temporary': False, 'hidden': False
-            }, {
-                'desc': translate('OpenLP.FormattingTags', 'Subscript'),
-                'start tag': '{sb}', 'start html': '<sub>',
-                'end tag': '{/sb}', 'end html': '</sub>', 'protected': True,
-                'temporary': False, 'hidden': False
-            }, {
-                'desc': translate('OpenLP.FormattingTags', 'Paragraph'),
-                'start tag': '{p}', 'start html': '<p>', 'end tag': '{/p}',
-                'end html': '</p>', 'protected': True,
-                'temporary': False, 'hidden': False
-            }, {
-                'desc': translate('OpenLP.FormattingTags', 'Bold'),
-                'start tag': '{st}', 'start html': '<strong>',
-                'end tag': '{/st}', 'end html': '</strong>',
-                'protected': True, 'temporary': False, 'hidden': False
-            }, {
-                'desc': translate('OpenLP.FormattingTags', 'Italics'),
-                'start tag': '{it}', 'start html': '<em>', 'end tag': '{/it}',
-                'end html': '</em>', 'protected': True, 'temporary': False,
                 'hidden': False
-            }, {
+            },
+            {
+                'desc': translate('OpenLP.FormattingTags', 'Superscript'),
+                'start tag': '{su}',
+                'start html': '<sup>',
+                'end tag': '{/su}',
+                'end html': '</sup>',
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
+                'desc': translate('OpenLP.FormattingTags', 'Subscript'),
+                'start tag': '{sb}',
+                'start html': '<sub>',
+                'end tag': '{/sb}',
+                'end html': '</sub>',
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
+                'desc': translate('OpenLP.FormattingTags', 'Paragraph'),
+                'start tag': '{p}',
+                'start html': '<p>',
+                'end tag': '{/p}',
+                'end html': '</p>',
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
+                'desc': translate('OpenLP.FormattingTags', 'Bold'),
+                'start tag': '{st}',
+                'start html': '<strong>',
+                'end tag': '{/st}',
+                'end html': '</strong>',
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
+                'desc': translate('OpenLP.FormattingTags', 'Italics'),
+                'start tag': '{it}',
+                'start html': '<em>',
+                'end tag': '{/it}',
+                'end html': '</em>',
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Underline'),
                 'start tag': '{u}',
                 'start html': '<span style="text-decoration: underline;">',
-                'end tag': '{/u}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/u}',
-                'temporary': False, 'hidden': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'UpperCase'),
                 'start tag': '{uc}',
                 'start html': '<span style="text-transform: uppercase;">',
-                'end tag': '{/uc}', 'end html': '</span>', 'protected': True,
+                'end tag': '{/uc}',
-                'temporary': False
+                'end html': '</span>',
-            }, {
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            },
+            {
                 'desc': translate('OpenLP.FormattingTags', 'Break'),
-                'start tag': '{br}', 'start html': '<br>', 'end tag': '',
+                'start tag': '{br}',
-                'end html': '', 'protected': True,
+                'start html': '<br>',
-                'temporary': False, 'hidden': False
+                'end tag': '',
-            }]
+                'end html': '',
+                'protected': True,
+                'temporary': False,
+                'hidden': False
+            }
+        ]
         # Append the base tags.
         FormattingTags.add_html_tags(base_tags)
         FormattingTags.add_html_tags(temporary_tags)