Opened 23 months ago
Closed 23 months ago
#34244 closed New feature (wontfix)
Allow overriding error params in BaseValidator
| Reported by: | Mateusz Kurowski | Owned by: | nobody |
|---|---|---|---|
| Component: | Core (Other) | Version: | dev |
| Severity: | Normal | Keywords: | validators, BaseValidator, params, error, override |
| Cc: | Triage Stage: | Unreviewed | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Hello. I wonder if that is a valid change i suggest: Right now params are hardcoded, see:
It would be nice to call self.get_error_params - we could override this method.
What do you think?
Change History (2)
comment:1 by , 23 months ago
| Summary: | allow overriding error params in BaseValidator → Allow overriding error params in BaseValidator |
|---|
comment:2 by , 23 months ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
BaseValidatoris a private API used only for the min-max validators, it's also quite small, so IMO it's not worth additional complexity.You can start a discussion on DevelopersMailingList, if you don't agree. Also, please follow triaging guidelines with regards to wontfix tickets.