[#1451] pep8 and a few things to make things a look nicer

ckan · Jan 24, 2014 · 155fd9b · amercader · Jan 24, 2014 · 155fd9b
1 parent a0b7780
commit 155fd9b
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/ckanext/datapusher/logic/action.py b/ckanext/datapusher/logic/action.py
@@ -45,9 +45,12 @@ def datapusher_submit(context, data_dict):
 
     datapusher_url = pylons.config.get('ckan.datapusher.url')
 
-    callback_url =  pylons.config['ckan.site_url'].rstrip('/') + p.toolkit.url_for(
-        controller='api', action='action', logic_function='datapusher_hook',
-        ver=3)
+    site_url = pylons.config['ckan.site_url']
+
+    callback_url = site_url + p.toolkit.url_for(
+        controller='api', action='action',
+        logic_function='datapusher_hook', ver=3
+    )
 
     user = p.toolkit.get_action('user_show')(context, {'id': context['user']})
 
@@ -86,7 +89,7 @@ def datapusher_submit(context, data_dict):
                 'job_type': 'push_to_datastore',
                 'result_url': callback_url,
                 'metadata': {
-                    'ckan_url': pylons.config['ckan.site_url'],
+                    'ckan_url': site_url,
                     'resource_id': res_id,
                     'set_url_type': data_dict.get('set_url_type', False)
                 }