render_to_response -> render
authorDavin Baragiotta <davin.baragiotta@auf.org>
Wed, 30 May 2012 19:32:57 +0000 (15:32 -0400)
committerDavin BARAGIOTTA <davin.baragiotta@auf.org>
Wed, 30 May 2012 19:32:57 +0000 (15:32 -0400)
project/dae/models.py
project/dae/views.py
project/recrutement/views.py
project/rh/views.py
project/views.py

index 4835e00..e0a78dd 100644 (file)
@@ -18,9 +18,6 @@ from project.dae.workflow import \
         DOSSIER_ETAT_DRH_FINALISATION, DOSSIER_ETAT_REGION_FINALISATION, \
         DOSSIER_ETAT_FINALISE
 
-# XXX: Saloperie, il faut importer rh.models à partir d'un autre namespace
-# à cause du hack app_context() dans project.rh.models. Très fragile. Il
-# faut régler ça aussi vite que possible.
 from project.rh import models as rh
 from project.rh.models import HELP_TEXT_DATE
 
index b8d9fdf..5dd8aee 100644 (file)
@@ -11,8 +11,7 @@ from django.core.exceptions import MultipleObjectsReturned
 from django.core.paginator import Paginator, InvalidPage
 from django.db.models import Q
 from django.http import Http404, HttpResponse, HttpResponseGone
-from django.shortcuts import redirect, render_to_response, get_object_or_404
-from django.template import RequestContext
+from django.shortcuts import redirect, render, get_object_or_404
 
 from sendfile import sendfile
 
@@ -57,7 +56,7 @@ def devises():
 
 @dae_groupe_requis
 def index(request):
-    return render_to_response('dae/index.html', {}, RequestContext(request))
+    return render(request, 'dae/index.html', {})
 
 
 ### POSTE
@@ -89,9 +88,7 @@ def poste_consulter(request, key):
         'comparaisons_internes': comparaisons_internes
     }
 
-    return render_to_response(
-        'dae/poste_consulter.html', vars, RequestContext(request)
-    )
+    return render(request, 'dae/poste_consulter.html', vars)
 
 
 def poste_importer(request, id):
@@ -103,9 +100,10 @@ def poste_importer(request, id):
         else:
             return redirect('poste_consulter', 'dae-' + id)
     else:
-        return render_to_response('dae/poste_importer.html', {
-            'poste': poste_dae
-        }, RequestContext(request))
+        c = {
+            'poste': poste_dae,
+        }
+        return render(request, 'dae/poste_importer.html', c)
 
 
 @dae_groupe_requis
@@ -208,7 +206,7 @@ def poste(request, key=None):
         comparaisons_formset=comparaisons_formset
     ))
 
-    return render_to_response('dae/poste.html', vars, RequestContext(request))
+    return render(request, 'dae/poste.html', vars)
 
 
 @dae_groupe_requis
@@ -226,10 +224,11 @@ def postes_liste(request):
             .extra(select=extra_select) \
             .filter(~Q(etat=POSTE_ETAT_FINALISE)) \
             .order_by('-date_creation')
-    return render_to_response('dae/postes_liste.html', {
+    c = {
         'postes_a_traiter': postes_a_traiter,
         'postes_en_cours': postes_en_cours,
-    }, RequestContext(request))
+    }
+    return render(request, 'dae/postes_liste.html', c)
 
 
 @login_required
@@ -285,10 +284,7 @@ def embauche_consulter(request, dossier_id):
         'comparaisons_internes': comparaisons_internes,
         'comparaisons': comparaisons
     }
-
-    return render_to_response(
-        'dae/embauche_consulter.html', vars, RequestContext(request)
-    )
+    return render(request, 'dae/embauche_consulter.html', vars)
 
 
 @dae_groupe_requis
@@ -302,16 +298,18 @@ def embauche_importer(request, dossier_id=None):
         else:
             return redirect('embauches_finalisees')
     else:
