Přidání chybové hlášky :3 #55

Merged
zelvuska merged 2 commits from divni-uzivatele into master 5 months ago
ledoian commented 6 months ago
Owner
There is no content yet.
ledoian added 1 commit 6 months ago
zelvuska reviewed 6 months ago
user = request.user
if not isinstance(user, AnonymousUser) and m.Osoba.objects.filter(user=user).count() != 1:
# m.Osoba.objects.get() v ostatních views selže
return HttpResponse(render(request, "universal.html", {
Poster
Owner

Tohle nejde napsat lépe? Fakt to musí být HttpResponse(render(request, atd.?

Tohle nejde napsat lépe? Fakt to musí být `HttpResponse(render(request,` atd.?
Poster
Owner

Myslím, že ta HttpResponse se tam doplní sama, ale jinak to tutoriál má stejně…

Myslím, že ta `HttpResponse` se tam doplní sama, ale jinak to [tutoriál](https://docs.djangoproject.com/en/5.0/intro/tutorial03/#s-a-shortcut-render) má stejně…
Poster
Owner

Smazáno, jdu otestovat…

Smazáno, jdu otestovat…
Poster
Owner

To HttpResponse na tom vypadalo nejpodivněji, takhle mi to přijde OK :)

To `HttpResponse` na tom vypadalo nejpodivněji, takhle mi to přijde OK :)
Poster
Owner

works on my machine

works on my machine
Poster
Owner

To HttpResponse na tom vypadalo nejpodivněji, takhle mi to přijde OK :)

Mně to přišlo intuitivní: render vyrobí z template a kontextu tělo, HttpResponse kolem toho přidá hlavičky, prostě jiná vrstva. Moc síťuju :-P

> To `HttpResponse` na tom vypadalo nejpodivněji, takhle mi to přijde OK :) Mně to přišlo intuitivní: `render` vyrobí z template a kontextu tělo, `HttpResponse` kolem toho přidá hlavičky, prostě jiná vrstva. Moc síťuju :-P
zelvuska marked this conversation as resolved
ledoian added 1 commit 6 months ago
zelvuska approved these changes 6 months ago
zelvuska left a comment

Beru…

zelvuska merged commit 69019baf4e into master 5 months ago
zelvuska deleted branch divni-uzivatele 5 months ago

Reviewers

zelvuska approved these changes 6 months ago
The pull request has been merged as 69019baf4e.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.