add pylint disable, r-string, line length, change isinstance

This commit is contained in:
Loic Coyle 2020-05-04 20:26:07 +02:00
parent 8e6f6d356c
commit cecc28cc52

View File

@ -12,6 +12,7 @@ from .settings import CACHE_DIR, CONF_DIR, MODULE_DIR
def template(colors, input_file, output_file=None): def template(colors, input_file, output_file=None):
"""Read template file, substitute markers and """Read template file, substitute markers and
save the file elsewhere.""" save the file elsewhere."""
# pylint: disable-msg=too-many-locals
template_data = util.read_file_raw(input_file) template_data = util.read_file_raw(input_file)
for i, l in enumerate(template_data): for i, l in enumerate(template_data):
for match in re.finditer(r"(?<=(?<!\{))(\{([^{}]+)\})(?=(?!\}))", l): for match in re.finditer(r"(?<=(?<!\{))(\{([^{}]+)\})(?=(?!\}))", l):
@ -25,7 +26,7 @@ def template(colors, input_file, output_file=None):
# Color to be modified copied into new one # Color to be modified copied into new one
new_color = util.Color(colors[cname].hex_color) new_color = util.Color(colors[cname].hex_color)
# Execute each function to be done # Execute each function to be done
for func in filter(None, re.split("\)|\.", funcs)): for func in filter(None, re.split(r"\)|\.", funcs)):
# Get function name and arguments # Get function name and arguments
func = func.split("(") func = func.split("(")
fname = func[0] fname = func[0]
@ -52,12 +53,13 @@ def template(colors, input_file, output_file=None):
replace_str += '.' + fname replace_str += '.' + fname
new_color = function new_color = function
if not isinstance(new_color, str): if isinstance(new_color, util.Color):
new_color = new_color.strip new_color = new_color.strip
# If the color was changed, replace with a unique identifier. # If the color was changed, replace with a unique identifier.
if new_color is not colors[cname]: if new_color is not colors[cname]:
new_color_clean = new_color.replace('[', '_').replace(']', '_') new_color_clean = new_color.replace('[', '_').replace(']', '_')
template_data[i] = l.replace(replace_str, "color" + new_color_clean) template_data[i] = l.replace(replace_str,
"color" + new_color_clean)
colors["color" + new_color_clean] = new_color colors["color" + new_color_clean] = new_color
try: try:
template_data = "".join(template_data).format(**colors) template_data = "".join(template_data).format(**colors)