pylint and coherency on Repository references

svn: r11799

src/plugins/quickview/Reporef.py

@@ -1,3 +1,5 @@
+#!/usr/bin/python
+# -*- coding: utf-8 -*-
 #
 # Gramps - a GTK+/GNOME based genealogy program
 #
@@ -19,58 +21,71 @@
 # Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 #
 
+# -------------------------------------------------------------------------
+#
+# gramps modules
+#
+# -------------------------------------------------------------------------
+
 """
 Display RepoRef for sources related to active repository
 """
 
-#-------------------------------------------------------------------------
-#
-# gramps modules
-#
-#-------------------------------------------------------------------------
-
-from Simple import SimpleAccess, SimpleDoc, SimpleTable
+from Simple import SimpleDoc
 from gettext import gettext as _
 from gen.plug import PluginManager
 from ReportBase import CATEGORY_QR_REPOSITORY
 import gen.lib
 
+
 def run(database, document, repo):
 
-    sa = SimpleAccess(database)
     sdoc = SimpleDoc(document)
+
     # First we find repository and add its text
-    sdoc.title("%s\n" % repo.get_name())
-    # Go over all the sources that refer to this repository                  
+
+    sdoc.title('%s\n' % repo.get_name())
+
+    # Go over all the sources that refer to this repository
+
     repo_handle = repo.handle
-    source_list = [item[1] for item in database.find_backlink_handles(repo_handle,['Source'])]
-    
-    for source_handle in source_list :
+    source_list = [item[1] for item in
+                   database.find_backlink_handles(repo_handle, ['Source'
+                   ])]
+
+    for source_handle in source_list:
         src = database.get_source_from_handle(source_handle)
-        # Get the list of references from this source to our repo
+
+        # Get the list of references from this source to our repo
         # (can be more than one, technically)
+
         for reporef in src.get_reporef_list():
             if reporef.ref == repo_handle:
-                # Determine the text for this source
-                mt = str(reporef.get_media_type())
-                cn = reporef.get_call_number()
-                sdoc.paragraph("%s, %s" % (mt, cn))
-                sdoc.paragraph("%s\n" % src.get_title())    
-            else:
-                continue 
 
-#------------------------------------------------------------------------
+                # Determine the text for this source
+
+                media = str(reporef.get_media_type())
+                call = reporef.get_call_number()
+                sdoc.paragraph('%s, %s' % (media, call))
+                sdoc.paragraph('%s\n' % src.get_title())
+            else:
+                continue
+
+
+# ------------------------------------------------------------------------
 #
 #
 #
-#------------------------------------------------------------------------
+# ------------------------------------------------------------------------
+
 pmgr = PluginManager.get_instance()
-         
+
 pmgr.register_quick_report(
-    name = 'RepoRef', 
-    category = CATEGORY_QR_REPOSITORY, 
-    run_func = run, 
-    translated_name = _("RepoRef"), 
-    status = _("Beta"), 
-    description= _("Display RepoRef for sources related to active repository"), 
+    name='RepoRef',
+    category=CATEGORY_QR_REPOSITORY,
+    run_func=run,
+    translated_name=_('RepoRef'),
+    status=_('Beta'),
+    description=_('Display RepoRef for sources related to active repository'
+                  ),
     )