kallithea Changeset - 2e294e1fb9e2

Changeset - 2e294e1fb9e2

Parent rev.

Child rev.

[Not reviewed]

default

0 3 0

Mads Kiilerich - 9 years ago 2016-08-04 14:23:36
madski@unity3d.com

routing: use POST for changeset_comment_delete and pullrequest_comment_delete instead of DELETE

3 files changed with 6 insertions and 7 deletions:

kallithea/config/routing.py

kallithea/public/js/base.js

kallithea/tests/functional/test_changeset_comments.py

0 comments (0 inline, 0 general)

kallithea/config/routing.py

➞

Show inline comments

@@ @@ -627,9 +627,9 @@ def make_map(config): @@
                 conditions=dict(function=check_repo))
     rmap.connect('changeset_comment_delete',
-                 '/{repo_name:.*?}/changeset-comment-delete/{comment_id}',
+                 '/{repo_name:.*?}/changeset-comment/{comment_id}/delete',
                 controller='changeset', action='delete_comment',
-                conditions=dict(function=check_repo, method=["DELETE"]))
+                conditions=dict(function=check_repo, method=["POST"]))
     rmap.connect('changeset_info', '/changeset_info/{repo_name:.*?}/{revision}',
                  controller='changeset', action='changeset_info')
@@ @@ -699,7 +699,7 @@ def make_map(config): @@
     rmap.connect('pullrequest_comment_delete',
                  '/{repo_name:.*?}/pull-request-comment/{comment_id}/delete',
                 controller='pullrequests', action='delete_comment',
-                conditions=dict(function=check_repo, method=["DELETE"]))
+                conditions=dict(function=check_repo, method=["POST"]))
     rmap.connect('summary_home_summary', '/{repo_name:.*?}/summary',
                 controller='summary', conditions=dict(function=check_repo))

kallithea/public/js/base.js

➞

Show inline comments

@@ @@ -758,7 +758,7 @@ function _comment_div_append_form($comme @@
 function deleteComment(comment_id) {
     var url = AJAX_COMMENT_DELETE_URL.replace('__COMMENT_ID__', comment_id);
-    var postData = {'_method': 'delete'};
     var postData = {};
     var success = function(o) {
         $('#comment-'+comment_id).remove();
         // Ignore that this might leave a stray Add button (or have a pending form with another comment) ...

kallithea/tests/functional/test_changeset_comments.py

➞

Show inline comments

@@ @@ -127,11 +127,10 @@ class TestChangeSetCommentsController(Te @@
         assert len(comments) == 1
         comment_id = comments[0].comment_id
         self.app.post(url(controller='changeset',
                                     action='delete_comment',
         self.app.post(url("changeset_comment_delete",
                                     repo_name=HG_REPO,
                                     comment_id=comment_id),
-            params={'_method': 'delete', '_authentication_token': self.authentication_token()})
             params={'_authentication_token': self.authentication_token()})
         comments = ChangesetComment.query().all()
         assert len(comments) == 0

0 comments (0 inline, 0 general)