Merge branch 'master' of git@stigma:platforma

[redakcja.git] / apps / explorer / views.py

diff --git a/apps/explorer/views.py b/apps/explorer/views.py
index 90115aa..8d80d02 100644 (file)
--- a/apps/explorer/views.py
+++ b/apps/explorer/views.py
@@ -1,5 +1,6 @@
 from librarian import html
 from librarian import html

-import hg, urllib2, json
+import hg, urllib2
+from django.utils import simplejson as json

 
 from django.views.generic.simple import direct_to_template
 from django.conf import settings
 
 from django.views.generic.simple import direct_to_template
 from django.conf import settings


@@ -14,15 +15,18 @@ def file_list(request):
         'objects': repo.all_files(),
     })
 
         'objects': repo.all_files(),
     })
 

-

 def file_xml(request, path):
     if request.method == 'POST':
         form = forms.BookForm(request.POST)
         if form.is_valid():
             repo.add_file(path, form.cleaned_data['text'])
 def file_xml(request, path):
     if request.method == 'POST':
         form = forms.BookForm(request.POST)
         if form.is_valid():
             repo.add_file(path, form.cleaned_data['text'])

-            # issues = _get_issues_for_file(path)
-            # commit_message = _add_references(form.cleaned_data['commit_message'], issued)
-            repo.commit(message=form.cleaned_data['commit_message'], user=form.cleaned_data['user'])
+            
+            # add references to comment
+            issues = _get_issues_for_file(path)
+            commit_message = _add_references(form.cleaned_data['commit_message'], issues)
+            print 'Commiting with: ' + commit_message
+
+            repo.commit(message=commit_message, user=form.cleaned_data['user'])

             return HttpResponseRedirect(request.get_full_path())
     else:
         form = forms.BookForm()
             return HttpResponseRedirect(request.get_full_path())
     else:
         form = forms.BookForm()


@@ -48,17 +52,19 @@ def folder_images(request, folder):
     })
 
 def _add_references(message, issues):
     })
 
 def _add_references(message, issues):

-    # TODO
-    pass
+    return message + " - " + ", ".join(map(lambda issue: "Refs #%d" % issue['id'], issues))

 
 def _get_issues_for_file(path):
     if not path.endswith('.xml'):
         raise ValueError('Path must end with .xml')
 
 
 def _get_issues_for_file(path):
     if not path.endswith('.xml'):
         raise ValueError('Path must end with .xml')
 

-    book_id = path[:-3]
-    uf = urllib2.urlopen(settings.REDMINE_URL + 'publications/%s/issues' % book_id)
+    book_id = path[:-4]
+    uf = None

 
     try:
 
     try:

+        uf = urllib2.urlopen(settings.REDMINE_URL + 'publications/issues/%s.json' % book_id)

         return json.loads(uf.read())
         return json.loads(uf.read())

+    except urllib2.HTTPError:
+        return []

     finally:
     finally:

-        uf.close()
+        if uf: uf.close()