From 9382a1af867683961d175dd936186f3a93c47795 Mon Sep 17 00:00:00 2001
From: Marcin Kulik <m@ku1ik.com>
Date: Fri, 15 Apr 2016 10:15:02 +0200
Subject: [PATCH] Fix serializer

---
 app/serializers/asciicast_serializer.rb | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/app/serializers/asciicast_serializer.rb b/app/serializers/asciicast_serializer.rb
index 34de05e..b49d8a5 100644
--- a/app/serializers/asciicast_serializer.rb
+++ b/app/serializers/asciicast_serializer.rb
@@ -11,11 +11,12 @@ class AsciicastSerializer < ActiveModel::Serializer
     if v0_url?
       object.stdout_frames_url
     else
-      url = object.data.url
-      if url.starts_with?("/")
-        asciicast_path(object, format: :json) # download URL for local carrierwave store
-      else
-        url # direct signed S3 URL (because XHR CORS redirect doesn't work under Safari)
+      if url = object.data.url
+        if url.starts_with?("/")
+          asciicast_path(object, format: :json) # download URL for local carrierwave store
+        else
+          url # direct signed S3 URL (because XHR CORS redirect doesn't work under Safari)
+        end
       end
     end
   end