-        return render_to_response('dae/embauche_importer.html', {
-            'dossier': dossier_dae
-        }, RequestContext(request))
+        c = {
+            'dossier': dossier_dae,
+        }
+        return render(request, 'dae/embauche_importer.html', c)
 
 
 @dae_groupe_requis
 def embauche_choisir_poste(request):
-    return render_to_response('dae/embauche-choisir-poste.html', {
-        'form': ChoosePosteForm(request=request)
-    }, RequestContext(request))
+    c = {
+        'form': ChoosePosteForm(request=request),
+    }
+    return render(request, 'dae/embauche-choisir-poste.html', c)
 
 
 @dae_groupe_requis
@@ -434,8 +432,7 @@ def embauche(request, key=None, dossier_id=None):
                 )
     except dae.Poste.DoesNotExist:
         comparaisons_internes = []
-
-    return render_to_response('dae/embauche.html', {
+    c = {
         'type_remun': filtered_type_remun(),
         'devises': devises(),
         'poste': poste,
@@ -444,8 +441,9 @@ def embauche(request, key=None, dossier_id=None):
         'remunForm': remunForm,
         'comparaisons_formset': comparaisons_formset,
         'forms': dict(employe=employe_form, dossier=dossier_form, ),
-        'comparaisons_internes': comparaisons_internes
-    }, RequestContext(request))
+        'comparaisons_internes': comparaisons_internes,
+    }
+    return render(request, 'dae/embauche.html', c)
 
 
 @dae_groupe_requis
@@ -466,10 +464,11 @@ def embauches_liste(request):
             .extra(select=extra_select) \
             .order_by('-date_creation') \
             .exclude(etat=DOSSIER_ETAT_FINALISE)
-    return render_to_response('dae/embauches_liste.html', {
+    c = {
         'embauches_a_traiter': embauches_a_traiter,
         'embauches_en_cours': embauches_en_cours,
-    }, RequestContext(request))
+    }
+    return render(request, 'dae/embauches_liste.html', c)
 
 
 @dae_groupe_requis
@@ -543,11 +542,12 @@ def embauches_finalisees(request):
         page = paginator.page(request.GET.get('page', 1))
     except InvalidPage:
         page = paginator.page(1)
-
-    return render_to_response('dae/embauches_finalisees.html', {
+        
+    c = {
         'embauches': page,
-        'search_form': search_form
-    }, RequestContext(request))
+        'search_form': search_form,
+    }
+    return render(request, 'dae/embauches_finalisees.html', c)
 
 
 def employe(request, key):
@@ -590,9 +590,10 @@ def contrat_supprimer(request, contrat):
         if 'oui' in request.POST:
             contrat.delete()
         return redirect('embauche_consulter', dossier_id=contrat.dossier.id)
-    return render_to_response('dae/contrat-supprimer.html', {
-        'contrat': contrat
-    }, RequestContext(request))
+    c = {
+        'contrat': contrat,
+    }
+    return render(request, 'dae/contrat-supprimer.html', c)
 
 
 @dae_groupe_requis
@@ -610,9 +611,11 @@ def embauche_ajouter_contrat(request, dossier_id=None):
             return redirect('embauche_consulter', dossier_id=dossier.id)
     else:
         form = ContratForm()
-    return render_to_response('dae/embauche-ajouter-contrat.html', {
-        'form': form
-    }, RequestContext(request))
+        
+    c = {
+        'form': form,
+    }
+    return render(request, 'dae/embauche-ajouter-contrat.html', c)
 
 
 ### DAE NUMERISEE
@@ -631,9 +634,11 @@ def dae_numerisee_modifier(request, dossier):
             return redirect('embauche_consulter', dossier_id=dossier.id)
     else:
         form = DAENumeriseeForm(instance=dossier)
-    return render_to_response('dae/dae_numerisee_modifier.html', {
-        'form': form
-    }, RequestContext(request))
+        
+    c = {
+        'form': form,
+    }
+    return render(request, 'dae/dae_numerisee_modifier.html', c)
 
 
 @get_object(dae.Dossier, 'modifier_dae_numerisee')
