odevzdavatko: hodnoceniReseniView -> hodnoceni_reseni_view

odevzdavatko/urls.py

@@ -7,7 +7,7 @@ Soubor sloužící jako „router“, tj. zde se definují url adresy a na co uk
 - ``org/reseni/`` (odevzdavatko_tabulka) :class:`~odevzdavatko.views.TabulkaOdevzdanychReseniView`
 - ``org/reseni/rocnik/<int:rocnik>/`` (odevzdavatko_tabulka) :class:`~odevzdavatko.views.TabulkaOdevzdanychReseniView`
 - ``org/reseni/<int:problem>/<int:resitel>/`` (odevzdavatko_reseni_resitele_k_problemu) :class:`~odevzdavatko.views.ReseniProblemuView`
-- ``org/reseni/<int:pk>/`` (odevzdavatko_detail_reseni) :func:`~seminar.utils.viewMethodSwitch` + :class:`~odevzdavatko.views.DetailReseniView` + :func:`~odevzdavatko.views.hodnoceniReseniView`
+- ``org/reseni/<int:pk>/`` (odevzdavatko_detail_reseni) :func:`~seminar.utils.viewMethodSwitch` + :class:`~odevzdavatko.views.DetailReseniView` + :func:`~odevzdavatko.views.hodnoceni_reseni_view`
 - ``org/reseni/all`` :class:`~odevzdavatko.views.SeznamReseniView`
 - ``org/reseni/akt`` :class:`~odevzdavatko.views.TabulkaOdevzdanychReseniView`
 - ``resitel/reseni/<int:pk>`` (odevzdavatko_resitel_reseni) :class:`~odevzdavatko.views.ResitelReseniView`
@@ -26,7 +26,7 @@ urlpatterns = [
 	path('org/reseni/', org_required(views.TabulkaOdevzdanychReseniView.as_view()), name='odevzdavatko_tabulka'),
 	path('org/reseni/rocnik/<int:rocnik>/', org_required(views.TabulkaOdevzdanychReseniView.as_view()), name='odevzdavatko_tabulka'),
 	path('org/reseni/<int:problem>/<int:resitel>/', org_required(views.ReseniProblemuView.as_view()), name='odevzdavatko_reseni_resitele_k_problemu'),
-	path('org/reseni/<int:pk>', org_required(viewMethodSwitch(get=views.DetailReseniView.as_view(), post=views.hodnoceniReseniView)), name='odevzdavatko_detail_reseni'),
+	path('org/reseni/<int:pk>', org_required(viewMethodSwitch(get=views.DetailReseniView.as_view(), post=views.hodnoceni_reseni_view)), name='odevzdavatko_detail_reseni'),
 	path('org/reseni/all', org_required(views.SeznamReseniView.as_view())),
 	path('org/reseni/akt', org_required(views.SeznamAktualnichReseniView.as_view())),
 

odevzdavatko/views.py

@@ -219,28 +219,16 @@ class DetailReseniView(DetailView):
 	model = m.Reseni
 	template_name = 'odevzdavatko/detail.html'
 	
-	def aktualni_hodnoceni(self):
-		self.reseni = get_object_or_404(m.Reseni, id=self.kwargs['pk'])
-		result = [] # Slovníky s klíči problem, body, deadline_body -- initial data pro f.OhodnoceniReseniFormSet
-		for hodn in m.Hodnoceni.objects.filter(reseni=self.reseni):
-			result.append({
-				"problem": hodn.problem,
-				"body": hodn.body,
-				"deadline_body": hodn.deadline_body,
-				"feedback": hodn.feedback,
-				})
-		return result
-
 	def get_context_data(self, **kw):
+		reseni = get_object_or_404(m.Reseni, id=self.kwargs['pk'])
+		initial = m.Hodnoceni.objects.filter(reseni=reseni).values('problem', 'body', 'deadline_body', 'feedback',)
 		ctx = super().get_context_data(**kw)
-		ctx['form'] = f.OhodnoceniReseniFormSet(
-				initial = self.aktualni_hodnoceni()
-				)
-		ctx['poznamka_form'] = f.PoznamkaReseniForm(instance=self.reseni)
+		ctx['form'] = f.OhodnoceniReseniFormSet(initial=initial)
+		ctx['poznamka_form'] = f.PoznamkaReseniForm(instance=reseni)
 		return ctx
 
 
-def hodnoceniReseniView(request, pk, *args, **kwargs):
+def hodnoceni_reseni_view(request, pk, *args, **kwargs):
 	reseni = get_object_or_404(m.Reseni, pk=pk)
 	template_name = 'odevzdavatko/detail.html'
 	form_class = f.OhodnoceniReseniFormSet