Clean up gui imports, and some other finagery

gui/slick/views/IRC.mako

 <%inherit file="/layouts/main.mako"/>
 <%block name="content">
 <%
-from sickbeard import GIT_USERNAME
-username = ("SickRageUI|?", GIT_USERNAME)[bool(GIT_USERNAME)]
+import sickbeard
+username = ("SickRageUI|?", sickbeard.GIT_USERNAME)[bool(sickbeard.GIT_USERNAME)]
 %>
 <iframe id="extFrame" src="https://kiwiirc.com/client/irc.freenode.net/?nick=${username}&theme=basic#sickrage-issues" width="100%" height="500" frameBorder="0" style="border: 1px black solid;"></iframe>
 </%block>

gui/slick/views/addShows.mako

 <%inherit file="/layouts/main.mako"/>
-<%!
-    import os.path
-    import urllib
-    import sickbeard
-%>
 <%block name="content">
 % if not header is UNDEFINED:
     <h1 class="header">${header}</h1>

gui/slick/views/addShows_addExistingShow.mako

 <%inherit file="/layouts/main.mako"/>
-<%!
-    import sickbeard
-%>
 <%block name="scripts">
 <script type="text/javascript" src="${srRoot}/js/qualityChooser.js?${sbPID}"></script>
 <script type="text/javascript" src="${srRoot}/js/addShowOptions.js?${sbPID}"></script>

gui/slick/views/addShows_popularShows.mako

@@ -30,7 +30,7 @@
     </select>
 </div>
 
-<% imdb_tt = [show.imdbid for show in sickbeard.showList if show.imdbid] %>
+<% imdb_tt = {show.imdbid for show in sickbeard.showList if show.imdbid} %>
 
 <br>
 <div id="popularShows">

gui/slick/views/config_backuprestore.mako

 <%inherit file="/layouts/main.mako"/>
 <%!
-    import datetime
-    import locale
     import sickbeard
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
-    from sickbeard.common import Quality, qualityPresets, statusStrings, qualityPresetStrings, cpu_presets
-    from sickbeard.sbdatetime import sbdatetime, date_presets, time_presets
-    from sickbeard import config
-    from sickbeard import metadata
-    from sickbeard.metadata.generic import GenericMetadata
 %>
 <%block name="content">
 % if not header is UNDEFINED:

gui/slick/views/config_general.mako

 <%inherit file="/layouts/main.mako"/>
 <%!
     import datetime
-    import locale
     import sickbeard
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
-    from sickbeard.common import Quality, qualityPresets, statusStrings, qualityPresetStrings, cpu_presets
+    from sickbeard.common import SKIPPED, ARCHIVED, IGNORED, statusStrings, cpu_presets
     from sickbeard.sbdatetime import sbdatetime, date_presets, time_presets
-    from sickbeard import config
-    from sickbeard import metadata
-    from sickbeard.metadata.generic import GenericMetadata
     from sickbeard.helpers import anon_url
 %>
 <%block name="scripts">
@@ -21,11 +16,6 @@
     <h1 class="title">${title}</h1>
 % endif
 
-<% indexer = 0 %>
-% if sickbeard.INDEXER_DEFAULT:
-    <% indexer = sickbeard.INDEXER_DEFAULT %>
-% endif
-
 <div id="config">
     <div id="config-content">
 
@@ -142,7 +132,7 @@
                                 <span class="component-title">Use initial indexer set to</span>
                                 <span class="component-desc">
                                     <select id="indexer_default" name="indexer_default" class="form-control input-sm">
-                                        <option value="0" ${('', 'selected="selected"')[indexer == 0]}>All Indexers</option>
+                                        <option value="0" ${('', 'selected="selected"')[sickbeard.INDEXER_DEFAULT == 0]}>All Indexers</option>
                                         % for indexer in sickbeard.indexerApi().indexers:
                                         <option value="${indexer}" ${('', 'selected="selected"')[sickbeard.INDEXER_DEFAULT == indexer]}>${sickbeard.indexerApi().indexers[indexer]}</option>
                                         % endfor

gui/slick/views/config_notifications.mako