@@ -643,9 +648,7 @@ def dae_numerisee_supprimer(request, dossier):
             dossier.dae_numerisee = None
             dossier.save()
             return redirect('embauche_consulter', dossier_id=dossier.id)
-    return render_to_response(
-        'dae/dae_numerisee_supprimer.html', {}, RequestContext(request)
-    )
+    return render(request, 'dae/dae_numerisee_supprimer.html', {})
 
 
 # AJAX SECURISE
@@ -715,8 +718,7 @@ def dossier(request, poste_key, employe_key):
 
     dossier_form = DossierForm(initial=data, instance=dossier)
     vars = dict(form=dossier_form, poste=poste, employe=employe)
-    return render_to_response('dae/embauche-dossier.html', vars,
-                          RequestContext(request))
+    return render(request, 'dae/embauche-dossier.html', vars)
 
 
 #  Cette fonction est appelée à partir de fonctions déjà sécurisée
@@ -950,8 +952,10 @@ def add_remun(request, dossier, type_remun):
     dae.Remuneration(dossier=dossier, devise=dossier.devise,
                      type=type_remun).save()
 
-    return render_to_response('dae/embauche-remun.html', dict(dossier=dossier),
-                              RequestContext(request))
+    c = {
+        'dossier': dossier,
+    }
+    return render(request, 'dae/embauche-remun.html', c)
 
 
 def salaire(request, implantation, devise, classement):
index 1dfe1d3..30735be 100644 (file)
@@ -4,8 +4,8 @@ from django.contrib import messages
 from django.core.mail import EmailMultiAlternatives
 from django.core.urlresolvers import reverse
 from django.http import HttpResponseRedirect
-from django.shortcuts import render_to_response, redirect
-from django.template import Context, RequestContext, Template
+from django.shortcuts import render, redirect
+from django.template import Context, Template
 from django.views.static import serve
 
 from auf.django.emploi import models as emploi
@@ -37,9 +37,8 @@ def mediaserve(request, path, document_root=None, show_indexes=False):
 
 
 def index(request):
-    return render_to_response(
-        'recrutement/index.html', {}, RequestContext(request)
-    )
+    c = {}
+    return render(request, 'recrutement/index.html', c)
 
 
 def selectionner_template(request):
@@ -58,9 +57,10 @@ def selectionner_template(request):
     else:
         form = forms.CandidatCourrielTemplateForm()
 
-    return render_to_response("recrutement/selectionner_template.html", {
-        'form': form
-    }, context_instance=RequestContext(request))
+    c = {
+        'form': form,
+    }
+    return render(request, "recrutement/selectionner_template.html", c)
 
 
 def envoyer_courriel_candidats(request):
@@ -96,10 +96,10 @@ def envoyer_courriel_candidats(request):
                 'plain_text': template.plain_text
             }
         )
-
-    return render_to_response("recrutement/envoyer_courriel_candidats.html", {
-        'form': form
-    }, context_instance=RequestContext(request))
+    c = {
+        'form': form,
+    }
+    return render(request, "recrutement/envoyer_courriel_candidats.html", c)
 
 
 def affecter_evaluateurs_offre_emploi(request):
@@ -116,10 +116,10 @@ def affecter_evaluateurs_offre_emploi(request):
             return redirect("admin:recrutement_proxyoffreemploi_changelist")
     else:
         form = forms.EvaluateurForm(offres_emploi=offres_emploi)
-
-    return render_to_response("recrutement/affecter_evaluateurs.html", {
-        'form': form
-    }, context_instance=RequestContext(request))
+    c = {
+        'form': form,
+    }
+    return render(request, "recrutement/affecter_evaluateurs.html", c)
 
 
 def send_templated_email(candidat, template):
@@ -151,7 +151,7 @@ def send_templated_email(candidat, template):
 def candidat_pdf(request):
     candidat_id = request.GET.get('id')
     candidat = models.Candidat.objects.get(id=candidat_id)
