commit c6e05f333f61e33f878f9c89c2ece43ea50976ef
author Edwin Kempin <edwin.kempin@sap.com> Mon Feb 03 15:59:09 2014 +0100
committer Edwin Kempin <edwin.kempin@sap.com> Mon Feb 03 15:59:09 2014 +0100
tree 9265661f38a036ea4f99b5b7e528b2490eeef012
parent d35c813b20ea34ef01a8bd112e3615f33e535c10

Use the 'ref/images/' ref in the image URL rather than the commit ID

This makes the resolution of the URL faster since the read access right
can be checked for this ref rather than finding all refs that contain
the commit ID and then checking the read access right on those refs.

Change-Id: I9bb45d75ddca6955d2584d8085b31999be07c193
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>

src/main/java/com/googlesource/gerrit/plugins/imagare/PostImage.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/imagare/PostImage.java b/src/main/java/com/googlesource/gerrit/plugins/imagare/PostImage.java
index 33470ef..5abfc5c 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/imagare/PostImage.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/imagare/PostImage.java
@@ -142,7 +142,7 @@
       ObjectId commitId = repo.resolve(ref);
       if (commitId != null) {
         // this image exists already
-        return getUrl(pc.getProject().getNameKey(), commitId.getName(), fileName);
+        return getUrl(pc.getProject().getNameKey(), ref, fileName);
       }
 
       RevWalk rw = new RevWalk(repo);
@@ -183,7 +183,7 @@
                 pc.getProject().getName(), ru.getResult()));
           }
 
-          return getUrl(pc.getProject().getNameKey(), commitId.getName(), fileName);
+          return getUrl(pc.getProject().getNameKey(), ref, fileName);
         } finally {
           oi.release();
         }