Download as
Browse Code »

Commit 0cc411fca037d2f52de9660f20b6274054c6eefc

Authored by Antonio Terceiro
1 parent 4c9ca01b
Exists in master and in 22 other branches angular_poc, api-article-archived, article-readonly, backup-7-jan-2016, captcha_serpro_plugin, content-manager-hostspot, export-comment-paragraph, fix_event_date_issue, login-captcha, master_rails3, new_video_plugin, pg_search_rank, production, refactor_with_role, refactor_with_role_scope, remove_profile_cat_icons, resend_confirmation_email, staging, staging_rails3, tasks_keep_filter_params, theme-brasil-digital-from-staging, travis

Avoiding crash when passing nil to YAML.load() 

(Again!)

(ActionItem894)
Showing 1 changed file with 1 additions and 1 deletions   Show diff stats
db/migrate/20101205034144_add_language_and_translation_of_id_to_article.rb
  Diff comments   View file @0cc411f
@@ -9,7 +9,7 @@ class AddLanguageAndTranslationOfIdToArticle < ActiveRecord::Migration @@ -9,7 +9,7 @@ class AddLanguageAndTranslationOfIdToArticle < ActiveRecord::Migration
9 add_index :articles, :translation_of_id 9 add_index :articles, :translation_of_id
10 10
11 select_all("select id, setting from articles where type = 'Blog'").each do |blog| 11 select_all("select id, setting from articles where type = 'Blog'").each do |blog|
12 - settings = YAML.load(blog['setting']) 12 + settings = YAML.load(blog['setting'] || {}.to_yaml)
13 settings[:display_posts_in_current_language] = true 13 settings[:display_posts_in_current_language] = true
14 assignments = ActiveRecord::Base.sanitize_sql_for_assignment(:setting => settings.to_yaml) 14 assignments = ActiveRecord::Base.sanitize_sql_for_assignment(:setting => settings.to_yaml)
15 update("update articles set %s where id = %d" % [assignments, blog['id']]) 15 update("update articles set %s where id = %d" % [assignments, blog['id']])