-
-    return render_to_response("recrutement/candidat_pdf.html", {
-        'candidat': candidat
-    }, context_instance=RequestContext(request))
+    c = {
+        'candidat': candidat,
+    }
+    return render(request, "recrutement/candidat_pdf.html", c)
index eb9f40e..29a7d02 100644 (file)
@@ -14,23 +14,23 @@ from django.core.servers.basehttp import FileWrapper
 from django.core.urlresolvers import reverse
 from django.db.models import Q
 from django.http import HttpResponse
-from django.shortcuts import render_to_response, get_object_or_404
+from django.shortcuts import render, get_object_or_404
 from django.template import RequestContext
 from django.utils.encoding import smart_str
 
 from datamaster_modeles import models as ref
 
 from project.rh import models as rh
-from project.rh.lib import calc_remun
-from project.rh.decorators import drh_or_admin_required
-from project.rh.templatetags.rapports import SortHeaders
-from project.rh.change_list import RechercheTemporelle
 from project.rh import graph as rh_graph
+from project.rh.change_list import RechercheTemporelle
+from project.rh.decorators import drh_or_admin_required
+from project.rh.lib import calc_remun
 from project.rh.masse_salariale import MasseSalariale
+from project.rh.templatetags.rapports import SortHeaders
 
 # pas de reference a DAE devrait etre refactorisé
-from project.dae.utils import get_employe_from_user
 from project.dae.decorators import redirect_interdiction
+from project.dae.utils import get_employe_from_user
 from project.dae.workflow import grp_drh, grp_correspondants_rh
 
 
@@ -44,7 +44,7 @@ def profil(request):
 
     c['user'] = request.user
     c['employe'] = employe
-    return render_to_response('rh/profil.html', c, rc)
+    return render(request, 'rh/profil.html', c)
 
 
 @login_required
@@ -59,9 +59,7 @@ def employes_liste(request):
         'user': request.user,
         'employes': employes,
         }
-    return render_to_response('rh/employes_liste.html',
-            c,
-            RequestContext(request))
+    return render(request, 'rh/employes_liste.html', c)
 
 
 @login_required
@@ -75,7 +73,7 @@ def employe(request, id):
         'user': request.user,
         'employe': employe,
         }
-    return render_to_response('rh/employe.html', c, RequestContext(request))
+    return render(request, 'rh/employe.html', c)
 
 
 @login_required
@@ -169,9 +167,7 @@ def rapports_poste(request):
         'headers': list(h.headers()),
     }
 
-    return render_to_response(
-        'rh/rapports/postes.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/postes.html', c)
 
 
 @login_required
@@ -238,9 +234,7 @@ def rapports_contrat(request):
         'headers': list(h.headers()),
     }
 
-    return render_to_response(
-        'rh/rapports/contrats.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/contrats.html', c)
 
 
 @login_required
@@ -349,9 +343,7 @@ def rapports_masse_salariale(request):
                 c['url_ods'] = "%s?%s" % (
                         reverse('rhr_masse_salariale'), query_string)
 
-    return render_to_response(
-        'rh/rapports/masse_salariale.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/masse_salariale.html', c)
 
 
 @login_required
@@ -433,9 +425,7 @@ def rapports_remuneration(request):
         'headers': list(h.headers()),
     }
 
-    return render_to_response(
-        'rh/rapports/remuneration.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/remuneration.html', c)
 
 
 @login_required
@@ -496,9 +486,7 @@ def rapports_employe_sans_contrat(request):
         'headers': list(h.headers()),
     }
 
-    return render_to_response(
-        'rh/rapports/employes_sans_contrat.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/employes_sans_contrat.html', c)
 
 
 @login_required
@@ -526,9 +514,7 @@ def rapports_postes_modelisation(request):
 
     c['data'] = data
 
-    return render_to_response(
-        'rh/rapports/postes_modelisation.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/postes_modelisation.html', c)
 
 
 @login_required
@@ -551,9 +537,7 @@ def rapports_postes_implantation(request):
 
     c['data'] = data
 
