Store :gauth_enabled as a boolean

lib/devise_google_authenticatable/models/google_authenticatable.rb

@@ -19,7 +19,7 @@ def find_by_gauth_tmp(gauth_tmp)
       end
 
       def needs_gauth?
-        gauth_enabled == 1 && gauth_secret?
+        gauth_enabled && gauth_secret?
       end
 
       def get_qr
@@ -69,11 +69,14 @@ def validate_token(token)
       end
 
       def google_authenticator_qrcode_url
-        return unless gauth_secret?
+        data = Rack::Utils.escape google_authenticator_registration_url
+        "https://chart.googleapis.com/chart?chs=200x200&chld=M|0&cht=qr&chl=#{data}"
+      end
 
-        data = Rack::Utils.escape "otpauth://totp/#{ga_username_from_email(email)}@#{Devise.http_authentication_realm || Rails.application.class.parent_name}?secret=#{gauth_secret}"
+      def google_authenticator_registration_url
+        return unless gauth_secret?
 
-        "https://chart.googleapis.com/chart?chs=200x200&chld=M|0&cht=qr&chl=#{data}"
+        "otpauth://totp/#{ga_username_from_email(email)}@#{Devise.http_authentication_realm || Rails.application.class.parent_name}?secret=#{gauth_secret}"
       end
 
       private
@@ -87,4 +90,4 @@ def ga_username_from_email(email)
       end
     end
   end
-end
+end

lib/devise_google_authenticatable/schema.rb

@@ -22,7 +22,7 @@ def gauth_secret
     end
 
     def gauth_enabled
-      apply_devise_schema :gauth_enabled, Integer, {:default => 0}
+      apply_devise_schema :gauth_enabled, Boolean, { default: false }
     end
 
     def gauth_tmp

lib/generators/active_record/templates/migration.rb

@@ -2,7 +2,7 @@ class DeviseGoogleAuthenticatorAddTo<%= table_name.camelize %> < ActiveRecord::M
   def self.up
     change_table :<%= table_name %> do |t|
       t.string  :gauth_secret
-      t.string  :gauth_enabled, :default => "f"
+      t.boolean  :gauth_enabled, default: false
       t.string  :gauth_tmp
       t.datetime  :gauth_tmp_datetime
     end

test/models/google_authenticatable_test.rb

@@ -12,21 +12,21 @@ def setup
 	end
 
 	test 'new users should have gauth_enabled disabled by default' do
-		assert_equal 0, User.find(1).gauth_enabled.to_i
+		assert_equal 0, User.find(1).gauth_enabled
 	end
 
 	test 'get_qr method works' do
 		assert_not_nil User.find(1).get_qr
 	end
 
 	test 'updating gauth_enabled to true' do
-		User.find(1).set_gauth_enabled(:gauth_enabled => 1)
-		assert_equal 1, User.find(1).gauth_enabled.to_i
+		User.find(1).set_gauth_enabled(gauth_enable: true)
+		assert_equal 1, User.find(1).gauth_enabled
 	end
 
 	test 'updating gauth_enabled back to false' do
-		User.find(1).set_gauth_enabled(:gauth_enabled => 0)
-		assert_equal 0, User.find(1).gauth_enabled.to_i
+		User.find(1).set_gauth_enabled(gauth_enabled: true)
+		assert_equal 0, User.find(1).gauth_enabled
 	end
 
 	test 'updating the gauth_tmp key' do
@@ -61,4 +61,4 @@ def setup
 		assert User.find(1).validate_token(ROTP::TOTP.new(User.find(1).get_qr).at(Time.now))
 	end
 
-end
+end