svn commit: r1503125 - in /bloodhound/trunk/bloodhound_relations/bhrelations: tests/web_ui.py web_ui.py

[as...@apache.org]

Author: astaric
Date: Mon Jul 15 07:30:47 2013
New Revision: 1503125

URL: http://svn.apache.org/r1503125
Log:
Check path_info instead of presence of 'ticket' in data to detect ticket page.

Fix suggested by rjollos in #588:comment:3.
Refs: #588

Modified:
    bloodhound/trunk/bloodhound_relations/bhrelations/tests/web_ui.py
    bloodhound/trunk/bloodhound_relations/bhrelations/web_ui.py

Modified: bloodhound/trunk/bloodhound_relations/bhrelations/tests/web_ui.py
URL: http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_relations/bhrelations/tests/web_ui.py?rev=1503125&r1=1503124&r2=1503125&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_relations/bhrelations/tests/web_ui.py (original)
+++ bloodhound/trunk/bloodhound_relations/bhrelations/tests/web_ui.py Mon Jul 15 07:30:47 2013
@@ -149,6 +149,11 @@ class ResolveTicketIntegrationTestCase(B
         self.req.path_info = '/newticket'
         self.process_request()
 
+    def test_post_process_request_can_handle_none_data(self):
+        self.req.path_info = '/source'
+        RelationManagementModule(self.env).post_process_request(
+            self.req, '', None, '')
+
     def resolve_as_duplicate(self, ticket, duplicate_id):
         self.req.method = 'POST'
         self.req.path_info = '/ticket/%d' % ticket.id

Modified: bloodhound/trunk/bloodhound_relations/bhrelations/web_ui.py
URL: http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_relations/bhrelations/web_ui.py?rev=1503125&r1=1503124&r2=1503125&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_relations/bhrelations/web_ui.py (original)
+++ bloodhound/trunk/bloodhound_relations/bhrelations/web_ui.py Mon Jul 15 07:30:47 2013
@@ -131,7 +131,7 @@ class RelationManagementModule(Component
         return handler
 
     def post_process_request(self, req, template, data, content_type):
-        if 'ticket' in data:
+        if req.path_info.startswith('/ticket/'):
             ticket = data['ticket']
             rls = RelationsSystem(self.env)
             try: