| Dir : /opt/cloudlinux/venv/lib64/python3.11/site-packages/pylint_django/checkers/ |
| Server: Linux premium279.web-hosting.com 4.18.0-553.45.1.lve.el8.x86_64 #1 SMP Wed Mar 26 12:08:09 UTC 2025 x86_64 IP: 66.29.132.192 |
| Dir : //opt/cloudlinux/venv/lib64/python3.11/site-packages/pylint_django/checkers/forms.py |
"""Models."""
from astroid.nodes import Assign, AssignName, ClassDef
from pylint.checkers import BaseChecker
from pylint.checkers.utils import check_messages
from pylint.interfaces import IAstroidChecker
from pylint_django.__pkginfo__ import BASE_ID
from pylint_django.utils import node_is_subclass
def _get_child_meta(node):
for child in node.get_children():
if isinstance(child, ClassDef) and child.name == "Meta":
return child
return None
class FormChecker(BaseChecker):
"""Django model checker."""
__implements__ = IAstroidChecker
name = "django-form-checker"
msgs = {
f"W{BASE_ID}04": (
"Use explicit fields instead of exclude in ModelForm",
"modelform-uses-exclude",
"Prevents accidentally allowing users to set fields, especially when adding new fields to a Model",
)
}
@check_messages("modelform-uses-exclude")
def visit_classdef(self, node):
"""Class visitor."""
if not node_is_subclass(node, "django.forms.models.ModelForm", ".ModelForm"):
# we only care about forms
return
meta = _get_child_meta(node)
if not meta:
return
for child in meta.get_children():
if not isinstance(child, Assign) or not isinstance(child.targets[0], AssignName):
continue
if child.targets[0].name == "exclude":
self.add_message(f"W{BASE_ID}04", node=child)
break