Migration project code_review to django

Author: xtao

app.py

@@ -35,6 +35,7 @@
              (re.compile(r'/\w+/\w+/blame'), django_app),
              (re.compile(r'/\w+/\w+/raw'), django_app),
              (re.compile(r'/\w+/\w+/browsefiles'), django_app),
+             (re.compile(r'/\w+/\w+/code_review'), django_app),
              (re.compile(r'/vilya'), django_app),
              (re.compile(r'/.*'), web)]
 

vilya/urls.py

@@ -106,6 +106,8 @@
     url(r'^(?P<username>\w+)/(?P<projectname>\w+)/commits/(?P<revision>\w+)/(?P<path>.*)$', project.ProjectCommitsView.as_view(), name="project_commits"),
     url(r'^(?P<username>\w+)/(?P<projectname>\w+)/tree/(?P<revision>\w+)/(?P<path>.*)$', project.ProjectTreeView.as_view(), name="project_tree"),
     url(r'^(?P<username>\w+)/(?P<projectname>\w+)/browsefiles/?$', project.browsefiles, name="project_browsefiles"),
+    url(r'^(?P<username>\w+)/(?P<projectname>\w+)/code_review/(?P<id>[0-9]+)/delete/?$', project.codereview_delete, name="project_codereview_delete"),
+    url(r'^(?P<username>\w+)/(?P<projectname>\w+)/code_review/(?P<id>[0-9]+)/edit/?$', project.codereview_edit, name="project_codereview_edit"),
     url(r'^(?P<username>\w+)/(?P<projectname>\w+)/watchers/?$', project.watchers, name="project_watchers"),
     url(r'^(?P<username>\w+)/(?P<projectname>\w+)/forkers/?$', project.forkers, name="project_forkers"),
     url(r'^(?P<username>\w+)/(?P<projectname>\w+)/archive/(?P<revision>\w+)/?$', project.archive, name="project_archive"),

vilya/views/__init__.py

@@ -18,7 +18,6 @@
 from vilya.views.uis.compare import CompareUI
 from vilya.views.uis.comments import CommentUI
 from vilya.views.uis.line_comments import LineCommentUI
-from vilya.views.uis.code_review import CodeReviewUI
 from vilya.views.uis.pr_comment import PrCommentUI
 from vilya.views.uis.issue import IssueBoardUI, IssueCommentUI
 from vilya.views.util import jsonize
@@ -128,7 +127,7 @@ class CodeUI:
     _q_exports = [
         'hooks', 'graph', 'commit', 'pull', 'newpull', 'comments',
         'compare', 'line_comments', 'pulls',
-        'docs', 'remove', 'code_review', 'pr_comment', 'issues',
+        'docs', 'remove', 'pr_comment', 'issues',
         'issue_comments', 're_index_docs', 'src_index',
         'search', 'pages', 'xdocs', 'dashboard',
     ]
@@ -211,10 +210,6 @@ def comments(self):
     def line_comments(self):
         return LineCommentUI(self.proj_name)
 
-    @property
-    def code_review(self):
-        return CodeReviewUI(self.proj_name)
-
     @property
     def pr_comment(self):
         return PrCommentUI(self.proj_name)

vilya/views/django/project.py

@@ -643,3 +643,40 @@ def _add_file_type_and_warns(node):
         return HttpResponse(json.dumps(allfiles))
     else:
         return HttpResponse(st('browsefiles.html', **locals()))
+
+
+@csrf_exempt
+def codereview_delete(request, username, projectname, id):
+    from vilya.models.linecomment import PullLineComment
+    comment = PullLineComment.get(id)
+    if not comment:
+        raise Http404("Unable to find comment %s" % id)
+
+    user = request.user
+    if comment.author == user.name:
+        ok = comment.delete()
+        if ok:
+            return HttpResponse(json.dumps({'r': 1}))  # FIXME: 这里 r=1 表示成功，跟其他地方不统一
+    return HttpResponse(json.dumps({'r': 0}))
+
+
+@csrf_exempt
+def codereview_edit(request, username, projectname, id):
+    from vilya.models.linecomment import PullLineComment
+    from vilya.models.project import CodeDoubanProject
+    name = '/'.join([username, projectname])
+    comment = PullLineComment.get(id)
+    if not comment:
+        raise Http404("Unable to find comment %s" % id)
+
+    user = request.user
+    project = CodeDoubanProject.get_by_name(name)
+    content = request.POST.get(
+        'pull_request_review_comment', '').decode('utf-8')
+    if comment.author == user.name:
+        comment.update(content)
+        linecomment = PullLineComment.get(comment.id)
+        pullreq = True
+        return HttpResponse(json.dumps(dict(
+            r=0, html=st('/pull/ticket_linecomment.html', **locals()))))
+    return HttpResponse(json.dumps(dict(r=1)))