38 lines
1.3 KiB
Python
38 lines
1.3 KiB
Python
import astroid
|
|
import pylint.interfaces
|
|
from pylint.checkers import BaseChecker
|
|
|
|
try:
|
|
from pylint.checkers.utils import only_required_for_messages
|
|
except ImportError:
|
|
from pylint.checkers.utils import check_messages as only_required_for_messages
|
|
|
|
def parse_version(s):
|
|
# can't use odoo.tools.parse_version because pythonpath is screwed from
|
|
# inside pylint on runbot
|
|
return [s.rjust(3, '0') for s in s.split('.')]
|
|
|
|
class OdooBaseChecker(BaseChecker):
|
|
if parse_version(pylint.__version__) < parse_version('2.14.0'):
|
|
__implements__ = pylint.interfaces.IAstroidChecker
|
|
name = 'odoo'
|
|
|
|
msgs = {
|
|
'E8502': (
|
|
'Bad usage of _, _lt function.',
|
|
'gettext-variable',
|
|
'See https://www.odoo.com/documentation/17.0/developer/misc/i18n/translations.html#variables'
|
|
)
|
|
}
|
|
|
|
@only_required_for_messages('gettext-variable')
|
|
def visit_call(self, node):
|
|
if isinstance(node.func, astroid.Name) and node.func.name in ('_', '_lt'):
|
|
first_arg = node.args[0]
|
|
if not (isinstance(first_arg, astroid.Const) and isinstance(first_arg.value, str)):
|
|
self.add_message('gettext-variable', node=node)
|
|
|
|
|
|
def register(linter):
|
|
linter.register_checker(OdooBaseChecker(linter))
|