Opened 10 years ago

Last modified 10 years ago

#24581 new Cleanup/optimization

Explicitly raise an exception if ManyToManyField._get_m2m_attr fails to match

Reported by: Patryk Zawadzki Owned by: nobody
Component: Database layer (models, ORM) Version: 1.8
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: yes Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

In cases like #24513 it results in surprising error messages and useless tracebacks.

According to the ticket's flags, the next step(s) to move this issue forward are:

  • To add tests to the patch, then uncheck the "Needs tests" flag on the ticket.

  • If creating a new pull request, include a link to the pull request in the ticket comment when making that update. The usual format is: [https://github.com/django/django/pull/#### PR].

Change History (2)

comment:1 by Patryk Zawadzki, 10 years ago

Has patch: set

Pull request: ​https://github.com/django/django/pull/4451

comment:2 by Tim Graham, 10 years ago

Needs tests: set
Triage Stage: Unreviewed → Accepted
Type: Uncategorized → Cleanup/optimization
Note: See TracTickets for help on using tickets.
Back to Top