[gedit-latex] bibtex: style cleanup in validator



commit b6194edc039535f689884cdbee0fc9e715594e78
Author: Ignacio Casal Quinteiro <icq gnome org>
Date:   Thu Oct 6 12:48:07 2011 +0200

    bibtex: style cleanup in validator

 latex/bibtex/validator.py |   21 +++++++++++----------
 1 files changed, 11 insertions(+), 10 deletions(-)
---
diff --git a/latex/bibtex/validator.py b/latex/bibtex/validator.py
index 6681d56..43b7ecd 100644
--- a/latex/bibtex/validator.py
+++ b/latex/bibtex/validator.py
@@ -52,7 +52,9 @@ class BibTeXValidator:
         for entry in document.entries:
             # check for duplicate keys
             if entry.key in entry_keys:
-                issue_handler.issue(Issue("Duplicate key <b>%s</b>" % entry.key, entry.start, entry.end, file, Issue.SEVERITY_ERROR))
+                issue_handler.issue(Issue("Duplicate key <b>%s</b>" % entry.key,
+                                          entry.start, entry.end, file,
+                                          Issue.SEVERITY_ERROR))
             else:
                 entry_keys.append(entry.key)
 
@@ -60,7 +62,8 @@ class BibTeXValidator:
             for field in entry.fields:
                 # check for duplicate fields
                 if field.name in field_names:
-                    issue_handler.issue(Issue("Duplicate field <b>%s</b>" % field.name, entry.start, entry.end, file, Issue.SEVERITY_ERROR))
+                    issue_handler.issue(Issue("Duplicate field <b>%s</b>" % field.name,
+                                        entry.start, entry.end, file, Issue.SEVERITY_ERROR))
                 else:
                     field_names.append(field.name)
 
@@ -68,22 +71,20 @@ class BibTeXValidator:
                 # check for missing required fields
                 required_field_names = set(map(lambda f: f.name, self._model.find_type(entry.type).required_fields))
                 missing_field_names = required_field_names.difference(set(field_names))
+
                 if len(missing_field_names) > 0:
-                    issue_handler.issue(Issue("Possibly missing field(s): <b>%s</b>" % ",".join(missing_field_names), entry.start, entry.end, file, Issue.SEVERITY_WARNING))
+                    issue_handler.issue(Issue("Possibly missing field(s): <b>%s</b>" % ",".join(missing_field_names),
+                                              entry.start, entry.end, file, Issue.SEVERITY_WARNING))
 
                 # check for unused fields
                 optional_field_names = set(map(lambda f: f.name, self._model.find_type(entry.type).optional_fields))
                 unused_field_names = set(field_names).difference(optional_field_names.union(required_field_names))
+
                 if len(unused_field_names) > 0:
-                    issue_handler.issue(Issue("Possibly unused field(s): <b>%s</b>" % ",".join(unused_field_names), entry.start, entry.end, file, Issue.SEVERITY_WARNING))
+                    issue_handler.issue(Issue("Possibly unused field(s): <b>%s</b>" % ",".join(unused_field_names),
+                                              entry.start, entry.end, file, Issue.SEVERITY_WARNING))
             except KeyError:
                 #self._log.debug("Type not found: %s" % entry.type)
                 pass
 
-
-
-
-
-
-
 # ex:ts=4:et:



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]