Re: [Django] #10494: show query params DoesNotExist from QuerySet.get

[Django]

#10494: show query params DoesNotExist from QuerySet.get
-------------------------------------+-------------------------------------
Reporter: brondsem | Owner: nobody
Type: New feature | Status: new
Component: Database layer | Version: 1.0
(models, ORM) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: | Needs documentation: 0
Has patch: 0 | Patch needs improvement: 0
Needs tests: 0 | UI/UX: 0
Easy pickings: 1 |
-------------------------------------+-------------------------------------
Changes (by aaugustin):

* ui_ux: => 0
* easy: => 1
* stage: Design decision needed => Accepted

Comment:

I'm not sure why this is flagged as DDN. Informative exceptions are good.

--
Ticket URL: <https://code.djangoproject.com/ticket/10494#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

[Django]

#10494: show query params DoesNotExist from QuerySet.get
-------------------------------------+-------------------------------------

Reporter: brondsem | Owner: zefciu
Type: New feature | Status: assigned

Component: Database layer | Version: 1.0
(models, ORM) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: | Needs documentation: 0
Has patch: 0 | Patch needs improvement: 0
Needs tests: 0 | UI/UX: 0
Easy pickings: 1 |
-------------------------------------+-------------------------------------

Changes (by zefciu):

* owner: nobody => zefciu
* status: new => assigned

--
Ticket URL: <https://code.djangoproject.com/ticket/10494#comment:4>

[Django]

#10494: show query params DoesNotExist from QuerySet.get
-------------------------------------+-------------------------------------
Reporter: brondsem | Owner: zefciu
Type: New feature | Status: assigned
Component: Database layer | Version: 1.0
(models, ORM) | Resolution:
Severity: Normal | Triage Stage: Accepted
Keywords: | Needs documentation: 0

Has patch: 1 | Patch needs improvement: 0

Needs tests: 0 | UI/UX: 0
Easy pickings: 1 |
-------------------------------------+-------------------------------------
Changes (by zefciu):

* has_patch: 0 => 1

Comment:

Created a patch as a pull request:
https://github.com/django/django/pull/110

--
Ticket URL: <https://code.djangoproject.com/ticket/10494#comment:5>

[Django]

#10494: show query params DoesNotExist from QuerySet.get
-------------------------------------+-------------------------------------
Reporter: brondsem | Owner: zefciu
Type: New feature | Status: assigned
Component: Database layer | Version: 1.0
(models, ORM) | Resolution:

Severity: Normal | Triage Stage: Ready for
Keywords: | checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by oinopion):

* stage: Accepted => Ready for checkin

Comment:

Looks good.

--
Ticket URL: <https://code.djangoproject.com/ticket/10494#comment:6>

[Django]

#10494: show query params DoesNotExist from QuerySet.get
-------------------------------------+-------------------------------------
Reporter: brondsem | Owner: zefciu

Type: New feature | Status: closed

Component: Database layer | Version: 1.0

(models, ORM) | Resolution: fixed

Severity: Normal | Triage Stage: Ready for
Keywords: | checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by akaariai):

* status: assigned => closed
* resolution: => fixed

Comment:

In [17917]:
{{{
#!CommitTicketReference repository="" revision="17917"
Fixed #10494 -- Added kwargs to QuerySet.get() error message in the case
no objects were found.

Thanks brondsem for the report, Szymon Pyzalski for the patch and oinopion
for review.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/10494#comment:7>

[Django]

#10494: show query params DoesNotExist from QuerySet.get
-------------------------------------+-------------------------------------

Reporter: Dave Brondsema | Owner: zefciu

Type: New feature | Status: closed
Component: Database layer | Version: 1.0
(models, ORM) |

Severity: Normal | Resolution: wontfix
Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by Mariusz Felisiak):

* has_patch: 1 => 0
* resolution: fixed => wontfix
* stage: Ready for checkin => Unreviewed

Comment:

#32522 is a duplicate,

This ticket was fixed in d5b93d3281fe93cbef5de84a52001f64dbd839bd but the
patch was reverted in 266c0bb23e9d64c47ace4d162e582febd5a1e336. I'm not
sure it's worth additional complexity and really helpful for beginners.

We could reopen after a discussion and reaching consensus on
DevelopersMailingList.

--
Ticket URL: <https://code.djangoproject.com/ticket/10494#comment:8>

[Django]

#10494: show query params DoesNotExist from QuerySet.get
-------------------------------------+-------------------------------------
Reporter: Dave Brondsema | Owner: zefciu
Type: New feature | Status: closed
Component: Database layer | Version: 1.0
(models, ORM) |
Severity: Normal | Resolution: wontfix
Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Anders Hovmöller):

It's fairly common that beginners are confused in ways that would be
trivially solved by this improvement. I base this on experience in the
Django Discord.

On a more personal note, this would help me a lot in speeding up debugging
in the rare cases it happens to me in production.

--
Ticket URL: <https://code.djangoproject.com/ticket/10494#comment:9>