diff -r 16905f0f48be -r 01ac1f364b38 Plugins/CheckerPlugins/Pep8/Pep8Fixer.py
--- a/Plugins/CheckerPlugins/Pep8/Pep8Fixer.py Wed Sep 18 19:48:12 2013 +0200
+++ b/Plugins/CheckerPlugins/Pep8/Pep8Fixer.py Wed Sep 18 19:53:54 2013 +0200
@@ -21,8 +21,10 @@
import Utilities
# TODO: add fixers for some docstring issues
+# D111, D112, D113
-Pep8FixableIssues = ["E101", "E111", "E121", "E122", "E123", "E124",
+Pep8FixableIssues = ["D121", "D131", "D141", "D142", "D143", "D144", "D145",
+ "E101", "E111", "E121", "E122", "E123", "E124",
"E125", "E126", "E127", "E128", "E133", "E201",
"E202", "E203", "E211", "E221", "E222", "E223",
"E224", "E225", "E226", "E227", "E228", "E231",
@@ -78,6 +80,13 @@
"fixed_" + os.path.basename(self.__filename))
self.__fixes = {
+ "D121": self.__fixD121,
+ "D131": self.__fixD131,
+ "D141": self.__fixD141,
+ "D142": self.__fixD142,
+ "D143": self.__fixD143,
+ "D144": self.__fixD144,
+ "D145": self.__fixD145,
"E101": self.__fixE101,
"E111": self.__fixE101,
"E121": self.__fixE121,
@@ -1333,6 +1342,211 @@
"""
self.__source[line - 1] = self.__source[line - 1].replace("<>", "!=")
return (1, self.trUtf8("'<>' replaced by '!='."), 0)
+
+ def __fixD121(self, code, line, pos, apply=False):
+ """
+ Private method to fix a single line docstring on multiple lines (D121).
+
+ @param code code of the issue (string)
+ @param line line number of the issue (integer)
+ @param pos position inside line (integer)
+ @keyparam apply flag indicating, that the fix should be applied
+ (boolean)
+ @return value indicating an applied/deferred fix (-1, 0, 1),
+ a message for the fix (string) and an ID for a deferred
+ fix (integer)
+ """
+ if apply:
+ line = line - 1
+ if not self.__source[line].lstrip().startswith(
+ ('"""', 'r"""', 'u"""')):
+ # only correctly formatted docstrings will be fixed
+ return (0, "", 0)
+
+ docstring = self.__source[line].rstrip() + \
+ self.__source[line + 1].strip()
+ if docstring.endswith('"""'):
+ docstring += self.__getEol()
+ else:
+ docstring += self.__source[line + 2].lstrip()
+ self.__source[line + 2] = ""
+
+ self.__source[line] = docstring
+ self.__source[line + 1] = ""
+ return (
+ 1,
+ self.trUtf8("Single line docstring put on one line."),
+ 0)
+ else:
+ id = self.__getID()
+ self.__stack.append((id, code, line, pos))
+ return (-1, "", id)
+
+ def __fixD131(self, code, line, pos):
+ """
+ Private method to fix a single line docstring on multiple lines (D121).
+
+ @param code code of the issue (string)
+ @param line line number of the issue (integer)
+ @param pos position inside line (integer)
+ @return value indicating an applied/deferred fix (-1, 0, 1),
+ a message for the fix (string) and an ID for a deferred
+ fix (integer)
+ """
+ line = line - 1
+ newText = ""
+ if self.__source[line].rstrip().endswith(('"""', "'''")) and \
+ self.__source[line].lstrip().startswith( ('"""', 'r"""', 'u"""')):
+ # it is a one-liner
+ newText = self.__source[line].rstrip()[:-3] + "." + \
+ self.__source[line].rstrip()[-3:] + self.__getEol()
+ else:
+ if line < len(self.__source) - 1 and \
+ (not self.__source[line + 1].strip() or
+ self.__source[line + 1].strip() in ('"""', "'''")):
+ newText = self.__source[line].rstrip() + "." + self.__getEol()
+
+ if newText:
+ self.__source[line] = newText
+ return (1, self.trUtf8("Period added to summary line."), 0)
+ else:
+ return (0, "", 0)
+
+ def __fixD141(self, code, line, pos, apply=False):
+ """
+ Private method to fix a function/method docstring preceded by a
+ blank line (D141).
+
+ @param code code of the issue (string)
+ @param line line number of the issue (integer)
+ @param pos position inside line (integer)
+ @keyparam apply flag indicating, that the fix should be applied
+ (boolean)
+ @return value indicating an applied/deferred fix (-1, 0, 1),
+ a message for the fix (string) and an ID for a deferred
+ fix (integer)
+ """
+ if apply:
+ line = line - 1
+ self.__source[line - 1] = ""
+ return (
+ 1,
+ self.trUtf8(
+ "Blank line before function/method docstring removed."),
+ 0)
+ else:
+ id = self.__getID()
+ self.__stack.append((id, code, line, pos))
+ return (-1, "", id)
+
+ def __fixD142(self, code, line, pos, apply=False):
+ """
+ Private method to fix a class docstring not preceded by a
+ blank line (D142).
+
+ @param code code of the issue (string)
+ @param line line number of the issue (integer)
+ @param pos position inside line (integer)
+ @keyparam apply flag indicating, that the fix should be applied
+ (boolean)
+ @return value indicating an applied/deferred fix (-1, 0, 1),
+ a message for the fix (string) and an ID for a deferred
+ fix (integer)
+ """
+ if apply:
+ line = line - 1
+ self.__source[line] = self.__getEol() + self.__source[line]
+ return (
+ 1,
+ self.trUtf8("Blank line inserted before class docstring."),
+ 0)
+ else:
+ id = self.__getID()
+ self.__stack.append((id, code, line, pos))
+ return (-1, "", id)
+
+ def __fixD143(self, code, line, pos, apply=False):
+ """
+ Private method to fix a class docstring not followed by a
+ blank line (D143).
+
+ @param code code of the issue (string)
+ @param line line number of the issue (integer)
+ @param pos position inside line (integer)
+ @keyparam apply flag indicating, that the fix should be applied
+ (boolean)
+ @return value indicating an applied/deferred fix (-1, 0, 1),
+ a message for the fix (string) and an ID for a deferred
+ fix (integer)
+ """
+ if apply:
+ line = line - 1
+ self.__source[line] += self.__getEol()
+ return (
+ 1,
+ self.trUtf8("Blank line inserted after class docstring."),
+ 0)
+ else:
+ id = self.__getID()
+ self.__stack.append((id, code, line, pos))
+ return (-1, "", id)
+
+ def __fixD144(self, code, line, pos, apply=False):
+ """
+ Private method to fix a docstring summary not followed by a
+ blank line (D144).
+
+ @param code code of the issue (string)
+ @param line line number of the issue (integer)
+ @param pos position inside line (integer)
+ @keyparam apply flag indicating, that the fix should be applied
+ (boolean)
+ @return value indicating an applied/deferred fix (-1, 0, 1),
+ a message for the fix (string) and an ID for a deferred
+ fix (integer)
+ """
+ if apply:
+ line = line - 1
+ if not self.__source[line].rstrip().endswith("."):
+ # only correct summary lines can be fixed here
+ return (0, "", 0)
+
+ self.__source[line] += self.__getEol()
+ return (
+ 1,
+ self.trUtf8("Blank line inserted after docstring summary."),
+ 0)
+ else:
+ id = self.__getID()
+ self.__stack.append((id, code, line, pos))
+ return (-1, "", id)
+
+ def __fixD145(self, code, line, pos, apply=False):
+ """
+ Private method to fix the last paragraph of a multi-line docstring
+ not followed by a blank line (D143).
+
+ @param code code of the issue (string)
+ @param line line number of the issue (integer)
+ @param pos position inside line (integer)
+ @keyparam apply flag indicating, that the fix should be applied
+ (boolean)
+ @return value indicating an applied/deferred fix (-1, 0, 1),
+ a message for the fix (string) and an ID for a deferred
+ fix (integer)
+ """
+ if apply:
+ line = line - 1
+ self.__source[line] = self.__getEol() + self.__source[line]
+ return (
+ 1,
+ self.trUtf8("Blank line inserted after last paragraph"
+ " of docstring."),
+ 0)
+ else:
+ id = self.__getID()
+ self.__stack.append((id, code, line, pos))
+ return (-1, "", id)
class Pep8Reindenter(object):
