Download as
Browse Code »

Commit 70eccaa392e96bfa9cad1a600f9f770aca844f51

Authored by Moises Machado
Committed by Antonio Terceiro
1 parent 3b5490ff
Exists in master and in 29 other branches add_member_task_reject_details, admin_visible_profile, article-list-template, community_notifications, contact_admin_translation, event_fixes, fix_comments_pagination, fix_rails4_organization_ratings, fix_sign_up_form, follow_step_fix, forum_topic_creation, mirror_block_improvements, multi_env_on_remote_user, new_security, noosfero_spb_ci, organization_ratings_improvements, organization_ratings_link_to_profile_stable-spb-1.3, organization_ratings_translations_fix, profile_api_improvements, ratings_minor_fixes, remote_user_fix, send_email_to_admins, stable-spb-1.3, stable-spb-1.3-fixes, stable-spb-1.4, stable-spb-1.5, suggest_rejected_value, web_steps_improvements, xss_terminate_custom_options

ActionItem930: changed notify_comments default to true

Showing 3 changed files with 14 additions and 4 deletions   Show diff stats
db/migrate/062_article_notify_comments_change_default.rb 0 → 100644
  Diff comments   View file @70eccaa
... ... @@ -0,0 +1,9 @@
  1 +class ArticleNotifyCommentsChangeDefault < ActiveRecord::Migration
  2 + def self.up
  3 + change_column_default :articles, :notify_comments, true
  4 + end
  5 +
  6 + def self.down
  7 + change_column_default :articles, :notify_comments, false
  8 + end
  9 +end
... ...
db/schema.rb
  Diff comments   View file @70eccaa
... ... @@ -9,7 +9,7 @@
9 9 #
10 10 # It's strongly recommended to check this file into your version control system.
11 11  
12   -ActiveRecord::Schema.define(:version => 61) do
  12 +ActiveRecord::Schema.define(:version => 62) do
13 13  
14 14 create_table "article_versions", :force => true do |t|
15 15 t.integer "article_id"
... ... @@ -73,7 +73,7 @@ ActiveRecord::Schema.define(:version =&gt; 61) do
73 73 t.boolean "accept_comments", :default => true
74 74 t.integer "reference_article_id"
75 75 t.text "setting"
76   - t.boolean "notify_comments", :default => false
  76 + t.boolean "notify_comments", :default => true
77 77 t.integer "hits", :default => 0
78 78 end
79 79  
... ...
test/unit/article_test.rb
  Diff comments   View file @70eccaa
... ... @@ -584,9 +584,9 @@ class ArticleTest &lt; Test::Unit::TestCase
584 584 assert !a.belongs_to_blog?
585 585 end
586 586  
587   - should 'has comments notifier false by default' do
  587 + should 'has comments notifier true by default' do
588 588 a = Article.new
589   - assert !a.notify_comments?
  589 + assert a.notify_comments?
590 590 end
591 591  
592 592 should 'hold hits count' do
... ... @@ -615,4 +615,5 @@ class ArticleTest &lt; Test::Unit::TestCase
615 615 assert_respond_to a, :can_display_hits?
616 616 assert_equal true, a.can_display_hits?
617 617 end
  618 +
618 619 end
... ...