10046
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
1
|
# -*- coding: utf-8 -*- |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
2
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
3
|
# Copyright (c) 2021 - 2023 Detlev Offenbach <detlev@die-offenbachs.de> |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
4
|
# |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
5
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
6
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
7
|
Module implementing a checker for import statements. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
8
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
9
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
10
|
import ast |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
11
|
import copy |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
12
|
import re |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
13
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
14
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
15
|
class NameOrderChecker: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
16
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
17
|
Class implementing a checker for name ordering. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
18
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
19
|
Note: Name ordering is checked for import statements, the '__all__' statement |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
20
|
and exception names of exception handlers. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
21
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
22
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
23
|
Codes = [ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
24
|
## Imports order |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
25
|
"NO101", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
26
|
"NO102", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
27
|
"NO103", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
28
|
"NO104", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
29
|
"NO105", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
30
|
] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
31
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
32
|
def __init__(self, source, filename, tree, select, ignore, expected, repeat, args): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
33
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
34
|
Constructor |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
35
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
36
|
@param source source code to be checked |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
37
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
38
|
@param filename name of the source file |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
39
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
40
|
@param tree AST tree of the source code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
41
|
@type ast.Module |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
42
|
@param select list of selected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
43
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
44
|
@param ignore list of codes to be ignored |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
45
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
46
|
@param expected list of expected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
47
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
48
|
@param repeat flag indicating to report each occurrence of a code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
49
|
@type bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
50
|
@param args dictionary of arguments for the various checks |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
51
|
@type dict |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
52
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
53
|
self.__select = tuple(select) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
54
|
self.__ignore = ("",) if select else tuple(ignore) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
55
|
self.__expected = expected[:] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
56
|
self.__repeat = repeat |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
57
|
self.__filename = filename |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
58
|
self.__source = source[:] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
59
|
self.__tree = copy.deepcopy(tree) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
60
|
self.__args = args |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
61
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
62
|
# parameters for import sorting |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
63
|
if args["SortOrder"] == "native": |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
64
|
self.__sortingFunction = sorted |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
65
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
66
|
# naturally is the default sort order |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
67
|
self.__sortingFunction = self.__naturally |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
68
|
self.__sortCaseSensitive = args["SortCaseSensitive"] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
69
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
70
|
# statistics counters |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
71
|
self.counters = {} |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
72
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
73
|
# collection of detected errors |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
74
|
self.errors = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
75
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
76
|
checkersWithCodes = [ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
77
|
(self.__checkNameOrder, ("NO101", "NO102", "NO103", "NO104", "NO105")), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
78
|
] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
79
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
80
|
self.__checkers = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
81
|
for checker, codes in checkersWithCodes: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
82
|
if any(not (code and self.__ignoreCode(code)) for code in codes): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
83
|
self.__checkers.append(checker) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
84
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
85
|
def __ignoreCode(self, code): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
86
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
87
|
Private method to check if the message code should be ignored. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
88
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
89
|
@param code message code to check for |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
90
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
91
|
@return flag indicating to ignore the given code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
92
|
@rtype bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
93
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
94
|
return code.startswith(self.__ignore) and not code.startswith(self.__select) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
95
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
96
|
def __error(self, lineNumber, offset, code, *args): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
97
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
98
|
Private method to record an issue. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
99
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
100
|
@param lineNumber line number of the issue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
101
|
@type int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
102
|
@param offset position within line of the issue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
103
|
@type int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
104
|
@param code message code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
105
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
106
|
@param args arguments for the message |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
107
|
@type list |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
108
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
109
|
if self.__ignoreCode(code): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
110
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
111
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
112
|
if code in self.counters: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
113
|
self.counters[code] += 1 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
114
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
115
|
self.counters[code] = 1 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
116
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
117
|
# Don't care about expected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
118
|
if code in self.__expected: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
119
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
120
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
121
|
if code and (self.counters[code] == 1 or self.__repeat): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
122
|
# record the issue with one based line number |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
123
|
self.errors.append( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
124
|
{ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
125
|
"file": self.__filename, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
126
|
"line": lineNumber + 1, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
127
|
"offset": offset, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
128
|
"code": code, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
129
|
"args": args, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
130
|
} |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
131
|
) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
132
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
133
|
def run(self): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
134
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
135
|
Public method to check the given source against miscellaneous |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
136
|
conditions. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
137
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
138
|
if not self.__filename: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
139
|
# don't do anything, if essential data is missing |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
140
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
141
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
142
|
if not self.__checkers: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
143
|
# don't do anything, if no codes were selected |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
144
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
145
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
146
|
for check in self.__checkers: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
147
|
check() |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
148
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
149
|
####################################################################### |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
150
|
## Name Order |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
151
|
## |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
152
|
## adapted from: flake8-alphabetize v0.0.21 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
153
|
####################################################################### |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
154
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
155
|
def __checkNameOrder(self): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
156
|
""" |
10052
|
157
|
Private method to check the order of import statements and handled exceptions. |
10046
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
158
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
159
|
from .ImportNode import ImportNode |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
160
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
161
|
errors = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
162
|
imports = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
163
|
importNodes, aListNode, eListNodes = self.__findNodes(self.__tree) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
164
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
165
|
# check for an error in '__all__' |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
166
|
allError = self.__findErrorInAll(aListNode) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
167
|
if allError is not None: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
168
|
errors.append(allError) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
169
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
170
|
errors.extend(self.__findExceptionListErrors(eListNodes)) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
171
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
172
|
for importNode in importNodes: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
173
|
if isinstance(importNode, ast.Import) and len(importNode.names) > 1: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
174
|
# skip suck imports because its already handled by pycodestyle |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
175
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
176
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
177
|
imports.append( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
178
|
ImportNode( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
179
|
self.__args.get("ApplicationPackageNames", []), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
180
|
importNode, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
181
|
self, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
182
|
self.__args.get("SortIgnoringStyle", False), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
183
|
self.__args.get("SortFromFirst", False), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
184
|
) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
185
|
) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
186
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
187
|
lenImports = len(imports) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
188
|
if lenImports > 0: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
189
|
p = imports[0] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
190
|
if p.error is not None: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
191
|
errors.append(p.error) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
192
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
193
|
if lenImports > 1: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
194
|
for n in imports[1:]: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
195
|
if n.error is not None: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
196
|
errors.append(n.error) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
197
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
198
|
if n == p: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
199
|
if self.__args.get("CombinedAsImports", False) or ( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
200
|
not n.asImport and not p.asImport |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
201
|
): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
202
|
errors.append((n.node, "NO103", str(p), str(n))) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
203
|
elif n < p: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
204
|
errors.append((n.node, "NO101", str(n), str(p))) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
205
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
206
|
p = n |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
207
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
208
|
for error in errors: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
209
|
if not self.__ignoreCode(error[1]): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
210
|
node = error[0] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
211
|
reason = error[1] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
212
|
args = error[2:] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
213
|
self.__error(node.lineno - 1, node.col_offset, reason, *args) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
214
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
215
|
def __findExceptionListNodes(self, tree): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
216
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
217
|
Private method to find all exception types handled by given tree. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
218
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
219
|
@param tree reference to the ast node tree to be parsed |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
220
|
@type ast.AST |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
221
|
@return list of exception types |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
222
|
@rtype list of ast.Name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
223
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
224
|
nodes = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
225
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
226
|
for node in ast.walk(tree): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
227
|
if isinstance(node, ast.ExceptHandler): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
228
|
nodeType = node.type |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
229
|
if isinstance(nodeType, (ast.List, ast.Tuple)): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
230
|
nodes.append(nodeType) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
231
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
232
|
return nodes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
233
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
234
|
def __findNodes(self, tree): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
235
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
236
|
Private method to find all import and import from nodes of the given |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
237
|
tree. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
238
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
239
|
@param tree reference to the ast node tree to be parsed |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
240
|
@type ast.AST |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
241
|
@return tuple containing a list of import nodes, the '__all__' node and |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
242
|
exception nodes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
243
|
@rtype tuple of (ast.Import | ast.ImportFrom, ast.List | ast.Tuple, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
244
|
ast.List | ast.Tuple) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
245
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
246
|
importNodes = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
247
|
aListNode = None |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
248
|
eListNodes = self.__findExceptionListNodes(tree) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
249
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
250
|
if isinstance(tree, ast.Module): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
251
|
body = tree.body |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
252
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
253
|
for n in body: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
254
|
if isinstance(n, (ast.Import, ast.ImportFrom)): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
255
|
importNodes.append(n) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
256
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
257
|
elif isinstance(n, ast.Assign): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
258
|
for t in n.targets: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
259
|
if isinstance(t, ast.Name) and t.id == "__all__": |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
260
|
value = n.value |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
261
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
262
|
if isinstance(value, (ast.List, ast.Tuple)): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
263
|
aListNode = value |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
264
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
265
|
return importNodes, aListNode, eListNodes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
266
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
267
|
def __findErrorInAll(self, node): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
268
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
269
|
Private method to check the '__all__' node for errors. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
270
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
271
|
@param node reference to the '__all__' node |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
272
|
@type ast.List or ast.Tuple |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
273
|
@return tuple containing a reference to the node an error code and the error |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
274
|
arguments |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
275
|
@rtype tuple of (ast.List | ast.Tuple, str, str) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
276
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
277
|
if node is not None: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
278
|
actualList = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
279
|
for el in node.elts: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
280
|
if isinstance(el, ast.Constant): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
281
|
actualList.append(el.value) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
282
|
elif isinstance(el, ast.Str): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
283
|
actualList.append(el.s) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
284
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
285
|
# Can't handle anything that isn't a string literal |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
286
|
return None |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
287
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
288
|
expectedList = self.sorted( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
289
|
actualList, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
290
|
key=lambda k: self.moduleKey(k, subImports=True), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
291
|
) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
292
|
if expectedList != actualList: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
293
|
return (node, "NO104", ", ".join(expectedList)) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
294
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
295
|
return None |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
296
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
297
|
def __findExceptionListStr(self, node): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
298
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
299
|
Private method to get the exception name out of an exception handler type node. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
300
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
301
|
@param node node to be treated |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
302
|
@type ast.Name or ast.Attribute |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
303
|
@return string containing the exception name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
304
|
@rtype str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
305
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
306
|
if isinstance(node, ast.Name): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
307
|
return node.id |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
308
|
elif isinstance(node, ast.Attribute): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
309
|
return f"{self.__findExceptionListStr(node.value)}.{node.attr}" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
310
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
311
|
return "" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
312
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
313
|
def __findExceptionListErrors(self, nodes): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
314
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
315
|
Private method to check the exception node for errors. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
316
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
317
|
@param nodes list of exception nodes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
318
|
@type list of ast.List or ast.Tuple |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
319
|
@return DESCRIPTION |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
320
|
@rtype TYPE |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
321
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
322
|
errors = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
323
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
324
|
for node in nodes: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
325
|
actualList = [self.__findExceptionListStr(elt) for elt in node.elts] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
326
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
327
|
expectedList = self.sorted(actualList) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
328
|
if expectedList != actualList: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
329
|
errors.append((node, "NO105", ", ".join(expectedList))) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
330
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
331
|
return errors |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
332
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
333
|
def sorted(self, toSort, key=None, reverse=False): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
334
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
335
|
Public method to sort the given list of names. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
336
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
337
|
@param toSort list of names to be sorted |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
338
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
339
|
@param key function to generate keys (defaults to None) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
340
|
@type function (optional) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
341
|
@param reverse flag indicating a reverse sort (defaults to False) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
342
|
@type bool (optional) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
343
|
@return sorted list of names |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
344
|
@rtype list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
345
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
346
|
return self.__sortingFunction(toSort, key=key, reverse=reverse) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
347
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
348
|
def __naturally(self, toSort, key=None, reverse=False): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
349
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
350
|
Private method to sort the given list of names naturally. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
351
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
352
|
Note: Natural sorting maintains the sort order of numbers (i.e. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
353
|
[Q1, Q10, Q2] is sorted as [Q1, Q2, Q10] while the Python |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
354
|
standard sort would yield [Q1, Q10, Q2]. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
355
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
356
|
@param toSort list of names to be sorted |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
357
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
358
|
@param key function to generate keys (defaults to None) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
359
|
@type function (optional) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
360
|
@param reverse flag indicating a reverse sort (defaults to False) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
361
|
@type bool (optional) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
362
|
@return sorted list of names |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
363
|
@rtype list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
364
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
365
|
if key is None: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
366
|
keyCallback = self.__naturalKeys |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
367
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
368
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
369
|
def keyCallback(text): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
370
|
return self.__naturalKeys(key(text)) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
371
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
372
|
return sorted(toSort, key=keyCallback, reverse=reverse) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
373
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
374
|
def __atoi(self, text): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
375
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
376
|
Private method to convert the given text to an integer number. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
377
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
378
|
@param text text to be converted |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
379
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
380
|
@return integer number |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
381
|
@rtype int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
382
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
383
|
return int(text) if text.isdigit() else text |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
384
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
385
|
def __naturalKeys(self, text): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
386
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
387
|
Private method to generate keys for natural sorting. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
388
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
389
|
@param text text to generate a key for |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
390
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
391
|
@return key for natural sorting |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
392
|
@rtype list of str or int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
393
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
394
|
return [self.__atoi(c) for c in re.split(r"(\d+)", text)] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
395
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
396
|
def moduleKey(self, moduleName, subImports=False): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
397
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
398
|
Public method to generate a key for the given module name. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
399
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
400
|
@param moduleName module name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
401
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
402
|
@param subImports flag indicating a sub import like in |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
403
|
'from foo import bar, baz' (defaults to False) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
404
|
@type bool (optional) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
405
|
@return generated key |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
406
|
@rtype str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
407
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
408
|
prefix = "" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
409
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
410
|
if subImports: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
411
|
if moduleName.isupper() and len(moduleName) > 1: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
412
|
prefix = "A" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
413
|
elif moduleName[0:1].isupper(): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
414
|
prefix = "B" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
415
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
416
|
prefix = "C" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
417
|
if not self.__sortCaseSensitive: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
418
|
moduleName = moduleName.lower() |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
419
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
420
|
return f"{prefix}{moduleName}" |