| commit | cafc080ceb653d9a35c125f7d588dc8024308a2b | [log] [tgz] |
|---|---|---|
| author | David Ostrovsky <david@ostrovsky.org> | Fri Feb 06 09:36:03 2015 +0100 |
| committer | David Ostrovsky <david@ostrovsky.org> | Fri Feb 06 09:41:08 2015 +0100 |
| tree | c4192a210fd425195ab0d577099eb02f13aea749 | |
| parent | 4cee8e9d16c8865b5604f1d37a85eeab85737bb5 [diff] |
Fix binary file diff for anonymous users Restore Gerrit.isSignedIn() check that was erroneously removed in I7ec4a0beb. Change-Id: Idbfd873e555c682b79e61ab220d6b57db9039e2e
diff --git a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchTable.java b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchTable.java index 218a6c3..178a583 100644 --- a/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchTable.java +++ b/gerrit-gwtui/src/main/java/com/google/gerrit/client/patches/PatchTable.java
@@ -261,8 +261,9 @@ private static boolean isUnifiedPatchLink(final Patch patch) { return (patch.getPatchType().equals(PatchType.BINARY) - || Gerrit.getUserAccount().getGeneralPreferences().getDiffView() - .equals(DiffView.UNIFIED_DIFF)); + || (Gerrit.isSignedIn() + && Gerrit.getUserAccount().getGeneralPreferences().getDiffView() + .equals(DiffView.UNIFIED_DIFF))); } private static String getFileNameOnly(Patch patch) {