@@ -3,8 +3,6 @@
     import sickbeard
     import re
     from sickbeard.helpers import anon_url
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
-    from sickbeard.common import Quality, qualityPresets, statusStrings, qualityPresetStrings, cpu_presets, MULTI_EP_STRINGS
 %>
 <%block name="content">
 % if not header is UNDEFINED:

gui/slick/views/config_postProcessing.mako

@@ -3,11 +3,7 @@
     import os.path
     import datetime
     import sickbeard
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
-    from sickbeard.common import Quality, qualityPresets, statusStrings, qualityPresetStrings, cpu_presets, MULTI_EP_STRINGS
-    from sickbeard import config
-    from sickbeard import metadata
-    from sickbeard.metadata.generic import GenericMetadata
+    from sickbeard.common import MULTI_EP_STRINGS
     from sickbeard import naming
     from sickrage.helper.encoding import ek
 %>
@@ -1080,9 +1076,8 @@
                             <label>
                                 <span class="component-title">Metadata Type:</span>
                                 <span class="component-desc">
-                                    <% m_dict = metadata.get_metadata_generator_dict() %>
                                     <select id="metadataType" class="form-control input-sm">
-                                    % for (cur_name, cur_generator) in sorted(m_dict.iteritems()):
+                                    % for (cur_name, cur_generator) in sorted(sickbeard.metadata_provider_dict.iteritems()):
                                         <option value="${cur_generator.get_id()}">${cur_name}</option>
                                     % endfor
                                     </select>
@@ -1091,9 +1086,11 @@
                             <span>Toggle the metadata options that you wish to be created. <b>Multiple targets may be used.</b></span>
                         </div>
 
-                        % for (cur_name, cur_generator) in m_dict.iteritems():
-                        <% cur_metadata_inst = sickbeard.metadata_provider_dict[cur_generator.name] %>
-                        <% cur_id = cur_generator.get_id() %>
+                        % for (cur_name, cur_generator) in sickbeard.metadata_provider_dict.iteritems():
+                        <%
+                            cur_metadata_inst = sickbeard.metadata_provider_dict[cur_generator.name]
+                            cur_id = cur_generator.get_id()
+                        %>
                         <div class="metadataDiv" id="${cur_id}">
                             <div class="metadata_options_wrapper">
                                 <h4>Create:</h4>

gui/slick/views/config_search.mako

 <%inherit file="/layouts/main.mako"/>
 <%!
     import sickbeard
-    from sickbeard import clients
 %>
 <%block name="content">
 % if not header is UNDEFINED:

gui/slick/views/displayShow.mako

@@ -2,7 +2,6 @@
 <%!
     import datetime
     import urllib
-    import ntpath
     import sickbeard
     from sickbeard import subtitles, sbdatetime, network_timezones
     import sickbeard.helpers

gui/slick/views/editShow.mako

 <%inherit file="/layouts/main.mako"/>
 <%!
     import sickbeard
-    import adba
     from sickbeard import common
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
+    from sickbeard.common import SKIPPED, WANTED, IGNORED
     from sickbeard.common import statusStrings
-    from sickrage.helper import exceptions
-    from sickbeard import scene_exceptions
 %>
 
 <%block name="metas">

gui/slick/views/errorlogs.mako

 <%inherit file="/layouts/main.mako"/>
 <%!
     import sickbeard
-    from sickbeard import classes
 %>
 <%block name="css">
 <style>
