Retire des chemins hardcodés

Dans la vue `partitions.views.see`, les chemins vers les fichiers
ne sont plus hardcodés.

partitions/urls.py

@@ -6,7 +6,7 @@ urlpatterns = patterns('partitions.views',
     url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/upload/?$', 'upload'),
     url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/delete/?$', 'delete_morc'),
     url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/conf/?$', 'conf_delete_morc'),
-    url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/see/(?P<file_name>[^/]+)/?$', 'see'),
+    url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/see/(?P<partition_id>\d+)/?$', 'see'),
     url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/(?P<file_name>[^/]+)/$', 'download'),
     url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/(?P<id>[^/]+)/delete/?$', 'delete'),
     url(r'^(?P<nom>[^/]+)/(?P<auteur>[^/]+)/(?P<id>[^/]+)/conf/?$', 'conf_delete'),

partitions/views.py

@@ -3,6 +3,7 @@ from django.shortcuts import render, redirect, HttpResponse, render_to_response,
 from django.contrib.auth.decorators import login_required
 from django.utils.encoding import smart_str
 from django.forms.models import modelform_factory
+from django.core.files import File
 
 import os
 
@@ -46,21 +47,24 @@ def upload(request, nom, auteur):
     return render(request, 'partitions/upload.html', locals()) 
 
 @login_required
-def see(request, nom, auteur, file_name):
-    if ".pdf" in file_name:
-        abspath=open('/var/www_new/media/partitions/' + file_name, 'rb')
-        response=HttpResponse(content=abspath.read())
-        response["Content-Type"]="application/pdf"
-        response["Content-Disposition"]="inline; filename= %s" % (nom +'-'+ auteur +'-' + smart_str(file_name))
-        abspath.close()
+def see(request, nom, auteur, partition_id):
+    partition = get_object_or_404(Partition, id=partition_id)
+    _, extension = os.path.splitext(partition.part.path)
+    if ".pdf" == extension:
+        with open(partition.part.path, 'rb') as f:
+            myfile = File(f)
+            response = HttpResponse(content=myfile.read())
+            response["Content-Type"] = "application/pdf"
+            response["Content-Disposition"] = "inline; filename= %s-%s-%s" % (
+                nom, auteur, smart_str(os.path.basename(myfile.name)))
         return response
-    elif ".mp3" in file_name:
-        abspath=open('/var/www_new/media/partitions/' + file_name, 'rb')
-        response = HttpResponse()
-        response.write(abspath.read())
-        response["Content-Type"]="audio/mp3"
-        response["Content-Length"]= os.path.getsize('/var/www_new/media/partitions/' + file_name)     
-        abspath.close()
+    elif ".mp3" == extension:
+        with open(partition.part.path, 'rb') as f:
+            myfile = File(f)
+            response = HttpResponse()
+            response.write(myfile.read())
+            response["Content-Type"] = "audio/mp3"
+            response["Content-Length"] = myfile.size
         return response
     else:
         partitions = PartitionSet.objects.get(nom=nom, auteur=auteur)

templates/partitions/listepart.html

@@ -30,9 +30,9 @@
     <ul class="filelist">
     {% for p in part %}
         {% if user.is_authenticated and ".pdf" in p.part.url %}
-            <li><a href="{% url "partitions.views.see" nom auteur p.part.url|cuturl %}" class="fichier">{{ p.nom }}</a>
+            <li><a href="{% url "partitions.views.see" nom auteur p.id %}" class="fichier">{{ p.nom }}</a>
         {% elif user.is_authenticated and ".mp3" in p.part.url %}
-            <li><a href="{% url "partitions.views.see" nom auteur p.part.url|cuturl %}" class="fichier">{{ p.nom }}</a>
+            <li><a href="{% url "partitions.views.see" nom auteur p.id %}" class="fichier">{{ p.nom }}</a>
         {% else %}
             <li>{{ p.nom }}
         {% endif %}