Skip to content

Commit 1f09f46

Browse files
Merge pull request #140 from activeadmin/feature/optimization
Feature/optimization
2 parents 809e1c7 + 9ebe001 commit 1f09f46

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

lib/active_admin/mongoid/document.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ def column_for_attribute(name)
5353
module ClassMethods
5454
def content_columns
5555
# cannot cache this, since changes in time (while defining fields)
56-
fields.map(&:second).reject do |f|
56+
fields.map(&:second).reject! do |f|
5757
f.name =~ /(^_|^(created|updated)_at)/ or Mongoid::Fields::ForeignKey === f
5858
end
5959
end

lib/active_admin/mongoid/filter_form_builder.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ def default_input_type(method, options = {})
2525
end
2626

2727
def is_association?(method)
28-
@object.klass.associations.to_a.map(&:first).include?(method.to_s)
28+
@object.klass.associations.to_a.map!(&:first).include?(method.to_s)
2929
end
3030

3131
def column_for(method)

lib/meta_search/searches/mongoid.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ def method_missing name, *args, &block
9292
end
9393

9494
def metasearch_regexp
95-
field_names = klass.fields.map(&:second).map(&:name)
95+
field_names = klass.fields.map{ |field| field.second.name }
9696
conditions = MetaSearch::DEFAULT_WHERES.map {|condition| condition[0...-1]} # pop tail options
9797

9898
/\A(#{field_names.join('|')})_(#{conditions.join('|')})\z/

0 commit comments

Comments
 (0)