From 1d05e64b6dc5ba4ed1e21dd2975dcd7faf36795d Mon Sep 17 00:00:00 2001
From: Ryuta Kamizono <kamipo@gmail.com>
Date: Sun, 9 Apr 2017 14:53:21 +0900
Subject: Tweaks #28678

* Fix the comment on `formatted_version`
* Extract `define_params`
* Remove duplicated guard clause for `@version`
---
 activerecord/lib/active_record/schema_dumper.rb | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'activerecord/lib/active_record/schema_dumper.rb')

diff --git a/activerecord/lib/active_record/schema_dumper.rb b/activerecord/lib/active_record/schema_dumper.rb
index 657bd43b86..94d63765c9 100644
--- a/activerecord/lib/active_record/schema_dumper.rb
+++ b/activerecord/lib/active_record/schema_dumper.rb
@@ -47,17 +47,18 @@ module ActiveRecord
         @options = options
       end
 
-      # turns 20170404131909 into "2017_04_04131909"
+      # turns 20170404131909 into "2017_04_04_131909"
       def formatted_version
-        return "" unless @version
         stringified = @version.to_s
         return stringified unless stringified.length == 14
         stringified.insert(4, "_").insert(7, "_").insert(10, "_")
       end
 
-      def header(stream)
-        define_params = @version ? "version: #{formatted_version}" : ""
+      def define_params
+        @version ? "version: #{formatted_version}" : ""
+      end
 
+      def header(stream)
         stream.puts <<HEADER
 # This file is auto-generated from the current state of the database. Instead
 # of editing this file, please use the migrations feature of Active Record to
-- 
cgit v1.2.3