| Commit message (Collapse) | Author | Age | Files | Lines | |
|---|---|---|---|---|---|
| * | @klass also uses DynamicFinderMatch, so no need for it on the relation | Aaron Patterson | 2010-09-27 | 2 | -9/+5 |
| | | |||||
| * | no need for a case / when statement | Aaron Patterson | 2010-09-27 | 1 | -11/+7 |
| | | |||||
| * | do not need intermediate variable, avoid lasgn | Aaron Patterson | 2010-09-27 | 1 | -2/+3 |
| | | |||||
| * | DRY up our regular expression | Aaron Patterson | 2010-09-27 | 1 | -3/+3 |
| | | |||||
| * | reduce method calls in the dynamic finder matcher | Aaron Patterson | 2010-09-27 | 1 | -4/+4 |
| | | |||||
| * | Fix tests on 1.9.2. | José Valim | 2010-09-28 | 2 | -5/+1 |
| | | |||||
| * | revises RDoc of Hash#to_param to be more precise about ordering | Xavier Noria | 2010-09-28 | 1 | -2/+2 |
| | | |||||
| * | let Hash#to_param and Hash#to_query sort again | Xavier Noria | 2010-09-28 | 2 | -5/+10 |
| | | | | | | This was a regression introduced in 5c858220085dc4ddc1bec496747059dfbe32f1da. We bring sorting back because people rely on it, eg for constructing consistent cache keys. | ||||
| * | make sure we use the engine assigned to the table when quoting | Aaron Patterson | 2010-09-27 | 2 | -1/+19 |
| | | |||||
| * | Test for after_create callback order in ActiveSupport [#5703 state:resolved] | Neeraj Singh | 2010-09-27 | 1 | -0/+23 |
| | | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | after_create in ActiveModel should in the order specified | Neeraj Singh | 2010-09-27 | 2 | -1/+31 |
| | | | | | | | [#5650 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | renderer calls object.to_json when rendering :json => object [#5655 ↵ | Diego Carrion | 2010-09-27 | 2 | -1/+14 |
| | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Fix for #5579 involved the code change for both has_one and has_many ↵ | Neeraj Singh | 2010-09-27 | 1 | -1/+9 |
| | | | | | | | | | relationships. The path included test only for has_one. This patch adds test for has_many relationship. [#5706 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Fix for nested_attributes with has_many association fails when a single ↵ | Neeraj Singh | 2010-09-27 | 2 | -1/+14 |
| | | | | | | | | | record is being updated. [#5705 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Fix broken module namespacing in ActiveResource with Ruby 1.9 [#5699 ↵ | Matthijs Langenberg | 2010-09-27 | 4 | -5/+22 |
| | | | | | | | | | | | | | | | | | | | | | | | | state:resolved] Following namespace use case was broken with Ruby 1.9: class Author < ActiveRecord::Base ... end module Api class Book < ActiveResouce::Base end end Let's say XML contains <book><author><name>John</name></author>.... Api::Book.first.author.class.to_s #=> Ruby 1.8.7: "Api::Book::Author" (namespaced, correct), Ruby 1.9: "Author" (toplevel, broken) Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
| * | Properly initialize variables inside the initialize method. | José Valim | 2010-09-27 | 1 | -3/+8 |
| | | |||||
| * | Define @_routes inside method, makes more sense and will be initialized when ↵ | Emilio Tagua | 2010-09-27 | 2 | -2/+1 |
| | | | | | | | called anywhere. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
| * | Remove more warnings shadowing outer local variable. | Emilio Tagua | 2010-09-27 | 5 | -17/+17 |
| | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
| * | Remove warning "URI.unescape is obsolete" from actionpack. | Emilio Tagua | 2010-09-27 | 6 | -13/+16 |
| | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
| * | Merge remote branch 'gramos/ares-missing-prefix-value' | José Valim | 2010-09-27 | 5 | -1/+41 |
| |\ | |||||
| | * | - added mock to test ActiveResource::MissingPrefixParam in finder_test | Gaston Ramos | 2010-09-27 | 1 | -0/+1 |
| | | | |||||
| | * | - update exceptions documentation | Gaston Ramos | 2010-09-27 | 1 | -0/+11 |
| | | | |||||
| | * | - check prefix options in collection_path | Gaston Ramos | 2010-09-27 | 2 | -1/+8 |
| | | | |||||
| | * | - better name for prefix param test case | Gaston Ramos | 2010-09-27 | 1 | -1/+1 |
| | | | |||||
| | * | - refactoring, move prefix_options check to a custom method | Gaston Ramos | 2010-09-27 | 1 | -5/+9 |
| | | | |||||
| | * | - elmenth_path raise an ActiveResource::MissingPrefixParam exception when ↵ | Gaston Ramos | 2010-09-27 | 4 | -1/+18 |
| | | | | | | | | | | | | | | | prefix_options does not has all required prefix_options ex: class StreetAddress < ActiveResource::Base self.site = "http://37s.sunrise.i:3000/people/:person_id/" end StreetAddress.element_path(1) # => ActiveResource::MissingPrefixParam | ||||
| * | | Do not cache the script name outcome. | José Valim | 2010-09-27 | 1 | -9/+11 |
| | | | |||||
| * | | Merge remote branch 'miloops/warnings' | José Valim | 2010-09-27 | 43 | -150/+180 |
| |\ \ | | | | | | | | | | | | | Conflicts: actionpack/lib/action_controller/metal/url_for.rb | ||||
| | * | | _routes must be inside @controller conditional. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | | | | |||||
| | * | | Initialize @_routes if not defined yet, avoiding more warnings. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Rename duplicated test name. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | | | | |||||
| | * | | Refactor method to avoid warnings and not run unnecessary code. | Emilio Tagua | 2010-09-27 | 1 | -11/+11 |
| | | | | |||||
| | * | | Avoid uninitialized variable warning. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | | | | |||||
| | * | | Initialize @_request and @_response. | Emilio Tagua | 2010-09-27 | 1 | -1/+3 |
| | | | | |||||
| | * | | Remove method previous method if already defined. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Define @emitted_hidden_id if it doesn't exists and reuse it if it does. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | | | | |||||
| | * | | Avoid uninitialized variable warning. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Initialize @cookies. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Remove more warnings shadowing outer local variable. | Emilio Tagua | 2010-09-27 | 5 | -17/+17 |
| | | | | |||||
| | * | | Don't shadow outer local variables. | Emilio Tagua | 2010-09-27 | 2 | -5/+5 |
| | | | | |||||
| | * | | Remove method if already exists to avoid warnings. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Remove duplicated test. | Emilio Tagua | 2010-09-27 | 1 | -8/+0 |
| | | | | |||||
| | * | | Avoid uninitialized variable warning, reuse @integration_session. | Emilio Tagua | 2010-09-27 | 1 | -8/+14 |
| | | | | |||||
| | * | | Ask is @controller is defined to avoid warning. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | | | | |||||
| | * | | Initialize @_routes if it doesn't exists. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Remove warning "too many arguments for format string" when interpolating ↵ | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | | | | | | | | | | | | | with empty hash. | ||||
| | * | | Initialize @_etag. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Initialize @as before plural method is called. | Emilio Tagua | 2010-09-27 | 1 | -0/+1 |
| | | | | |||||
| | * | | Remove old method before redefining it. | Emilio Tagua | 2010-09-27 | 1 | -0/+2 |
| | | | | |||||
| | * | | Remove warning "URI.unescape is obsolete" from actionpack. | Emilio Tagua | 2010-09-27 | 5 | -14/+34 |
| | | | | |||||
 |
index : rails.git | |
| Mirror of official rails repo with custom fixes. | Harald Eilertsen |
| aboutsummaryrefslogtreecommitdiffstats |