-    return render_to_response(
-        'rh/rapports/postes_implantation.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/postes_implantation.html', c)
 
 
 @login_required
@@ -567,9 +551,7 @@ def rapports_postes_service(request):
         data.append({'service': s, 'num_postes': num_postes, 'postes': postes})
 
     c['data'] = data
-    return render_to_response(
-        'rh/rapports/postes_service.html', c, RequestContext(request)
-    )
+    return render(request, 'rh/rapports/postes_service.html', c)
 
 
 def region_protected(model):
@@ -605,9 +587,7 @@ def dossier_apercu(request, dossier_id):
         'commentaires': rh.DossierCommentaire.objects.filter(dossier=d).all(),
         'media_url': settings.PRIVE_MEDIA_URL,
     }
-    return render_to_response(
-        'admin/rh/dossier/apercu.html', c, RequestContext(request)
-    )
+    return render(request, 'admin/rh/dossier/apercu.html', c)
 
 
 @region_protected(rh.Poste)
@@ -633,9 +613,7 @@ def poste_apercu(request, poste_id):
         ),
         'media_url': settings.PRIVE_MEDIA_URL,
     }
-    return render_to_response(
-        'admin/rh/poste/apercu.html', c, RequestContext(request)
-    )
+    return render(request, 'admin/rh/poste/apercu.html', c)
 
 
 def employe_apercu(request, employe_id):
@@ -661,9 +639,7 @@ def employe_apercu(request, employe_id):
         'dossiers': dossiers,
         'media_url': settings.PRIVE_MEDIA_URL,
     }
-    return render_to_response(
-        'admin/rh/employe/apercu.html', c, RequestContext(request)
-    )
+    return render(request, 'admin/rh/employe/apercu.html', c)
 
 
 @login_required
@@ -738,14 +714,13 @@ def organigrammes_employe(request, id, level="all"):
     }
 
     if 'forcer' in request.GET:
-        response = HttpResponse(svg, mimetype='image/svg+xml')
+        response = HttpResponse(svg, content_type='image/svg+xml')
         response['Content-Disposition'] = \
                 'attachment; filename=organigramme.svg'
         return response
 
-    return render_to_response(
-        'rh/organigrammes/employe.html', c, RequestContext(request),
-        mimetype="image/svg+xml"
+    return render(request, 'rh/organigrammes/employe.html', c,
+        content_type="image/svg+xml"
     )
 
 
@@ -763,9 +738,8 @@ def organigrammes_service(request, id):
         'svg': svg
     }
 
-    return render_to_response(
-        'rh/organigrammes/vide.html', c, RequestContext(request),
-        mimetype="image/svg+xml"
+    return render(request, 'rh/organigrammes/vide.html', c, 
+        content_type="image/svg+xml"
     )
 
 
@@ -783,9 +757,8 @@ def organigrammes_implantation(request, id):
         'svg': svg
     }
 
-    return render_to_response(
-        'rh/organigrammes/vide.html', c, RequestContext(request),
-        mimetype="image/svg+xml"
+    return render(request, 'rh/organigrammes/vide.html', c, 
+        content_type="image/svg+xml"
     )
 
 
@@ -803,7 +776,7 @@ def organigrammes_region(request, id):
         'svg': svg
     }
 
-    return render_to_response(
-        'rh/organigrammes/vide.html', c, RequestContext(request),
-        mimetype="image/svg+xml"
+    return render(request, 
+        'rh/organigrammes/vide.html', c, 
+        content_type="image/svg+xml"
     )
index 58d5403..92df683 100644 (file)
@@ -4,11 +4,9 @@ import os
 from sendfile import sendfile
 
 from django.conf import settings
-from django.shortcuts import render_to_response
-from django.template import RequestContext
 from django.contrib.auth.decorators import login_required
 from django.contrib.auth.views import login
-from django.shortcuts import redirect, render_to_response, get_object_or_404
+from django.shortcuts import get_object_or_404
 
 # pas de reference a DAE devrait etre refactorisé
 from project.dae.decorators import dae_groupe_requis