Support empty PG database too

First check if there is a result on the migration, else there is nothing to do

Support empty PG database too
083c0e28 · Z.J. van de Weg · 52cc64a8 · 083c0e28
Commit 083c0e28 authored Sep 06, 2016 by Z.J. van de Weg
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 2 deletions

20160902122721_drop_gitorious_field_from_application_settings.rb ...2122721_drop_gitorious_field_from_application_settings.rb +1 -2

No files found.
--- a/db/migrate/20160902122721_drop_gitorious_field_from_application_settings.rb
+++ b/db/migrate/20160902122721_drop_gitorious_field_from_application_settings.rb
@@ -8,12 +8,11 @@ class DropGitoriousFieldFromApplicationSettings < ActiveRecord::Migration
    require 'yaml'

    import_sources = connection.execute('SELECT import_sources FROM application_settings;')
+    return unless import_sources.first # support empty databases

    yaml = if Gitlab::Database.postgresql?
             import_sources.values[0][0]
           else
-             return unless import_sources.first
-
             import_sources.first[0]
           end