rails_best_practices 0.3.22 → 0.3.23
Sign up to get free protection for your applications and to get access to all the features.
data/VERSION
CHANGED
@@ -1 +1 @@
|
|
1
|
-
0.3.
|
1
|
+
0.3.23
|
@@ -22,7 +22,7 @@ module RailsBestPractices
|
|
22
22
|
private
|
23
23
|
|
24
24
|
def others_finder?(node)
|
25
|
-
node.message
|
25
|
+
[:find, :all, :first, :last].include? node.message and node.subject.node_type == :call and node.arguments.size > 1
|
26
26
|
end
|
27
27
|
end
|
28
28
|
end
|
@@ -30,8 +30,8 @@ files = RailsBestPractices::analyze_files(ARGV, options)
|
|
30
30
|
files.each { |file| runner.check_file(file) }
|
31
31
|
|
32
32
|
runner.errors.each {|error| puts error}
|
33
|
+
puts "\nPlease go to http://rails-bestpractices.com to see more useful Rails Best Practices."
|
33
34
|
if runner.errors.size > 0
|
34
|
-
puts "\ngo to http://wiki.github.com/flyerhzm/rails_best_practices to see how to solve these errors."
|
35
35
|
puts "\nFound #{runner.errors.size} errors."
|
36
36
|
end
|
37
37
|
|
@@ -5,11 +5,11 @@
|
|
5
5
|
|
6
6
|
Gem::Specification.new do |s|
|
7
7
|
s.name = %q{rails_best_practices}
|
8
|
-
s.version = "0.3.
|
8
|
+
s.version = "0.3.23"
|
9
9
|
|
10
10
|
s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
|
11
11
|
s.authors = ["Richard Huang"]
|
12
|
-
s.date = %q{2010-07-
|
12
|
+
s.date = %q{2010-07-25}
|
13
13
|
s.default_executable = %q{rails_best_practices}
|
14
14
|
s.description = %q{check rails files according to ihower's presentation 'rails best practices'}
|
15
15
|
s.email = %q{flyerhzm@gmail.com}
|
@@ -91,28 +91,28 @@ Gem::Specification.new do |s|
|
|
91
91
|
s.rubygems_version = %q{1.3.6}
|
92
92
|
s.summary = %q{check rails files according to ihower's presentation 'rails best practices'}
|
93
93
|
s.test_files = [
|
94
|
-
"spec/rails_best_practices/checks/
|
95
|
-
"spec/rails_best_practices/checks/always_add_db_index_check_spec.rb",
|
96
|
-
"spec/rails_best_practices/checks/isolate_seed_data_check_spec.rb",
|
97
|
-
"spec/rails_best_practices/checks/keep_finders_on_their_own_model_check_spec.rb",
|
98
|
-
"spec/rails_best_practices/checks/law_of_demeter_check_spec.rb",
|
99
|
-
"spec/rails_best_practices/checks/many_to_many_collection_check_spec.rb",
|
100
|
-
"spec/rails_best_practices/checks/move_code_into_controller_check_spec.rb",
|
101
|
-
"spec/rails_best_practices/checks/move_code_into_helper_check_spec.rb",
|
102
|
-
"spec/rails_best_practices/checks/move_code_into_model_check_spec.rb",
|
103
|
-
"spec/rails_best_practices/checks/move_finder_to_named_scope_check_spec.rb",
|
94
|
+
"spec/rails_best_practices/checks/isolate_seed_data_check_spec.rb",
|
104
95
|
"spec/rails_best_practices/checks/move_model_logic_into_model_check_spec.rb",
|
105
|
-
"spec/rails_best_practices/checks/
|
106
|
-
"spec/rails_best_practices/checks/
|
107
|
-
"spec/rails_best_practices/checks/
|
96
|
+
"spec/rails_best_practices/checks/move_code_into_helper_check_spec.rb",
|
97
|
+
"spec/rails_best_practices/checks/move_code_into_controller_check_spec.rb",
|
98
|
+
"spec/rails_best_practices/checks/use_observer_check_spec.rb",
|
108
99
|
"spec/rails_best_practices/checks/overuse_route_customizations_check_spec.rb",
|
100
|
+
"spec/rails_best_practices/checks/nested_model_forms_check_spec.rb",
|
101
|
+
"spec/rails_best_practices/checks/always_add_db_index_check_spec.rb",
|
102
|
+
"spec/rails_best_practices/checks/many_to_many_collection_check_spec.rb",
|
103
|
+
"spec/rails_best_practices/checks/use_before_filter_check_spec.rb",
|
104
|
+
"spec/rails_best_practices/checks/law_of_demeter_check_spec.rb",
|
105
|
+
"spec/rails_best_practices/checks/use_scope_access_check_spec.rb",
|
109
106
|
"spec/rails_best_practices/checks/replace_complex_creation_with_factory_method_check_spec.rb",
|
107
|
+
"spec/rails_best_practices/checks/not_use_default_route_check_spec.rb",
|
108
|
+
"spec/rails_best_practices/checks/move_finder_to_named_scope_check_spec.rb",
|
109
|
+
"spec/rails_best_practices/checks/use_model_callback_check_spec.rb",
|
110
|
+
"spec/rails_best_practices/checks/add_model_virtual_attribute_check_spec.rb",
|
110
111
|
"spec/rails_best_practices/checks/replace_instance_variable_with_local_variable_check_spec.rb",
|
111
|
-
"spec/rails_best_practices/checks/use_before_filter_check_spec.rb",
|
112
112
|
"spec/rails_best_practices/checks/use_model_association_check_spec.rb",
|
113
|
-
"spec/rails_best_practices/checks/
|
114
|
-
"spec/rails_best_practices/checks/
|
115
|
-
"spec/rails_best_practices/checks/
|
113
|
+
"spec/rails_best_practices/checks/move_code_into_model_check_spec.rb",
|
114
|
+
"spec/rails_best_practices/checks/needless_deep_nesting_check_spec.rb",
|
115
|
+
"spec/rails_best_practices/checks/keep_finders_on_their_own_model_check_spec.rb",
|
116
116
|
"spec/spec_helper.rb"
|
117
117
|
]
|
118
118
|
|
@@ -22,6 +22,23 @@ describe RailsBestPractices::Checks::KeepFindersOnTheirOwnModelCheck do
|
|
22
22
|
errors[0].to_s.should == "app/models/post.rb:5 - keep finders on their own model"
|
23
23
|
end
|
24
24
|
|
25
|
+
it "should keep finders on thier own model with all method" do
|
26
|
+
content = <<-EOF
|
27
|
+
class Post < ActiveRecord::Base
|
28
|
+
has_many :comments
|
29
|
+
|
30
|
+
def find_valid_comments
|
31
|
+
self.comment.all(:conditions => { :is_spam => false },
|
32
|
+
:limit => 10)
|
33
|
+
end
|
34
|
+
end
|
35
|
+
EOF
|
36
|
+
@runner.check('app/models/post.rb', content)
|
37
|
+
errors = @runner.errors
|
38
|
+
errors.should_not be_empty
|
39
|
+
errors[0].to_s.should == "app/models/post.rb:5 - keep finders on their own model"
|
40
|
+
end
|
41
|
+
|
25
42
|
it "should not keep finders on thier own model with self finder" do
|
26
43
|
content = <<-EOF
|
27
44
|
class Post < ActiveRecord::Base
|
metadata
CHANGED
@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
|
|
5
5
|
segments:
|
6
6
|
- 0
|
7
7
|
- 3
|
8
|
-
-
|
9
|
-
version: 0.3.
|
8
|
+
- 23
|
9
|
+
version: 0.3.23
|
10
10
|
platform: ruby
|
11
11
|
authors:
|
12
12
|
- Richard Huang
|
@@ -14,7 +14,7 @@ autorequire:
|
|
14
14
|
bindir: bin
|
15
15
|
cert_chain: []
|
16
16
|
|
17
|
-
date: 2010-07-
|
17
|
+
date: 2010-07-25 00:00:00 +08:00
|
18
18
|
default_executable: rails_best_practices
|
19
19
|
dependencies:
|
20
20
|
- !ruby/object:Gem::Dependency
|
@@ -151,26 +151,26 @@ signing_key:
|
|
151
151
|
specification_version: 3
|
152
152
|
summary: check rails files according to ihower's presentation 'rails best practices'
|
153
153
|
test_files:
|
154
|
-
- spec/rails_best_practices/checks/add_model_virtual_attribute_check_spec.rb
|
155
|
-
- spec/rails_best_practices/checks/always_add_db_index_check_spec.rb
|
156
154
|
- spec/rails_best_practices/checks/isolate_seed_data_check_spec.rb
|
157
|
-
- spec/rails_best_practices/checks/keep_finders_on_their_own_model_check_spec.rb
|
158
|
-
- spec/rails_best_practices/checks/law_of_demeter_check_spec.rb
|
159
|
-
- spec/rails_best_practices/checks/many_to_many_collection_check_spec.rb
|
160
|
-
- spec/rails_best_practices/checks/move_code_into_controller_check_spec.rb
|
161
|
-
- spec/rails_best_practices/checks/move_code_into_helper_check_spec.rb
|
162
|
-
- spec/rails_best_practices/checks/move_code_into_model_check_spec.rb
|
163
|
-
- spec/rails_best_practices/checks/move_finder_to_named_scope_check_spec.rb
|
164
155
|
- spec/rails_best_practices/checks/move_model_logic_into_model_check_spec.rb
|
165
|
-
- spec/rails_best_practices/checks/
|
166
|
-
- spec/rails_best_practices/checks/
|
167
|
-
- spec/rails_best_practices/checks/
|
156
|
+
- spec/rails_best_practices/checks/move_code_into_helper_check_spec.rb
|
157
|
+
- spec/rails_best_practices/checks/move_code_into_controller_check_spec.rb
|
158
|
+
- spec/rails_best_practices/checks/use_observer_check_spec.rb
|
168
159
|
- spec/rails_best_practices/checks/overuse_route_customizations_check_spec.rb
|
160
|
+
- spec/rails_best_practices/checks/nested_model_forms_check_spec.rb
|
161
|
+
- spec/rails_best_practices/checks/always_add_db_index_check_spec.rb
|
162
|
+
- spec/rails_best_practices/checks/many_to_many_collection_check_spec.rb
|
163
|
+
- spec/rails_best_practices/checks/use_before_filter_check_spec.rb
|
164
|
+
- spec/rails_best_practices/checks/law_of_demeter_check_spec.rb
|
165
|
+
- spec/rails_best_practices/checks/use_scope_access_check_spec.rb
|
169
166
|
- spec/rails_best_practices/checks/replace_complex_creation_with_factory_method_check_spec.rb
|
167
|
+
- spec/rails_best_practices/checks/not_use_default_route_check_spec.rb
|
168
|
+
- spec/rails_best_practices/checks/move_finder_to_named_scope_check_spec.rb
|
169
|
+
- spec/rails_best_practices/checks/use_model_callback_check_spec.rb
|
170
|
+
- spec/rails_best_practices/checks/add_model_virtual_attribute_check_spec.rb
|
170
171
|
- spec/rails_best_practices/checks/replace_instance_variable_with_local_variable_check_spec.rb
|
171
|
-
- spec/rails_best_practices/checks/use_before_filter_check_spec.rb
|
172
172
|
- spec/rails_best_practices/checks/use_model_association_check_spec.rb
|
173
|
-
- spec/rails_best_practices/checks/
|
174
|
-
- spec/rails_best_practices/checks/
|
175
|
-
- spec/rails_best_practices/checks/
|
173
|
+
- spec/rails_best_practices/checks/move_code_into_model_check_spec.rb
|
174
|
+
- spec/rails_best_practices/checks/needless_deep_nesting_check_spec.rb
|
175
|
+
- spec/rails_best_practices/checks/keep_finders_on_their_own_model_check_spec.rb
|
176
176
|
- spec/spec_helper.rb
|