ssg.jinja.process_file() - Code Metrics - Inspection of "Add rule directory functionality to the build syst..." - ComplianceAsCode/content - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#3188)

by Alexander

created 2018-08-15 16:12 UTC

ssg.jinja.process_file() A

↳ Parent: ssg.jinja

Complexity

Conditions

Size

Total Lines	4
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	1

Importance

Changes

Metric	Value
cc	1
eloc	4
nop	2
dl	0
loc	4
rs	10
c	0
b	0
f	0
ccs	4
cts	4
cp	1
crap	1

from __future__ import absolute_import
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
from __future__ import print_function

import os.path
import jinja2

from .utils import required_key


class AbsolutePathFileSystemLoader(jinja2.BaseLoader):

    """Loads templates from the file system. This loader insists on absolute
    paths and fails if a relative path is provided.

    >>> loader = AbsolutePathFileSystemLoader()

    Per default the template encoding is ``'utf-8'`` which can be changed
    by setting the `encoding` parameter to something else.
    """

    def __init__(self, encoding='utf-8'):
        self.encoding = encoding

    def get_source(self, environment, template):
        if not os.path.isabs(template):
            raise jinja2.TemplateNotFound(template)

        template_file = jinja2.utils.open_if_exists(template)
        if template_file is None:
            raise jinja2.TemplateNotFound(template)
        try:
            contents = template_file.read().decode(self.encoding)
        finally:
            template_file.close()

        mtime = os.path.getmtime(template)

        def uptodate():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
            try:
                return os.path.getmtime(template) == mtime
            except OSError:
                return False
        return contents, template, uptodate


def _get_jinja_environment(substitutions_dict):
    if _get_jinja_environment.env is None:
        bytecode_cache = None
        if substitutions_dict.get("jinja2_cache_enabled") == "true":
            bytecode_cache = jinja2.FileSystemBytecodeCache(
                required_key(substitutions_dict, "jinja2_cache_dir")
            )

        # TODO: Choose better syntax?

        _get_jinja_environment.env = jinja2.Environment(
            block_start_string="{{%",
            block_end_string="%}}",
            variable_start_string="{{{",
            variable_end_string="}}}",
            comment_start_string="{{#",
            comment_end_string="#}}",
            loader=AbsolutePathFileSystemLoader(),
            bytecode_cache=bytecode_cache
        )

    return _get_jinja_environment.env


_get_jinja_environment.env = None


def extract_substitutions_dict_from_template(filename, substitutions_dict):

    """
    Treat the given filename as a jinja2 file containing macro definitions,
    and export definitions that don't start with _ as a name->macro dictionary.
    During macro compilation, symbols from substitutions_dict may be used in those definitions.
    """
    template = _get_jinja_environment(substitutions_dict).get_template(filename)
    all_symbols = template.make_module(substitutions_dict).__dict__
    symbols_to_export = dict()
    for name, symbol in all_symbols.items():
        if name.startswith("_"):
            continue
        symbols_to_export[name] = symbol
    return symbols_to_export


def process_file(filepath, substitutions_dict):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    filepath = os.path.abspath(filepath)
    template = _get_jinja_environment(substitutions_dict).get_template(filepath)
    return template.render(substitutions_dict)


1	1	from __future__ import absolute_import
		0 ignored issues – show Coding Style introduced 2018-06-22 19:05 UTC by Report Bug Copy Issue Report This module should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
2	1	from __future__ import print_function
3
4	1	import os.path
5	1	import jinja2
6
7	1	from .utils import required_key
8
9
10	1	class AbsolutePathFileSystemLoader(jinja2.BaseLoader):
		0 ignored issues – show Unused Code introduced 2018-06-13 12:58 UTC by Report Bug Copy Issue Report The variable `__class__` seems to be unused. Loading history...
11		"""Loads templates from the file system. This loader insists on absolute
12		paths and fails if a relative path is provided.
13
14		>>> loader = AbsolutePathFileSystemLoader()
15
16		Per default the template encoding is ``'utf-8'`` which can be changed
17		by setting the `encoding` parameter to something else.
18		"""
19
20	1	def __init__(self, encoding='utf-8'):
21	1	self.encoding = encoding
22
23	1	def get_source(self, environment, template):
24	1	if not os.path.isabs(template):
25		raise jinja2.TemplateNotFound(template)
26
27	1	template_file = jinja2.utils.open_if_exists(template)
28	1	if template_file is None:
29		raise jinja2.TemplateNotFound(template)
30	1	try:
31	1	contents = template_file.read().decode(self.encoding)
32		finally:
33	1	template_file.close()
34
35	1	mtime = os.path.getmtime(template)
36
37	1	def uptodate():
		0 ignored issues – show Coding Style introduced 2018-06-13 12:58 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
38	1	try:
39	1	return os.path.getmtime(template) == mtime
40		except OSError:
41		return False
42	1	return contents, template, uptodate
43
44
45	1	def _get_jinja_environment(substitutions_dict):
46	1	if _get_jinja_environment.env is None:
47	1	bytecode_cache = None
48	1	if substitutions_dict.get("jinja2_cache_enabled") == "true":
49		bytecode_cache = jinja2.FileSystemBytecodeCache(
50		required_key(substitutions_dict, "jinja2_cache_dir")
51		)
52
53		# TODO: Choose better syntax?
		0 ignored issues – show Coding Style introduced 2018-06-13 20:14 UTC by Report Bug Copy Issue Report `TODO` and `FIXME` comments should generally be avoided. Loading history...
54	1	_get_jinja_environment.env = jinja2.Environment(
55		block_start_string="{{%",
56		block_end_string="%}}",
57		variable_start_string="{{{",
58		variable_end_string="}}}",
59		comment_start_string="{{#",
60		comment_end_string="#}}",
61		loader=AbsolutePathFileSystemLoader(),
62		bytecode_cache=bytecode_cache
63		)
64
65	1	return _get_jinja_environment.env
66
67
68	1	_get_jinja_environment.env = None
69
70
71	1	def extract_substitutions_dict_from_template(filename, substitutions_dict):
		0 ignored issues – show Coding Style Naming introduced 2018-07-27 16:37 UTC by Report Bug Copy Issue Report The name `extract_substitutions_dict_from_template` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
72		"""
73		Treat the given filename as a jinja2 file containing macro definitions,
74		and export definitions that don't start with _ as a name->macro dictionary.
75		During macro compilation, symbols from substitutions_dict may be used in those definitions.
76		"""
77	1	template = _get_jinja_environment(substitutions_dict).get_template(filename)
78	1	all_symbols = template.make_module(substitutions_dict).__dict__
79	1	symbols_to_export = dict()
80	1	for name, symbol in all_symbols.items():
81	1	if name.startswith("_"):
82	1	continue
83	1	symbols_to_export[name] = symbol
84	1	return symbols_to_export
85
86
87	1	def process_file(filepath, substitutions_dict):
		0 ignored issues – show Coding Style introduced 2018-06-13 18:48 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
88	1	filepath = os.path.abspath(filepath)
89	1	template = _get_jinja_environment(substitutions_dict).get_template(filepath)
90		return template.render(substitutions_dict)
91

ComplianceAsCode / content

Pull Request — master (#3188)

ssg.jinja.process_file() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like