Download blob viewer is client-side

36e7b322 · Douwe Maan · 7cc4546b · 36e7b322 · 36e7b322
Commit 36e7b322 authored Apr 20, 2017 by Douwe Maan
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

app/models/blob_viewer/base.rb app/models/blob_viewer/base.rb +1 -1

app/models/blob_viewer/download.rb app/models/blob_viewer/download.rb +3 -1

No files found.
--- a/app/models/blob_viewer/base.rb
+++ b/app/models/blob_viewer/base.rb
@@ -75,7 +75,7 @@ module BlobViewer
    end
    def server_side_but_stored_in_lfs?
-      !client_side? && blob.valid_lfs_pointer?
+      server_side? && blob.valid_lfs_pointer?
    end
  end
 end
--- a/app/models/blob_viewer/download.rb
+++ b/app/models/blob_viewer/download.rb
 module BlobViewer
  class Download < Base
    include Simple
-    include ServerSide
+    # We pretend the Download viewer is rendered client-side so that it doesn't
+    # attempt to load the entire blob contents.
+    include ClientSide
    self.partial_name = 'download'
    self.text_based = false