Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | No parentheses for assert_equal | Pratik Naik | 2009-12-26 | 1 | -6/+6 |
| | |||||
* | Add Model.select/group/order/limit/joins/conditions/preload/eager_load class ↵ | Pratik Naik | 2009-12-26 | 4 | -59/+88 |
| | | | | | | | | | methods returning a lazy relation. Examples : posts = Post.select('id).order('name') # Returns a lazy relation posts.each {|p| puts p.id } # Fires "select id from posts order by name" | ||||
* | Merge remote branch 'jose/am' | Yehuda Katz | 2009-12-25 | 24 | -1422/+113 |
|\ | |||||
| * | Fix whiny_nil tests, improve error messages and make CI happy. | José Valim | 2009-12-25 | 1 | -6/+7 |
| | | |||||
| * | Refactor multiple parts logic and move Utils to PartContainer. | José Valim | 2009-12-25 | 4 | -31/+24 |
| | | |||||
| * | adv_attr_accessors in ActionMailer are not sent to the views, use the mailer ↵ | José Valim | 2009-12-25 | 6 | -42/+79 |
| | | | | | | | | object if you need to access the subject, recipients, from, etc. | ||||
| * | Move i18n to Gemfile. | José Valim | 2009-12-25 | 15 | -1345/+5 |
| | | |||||
| * | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-12-25 | 81 | -915/+960 |
| |\ | |||||
| * \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-12-22 | 35 | -294/+508 |
| |\ \ | |||||
| * \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-12-21 | 114 | -857/+1820 |
| |\ \ \ | |||||
| * \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-12-11 | 99 | -1592/+1833 |
| |\ \ \ \ | |||||
| * \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-12-02 | 77 | -295/+439 |
| |\ \ \ \ \ | |||||
| * \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-12-01 | 17 | -476/+545 |
| |\ \ \ \ \ \ | |||||
| * \ \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-11-25 | 20 | -142/+154 |
| |\ \ \ \ \ \ \ | |||||
| * \ \ \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-11-24 | 14 | -255/+177 |
| |\ \ \ \ \ \ \ \ | |||||
| * \ \ \ \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-11-23 | 2 | -3/+3 |
| |\ \ \ \ \ \ \ \ \ | |||||
| * \ \ \ \ \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-11-23 | 88 | -1537/+907 |
| |\ \ \ \ \ \ \ \ \ \ | |||||
| * \ \ \ \ \ \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-11-11 | 216 | -1447/+1689 |
| |\ \ \ \ \ \ \ \ \ \ \ | |||||
| * \ \ \ \ \ \ \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-11-02 | 447 | -7603/+6360 |
| |\ \ \ \ \ \ \ \ \ \ \ \ | |||||
| * \ \ \ \ \ \ \ \ \ \ \ \ | Merge branch 'master' of git://github.com/rails/rails | José Valim | 2009-10-16 | 42 | -90/+137 |
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | |||||
* | | | | | | | | | | | | | | | Model.scoped now returns a relation if invoked without any arguments | Pratik Naik | 2009-12-26 | 2 | -11/+25 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Example : posts = Post.scoped posts.size # Fires "select count(*) from posts" and returns the count posts.each {|p| puts p.name } # Fires "select * from posts" and loads post objects | ||||
* | | | | | | | | | | | | | | | The controller key shouldnt be part of the mapping if its not used | David Heinemeier Hansson | 2009-12-25 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | |||||
* | | | | | | | | | | | | | | | Fix tests | David Heinemeier Hansson | 2009-12-25 | 1 | -2/+2 |
| |_|_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | |||||
* | | | | | | | | | | | | | | This code was needed to work around ↵ | Yehuda Katz | 2009-12-24 | 1 | -4/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | http://redmine.ruby-lang.org/issues/show/2494. Since that bug is now closed in Ruby, we can remove it. | ||||
* | | | | | | | | | | | | | | Simplify and improve the performance of output_safety | Yehuda Katz | 2009-12-24 | 1 | -22/+6 |
| | | | | | | | | | | | | | | |||||
* | | | | | | | | | | | | | | Instead of marking raw text in templates as safe, and then putting them ↵ | Yehuda Katz | 2009-12-24 | 2 | -1/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | through String#<< which checks if the String is safe, use safe_concat, which uses the original (internal) String#<< and leaves the safe flag as is. Results in a significant performance improvement. | ||||
* | | | | | | | | | | | | | | Dead code | Yehuda Katz | 2009-12-24 | 1 | -15/+0 |
| | | | | | | | | | | | | | | |||||
* | | | | | | | | | | | | | | Translated strings in the view are assumed html_safe (Closes #3401) | David Heinemeier Hansson | 2009-12-24 | 1 | -1/+1 |
| | | | | | | | | | | | | | | |||||
* | | | | | | | | | | | | | | Stray carrier return | David Heinemeier Hansson | 2009-12-24 | 1 | -1/+0 |
| | | | | | | | | | | | | | | |||||
* | | | | | | | | | | | | | | The new routes shortform now also works for :as | David Heinemeier Hansson | 2009-12-24 | 1 | -1/+1 |
| | | | | | | | | | | | | | | |||||
* | | | | | | | | | | | | | | Merge | David Heinemeier Hansson | 2009-12-24 | 33 | -390/+441 |
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | |||||
| * | | | | | | | | | | | | | | Fix new schema test dependency on Hash#to_xml | Jeremy Kemper | 2009-12-23 | 1 | -0/+1 |
| | | | | | | | | | | | | | | | |||||
| * | | | | | | | | | | | | | | Allow named_routes to be used with root, and with new DSL short-form. | Sam Ruby | 2009-12-23 | 2 | -8/+16 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The real use case it to make all of the following act the same: root 'store#index', :as => 'store' match '/' => 'store#index', :as => 'store' match '/', :to => 'store#index', :as => 'store' The test case provided deviates from this in order to demonstrate all three forms in a single set of test routes. Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net> | ||||
| * | | | | | | | | | | | | | | Merge remote branch 'origin/master' | Carlhuda | 2009-12-23 | 10 | -79/+51 |
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: railties/lib/rails/application.rb | ||||
| | * | | | | | | | | | | | | | | Fix reference to Application#configuration to use #config | Jeremy Kemper | 2009-12-23 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | |||||
| | * | | | | | | | | | | | | | | Isolation tests intentionally avoid loading any state (because they're often ↵ | Carlhuda | 2009-12-23 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | testing things that have their own load path semantics that should not be polluted), so rack/test is not yet on the load path. Moving require "rack/test" into the setup means and after boot_rails means that it'll be required after the laod path has been altered to add in the Rails vendor/gems | ||||
| | * | | | | | | | | | | | | | | Routing: fix that route shorthand shouldn't ignore other options. Raise if ↵ | Jeremy Kemper | 2009-12-23 | 2 | -4/+8 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | :as option is given to root method since its name is always 'root' | ||||
| | * | | | | | | | | | | | | | | Fix Rack::Lock middleware condition: use *unless* we allow concurrency | Jeremy Kemper | 2009-12-23 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | |||||
| | * | | | | | | | | | | | | | | Shift more responsibility from application class to its singleton instance. ↵ | Jeremy Kemper | 2009-12-23 | 7 | -87/+56 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Treat instantiation and boot as separate steps. Use app.config rather than app.configuration. | ||||
| * | | | | | | | | | | | | | | | Rackup config.ru from the correct directory | Carlhuda | 2009-12-23 | 1 | -2/+4 |
| | | | | | | | | | | | | | | | | |||||
| * | | | | | | | | | | | | | | | Require active_support/all unless specifically requested to be left out. | Carlhuda | 2009-12-23 | 2 | -0/+10 |
| | | | | | | | | | | | | | | | | |||||
| * | | | | | | | | | | | | | | | Remove the ActionView::Base autoload because it creates crazy circular ↵ | Carlhuda | 2009-12-23 | 1 | -1/+1 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | autoload insanity | ||||
| * | | | | | | | | | | | | | | | Finish moving config.frameworks-dependent code to the framework plugin | Carlhuda | 2009-12-23 | 7 | -126/+110 |
| | | | | | | | | | | | | | | | | |||||
| * | | | | | | | | | | | | | | | Make /rails/info/properties work again. Also, the mocked up tests were ↵ | Carlhuda | 2009-12-23 | 2 | -1/+6 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | passing so we added a test that actually tested this functionality. | ||||
| * | | | | | | | | | | | | | | | Isolation tests intentionally avoid loading any state (because they're often ↵ | Carlhuda | 2009-12-23 | 1 | -3/+3 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | testing things that have their own load path semantics that should not be polluted), so rack/test is not yet on the load path. Moving require "rack/test" into the setup means and after boot_rails means that it'll be required after the laod path has been altered to add in the Rails vendor/gems | ||||
| * | | | | | | | | | | | | | | | Moving out some framework specific initializers into the framework libraries. | Carlhuda | 2009-12-23 | 8 | -96/+141 |
| | | | | | | | | | | | | | | | | |||||
| * | | | | | | | | | | | | | | | Raise an exception if an initializer is defined without a block | Carlhuda | 2009-12-23 | 2 | -0/+11 |
| | | | | | | | | | | | | | | | | |||||
| * | | | | | | | | | | | | | | | tests pass with requiring the frameworks in rails.rb | Carlhuda | 2009-12-23 | 8 | -118/+123 |
| |/ / / / / / / / / / / / / / | |||||
| * | | | | | | | | | | | | | | Fix bare string Rack response bodies | Jeremy Kemper | 2009-12-23 | 1 | -2/+2 |
| | | | | | | | | | | | | | | | |||||
| * | | | | | | | | | | | | | | Fix ActiveSupport::JSON encoding of control characters [\x00-\x1f] | Dwayne Litzenberger | 2009-12-23 | 2 | -3/+14 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to RFC 4627, only the following Unicode code points are allowed unescaped in JSON: unescaped = %x20-21 / %x23-5B / %x5D-10FFFF However, ActiveSupport::JSON did not escape the range %x00-1f. This caused parse errors when trying to decode the resulting output. [#3345 state:committed] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net> |