@@ -15,10 +14,10 @@ pre {
 <%block name="content">
 <%
     if logLevel == sickbeard.logger.WARNING:
-        errors = classes.WarningViewer.errors
+        errors = sickbeard.classes.WarningViewer.errors
         title = 'WARNING logs'
     else:
-        errors = classes.ErrorViewer.errors
+        errors = sickbeard.classes.ErrorViewer.errors
         title = 'ERROR logs'
 %>
 <h1 class="header">${title}</h1>

gui/slick/views/history.mako

@@ -3,14 +3,11 @@
     import sickbeard
     import os.path
     import datetime
-    import re
-    import time
 
-    from sickbeard import providers
-    from sickbeard import sbdatetime
+    from sickbeard import providers, sbdatetime
 
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED, DOWNLOADED, SUBTITLED
-    from sickbeard.common import Quality, statusStrings, Overview
+    from sickbeard.common import ARCHIVED, SNATCHED, FAILED, DOWNLOADED, SUBTITLED
+    from sickbeard.common import Quality, statusStrings
 
     from sickrage.show.History import History
     from sickrage.helper.encoding import ek

gui/slick/views/home.mako

@@ -112,7 +112,7 @@
     download_stat_tip = ''
     display_status = curShow.status
 
-    if None is not display_status:
+    if display_status:
         if re.search(r'(?i)(?:new|returning)\s*series', curShow.status):
             display_status = 'Continuing'
         elif re.search(r'(?i)(?:nded)', curShow.status):

gui/slick/views/inc_addShowOptions.mako

 <%
     import sickbeard
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
-    from sickbeard.common import Quality, qualityPresets, qualityPresetStrings, statusStrings
-    from sickbeard import subtitles
+    from sickbeard.common import SKIPPED, WANTED, IGNORED
+    from sickbeard.common import Quality, statusStrings
 %>
 
         <div class="field-pair alt">

gui/slick/views/inc_defs.mako

 <%!
-    import cgi
     from sickbeard.common import Quality, qualityPresets, qualityPresetStrings
 %>
-<%def name="renderQualityPill(quality, showTitle=False, overrideClass=None)"><%
+<%def name="renderQualityPill(quality, showTitle=False, overrideClass=None)">
+<%
     # Build a string of quality names to use as title attribute
     if showTitle:
         allowed_qualities, preferred_qualities = Quality.splitQuality(quality)
         title = 'Allowed Quality:\n'
-        if allowed_qualities:
-            for curQual in allowed_qualities:
+        for curQual in allowed_qualities or [None]:
             title += "  " + Quality.qualityStrings[curQual] + "\n"
-        else:
-            title += "  None\n"
+
         title += "\nPreferred Quality:\n"
-        if preferred_qualities:
-            for curQual in preferred_qualities:
+        for curQual in preferred_qualities or [None]:
             title += "  " + Quality.qualityStrings[curQual] + "\n"
-        else:
-            title += "  None\n"
-        title = ' title="' + cgi.escape(title.rstrip(), True) + '"'
     else:
         title = ""
 
@@ -77,9 +71,6 @@
         cssClass = "Custom"
         qualityString = "Custom"
 
-    if overrideClass is None:
-        cssClass = "quality " + cssClass
-    else:
-        cssClass = overrideClass
-
-%><span${title} class="${cssClass}">${qualityString}</span></%def>
+    cssClass = overrideClass or "quality " + cssClass
+%>
+<span title="${title | h}" class="${cssClass}">${qualityString}</span></%def>

gui/slick/views/inc_qualityChooser.mako

@@ -4,7 +4,7 @@
 %>
 
 <%
-if not show is UNDEFINED:
+if show is not UNDEFINED:
     __quality = int(show.quality)
 else:
     __quality = int(sickbeard.QUALITY_DEFAULT)

gui/slick/views/layouts/main.mako

 <%!
-    import datetime
     import re
+    import datetime
     import sickbeard
     from sickrage.helper.common import pretty_file_size
     from sickrage.show.Show import Show

gui/slick/views/manage.mako

 <%inherit file="/layouts/main.mako"/>
 <%!
     import sickbeard
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
     from sickbeard.common import statusStrings
 %>
 <%block name="scripts">

gui/slick/views/manage_backlogOverview.mako

 <%inherit file="/layouts/main.mako"/>
 <%!
-    import sickbeard
     import datetime
-    from sickbeard.common import SKIPPED, WANTED, UNAIRED, ARCHIVED, IGNORED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, FAILED
-    from sickbeard.common import Overview, Quality, qualityPresets, qualityPresetStrings
-    from sickrage.helper.common import episode_num
+    import sickbeard
     from sickbeard import sbdatetime, network_timezones
+    from sickbeard.common import WANTED, SNATCHED, SNATCHED_PROPER, SNATCHED_BEST, Overview, Quality
+    from sickrage.helper.common import episode_num
 %>
 <%block name="scripts">
 <script type="text/javascript">