impress/app/models/wardrobe_tip.rb

class WardrobeTip < ActiveRecord::Base
  translates :body

  scope :by_index, -> { order('`index` ASC') }
end
basic tip functionality 2014-04-02 21:00:50 -07:00			`class WardrobeTip < ActiveRecord::Base`
			`translates :body`

Update to new `scope` syntax Ohh ok, without this change all of our `scope`s were just immediately evaluating the argument and fetching _all_ such matching records immediately, instead of waiting to actually be called. This led to bugs like `pet_type.as_json` returning ALL pet states in the whole db, because the `PetState.emotion_order` scope was being treated as a single predefined query, rather than a query fragment to merge into the current context. This also explains what happened in 724ed83: that's why things before the scope in the query were being ignored. 2023-07-22 14:04:01 -07:00			scope :by_index, -> { order('`index` ASC') }
basic tip functionality 2014-04-02 21:00:50 -07:00			`end`