You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by tv...@apache.org on 2014/01/28 22:37:12 UTC

git commit: [#7114] Don't hardcode nbhd prefix

Updated Branches:
  refs/heads/tv/7114 [created] c6b5510a5


[#7114] Don't hardcode nbhd prefix

Signed-off-by: Tim Van Steenburgh <tv...@gmail.com>


Project: http://git-wip-us.apache.org/repos/asf/incubator-allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-allura/commit/c6b5510a
Tree: http://git-wip-us.apache.org/repos/asf/incubator-allura/tree/c6b5510a
Diff: http://git-wip-us.apache.org/repos/asf/incubator-allura/diff/c6b5510a

Branch: refs/heads/tv/7114
Commit: c6b5510a5d42cf2a23d69fd75b2ce62c9edd0576
Parents: 036905c
Author: Tim Van Steenburgh <tv...@gmail.com>
Authored: Tue Jan 28 21:36:57 2014 +0000
Committer: Tim Van Steenburgh <tv...@gmail.com>
Committed: Tue Jan 28 21:36:57 2014 +0000

----------------------------------------------------------------------
 Allura/allura/lib/import_api.py                     |  1 +
 ForgeTracker/forgetracker/scripts/import_tracker.py | 10 +++++++---
 scripts/allura_import.py                            |  7 ++++++-
 3 files changed, 14 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/c6b5510a/Allura/allura/lib/import_api.py
----------------------------------------------------------------------
diff --git a/Allura/allura/lib/import_api.py b/Allura/allura/lib/import_api.py
index 9e383b5..d8159c5 100644
--- a/Allura/allura/lib/import_api.py
+++ b/Allura/allura/lib/import_api.py
@@ -57,6 +57,7 @@ class AlluraImportApiClient(object):
                 resp = result.read()
                 return json.loads(resp)
             except urllib2.HTTPError, e:
+                e.msg += ' ({0})'.format(url)
                 if self.verbose:
                     error_content = e.read()
                     e.msg += '. Error response:\n' + error_content

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/c6b5510a/ForgeTracker/forgetracker/scripts/import_tracker.py
----------------------------------------------------------------------
diff --git a/ForgeTracker/forgetracker/scripts/import_tracker.py b/ForgeTracker/forgetracker/scripts/import_tracker.py
index ba8e93f..965fafa 100644
--- a/ForgeTracker/forgetracker/scripts/import_tracker.py
+++ b/ForgeTracker/forgetracker/scripts/import_tracker.py
@@ -28,7 +28,12 @@ log = logging.getLogger(__name__)
 
 def import_tracker(cli, project, tool, import_options, doc_txt,
                    validate=True, verbose=False, cont=False):
-    url = '/rest/p/' + project + '/' + tool
+    from allura import model as M
+    p = M.Project.query.get(shortname=project)
+    url = '/rest/{project_url}/{tool}'.format(
+            project_url=p.url().strip('/'),
+            tool=tool,
+            )
     if validate:
         url += '/validate_import'
     else:
@@ -36,8 +41,7 @@ def import_tracker(cli, project, tool, import_options, doc_txt,
 
     existing_map = {}
     if cont:
-        existing_tickets = cli.call(
-            '/rest/p/' + project + '/' + tool + '/')['tickets']
+        existing_tickets = cli.call(url + '/')['tickets']
         for t in existing_tickets:
             existing_map[t['ticket_num']] = t['summary']
 

http://git-wip-us.apache.org/repos/asf/incubator-allura/blob/c6b5510a/scripts/allura_import.py
----------------------------------------------------------------------
diff --git a/scripts/allura_import.py b/scripts/allura_import.py
index 56bc5d5..63d14f1 100644
--- a/scripts/allura_import.py
+++ b/scripts/allura_import.py
@@ -70,7 +70,12 @@ def main():
 
 
 def import_forum(cli, project, tool, user_map, doc_txt, validate=True):
-    url = '/rest/p/' + project + '/' + tool
+    from allura import model as M
+    p = M.Project.query.get(shortname=project)
+    url = '/rest/{project_url}/{tool}'.format(
+            project_url=p.url().strip('/'),
+            tool=tool,
+            )
     if validate:
         url += '/validate_import'
         print cli.call(url, doc=doc_txt, user_map=json.dumps(user_map))