Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | 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 |
| | |||||
* | Fix indentation. | Emilio Tagua | 2010-09-27 | 1 | -19/+18 |
| | |||||
* | Use parentheses when using assert_match followed by a regexp to avoid warnings. | Emilio Tagua | 2010-09-27 | 17 | -58/+58 |
| | |||||
* | Don't redefine existing attributes accessors. | Emilio Tagua | 2010-09-27 | 1 | -1/+3 |
| | |||||
* | Prevent shadowing outer local variable. | Emilio Tagua | 2010-09-27 | 1 | -3/+3 |
| | |||||
* | third parameter is not used | Aaron Patterson | 2010-09-26 | 1 | -1/+1 |
| | |||||
* | db:structure:dump should list current Rails.env adapter in errors, not ↵ | Nic Benders | 2010-09-26 | 1 | -1/+1 |
| | | | | | | | | always the test adapter [#5710 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Fix remove_index issue when provided :name is a symbol | Tim Connor | 2010-09-26 | 2 | -2/+2 |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Fix the precedence issue here | Santiago Pastorino | 2010-09-26 | 1 | -1/+1 |
| | |||||
* | Not need to do this double ternary | Santiago Pastorino | 2010-09-26 | 1 | -1/+1 |
| | |||||
* | port is appended twice to HTTP_HOST when host already has the port | Santiago Pastorino | 2010-09-26 | 1 | -3/+3 |
| | |||||
* | Bump up some deps | Santiago Pastorino | 2010-09-26 | 4 | -5/+5 |
| | |||||
* | Partial revert of #d650b71 'Remove deprecated stuff in ActionController' | Carlos Antonio da Silva | 2010-09-26 | 4 | -0/+11 |
| | | | | | This brings back the deprecated modules from ActionController, because they didn't have any deprecation warning. | ||||
* | We don't need to check if the format exists to generate the url, it already ↵ | Carlos Antonio da Silva | 2010-09-26 | 1 | -8/+7 |
| | | | | does it internally | ||||
* | More cleanup on form helpers | Carlos Antonio da Silva | 2010-09-26 | 1 | -11/+12 |
| | |||||
* | Remove last tests with deprecated form_for and cleanup form_for helper | Carlos Antonio da Silva | 2010-09-26 | 2 | -67/+51 |
| | | | | | | | This cleans up the last bits of deprecation stuff from form_for helper. However there is still a bug when using :as => foo[], with index. The classes and ids are being generated using [], such as foo[]_edit. This bug already existed but it was not detected before. | ||||
* | Select tags with array options are deprecated, removing | Carlos Antonio da Silva | 2010-09-26 | 2 | -10/+0 |
| | |||||
* | Refactor form_for helper | Carlos Antonio da Silva | 2010-09-26 | 1 | -21/+11 |
| | |||||
* | Review most of the form_for deprecated tests, missing tests with index like [] | Carlos Antonio da Silva | 2010-09-26 | 2 | -405/+322 |
| | |||||
* | Remove deprecated stuff in ActionController | Carlos Antonio da Silva | 2010-09-26 | 29 | -58/+47 |
| | | | | | | This removes all deprecated classes in ActionController related to Routing, Abstract Request/Response and Integration/IntegrationTest. All tests and docs were changed to ActionDispatch instead of ActionController. | ||||
* | There is no need to open AC::Base three times to setup tests. | Carlos Antonio da Silva | 2010-09-26 | 1 | -27/+20 |
| | |||||
* | There is no DeprecatedBlockHelpers in AV, so remove it | Carlos Antonio da Silva | 2010-09-26 | 1 | -1/+0 |
| | |||||
* | Fix active_resource assertions. | José Valim | 2010-09-25 | 1 | -2/+2 |
| | |||||
* | Fix app and actions generators tests | Piotr Sarnacki | 2010-09-25 | 2 | -0/+9 |
| | | | | | | | Normally Rails.application is an instance, but for those tests Rails.application needs to be class. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Refactor a few methods connected with namespacing in ↵ | Piotr Sarnacki | 2010-09-25 | 1 | -8/+9 |
| | | | | | | Rails::Generators::NamedBase Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Add namespacing to mailer generator | Piotr Sarnacki | 2010-09-25 | 3 | -0/+56 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Add namespacing for observer generator | Piotr Sarnacki | 2010-09-25 | 3 | -0/+26 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Add namespace for test_unit generators | Piotr Sarnacki | 2010-09-25 | 4 | -3/+9 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Allow per Resource format settings | Jacques Crocker | 2010-09-25 | 5 | -17/+25 |
| | | | | | | | | | | Previously, ActiveResource was using the connection level formatter for get requests. This made it impossible to use custom formatters per resource. Additionally this commit makes the Connection request methods more consistent. It always returns a Response. The base will then decode it each the response using its format setting. Merging this commit will allow users to add custom formatters on a per Resource basis. This enables handling pagination responses from the server side, a very common use case that was previously impossible without monkeypatching XmlFormat. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Allow ActiveResource to work with non-generated ids [#5660 state:resolved] | Jacques Crocker | 2010-09-25 | 5 | -5/+80 |
| | | | | | | This commit updates new? so that it knows whether or not the record was actually new or not, and doesn't rely solely on the presence of id. This enables the ability to set a custom primary_key that is not autogenerated by the server. Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Fix 'warning: method redefined' [#5551 state:resolved] | Krekoten' Marjan | 2010-09-25 | 1 | -4/+4 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Fix logging when cache key contains % sign [#5570 state:resolved] | Krekoten' Marjan | 2010-09-25 | 2 | -1/+17 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Remove duplicate helper method | Krekoten' Marjan | 2010-09-25 | 1 | -3/+0 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Update abort message (ht: tilsammans). | José Valim | 2010-09-25 | 1 | -1/+1 |
| | |||||
* | Don't act destructively on ActiveModel::Name#human options hash. [#5366 ↵ | John Firebaugh | 2010-09-25 | 3 | -2/+13 |
| | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Don't duplicate Rack::Response functionality. [#5320 state:resolved] | John Firebaugh | 2010-09-25 | 1 | -18/+4 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Add ActionDispatch::TestResponse tests. | John Firebaugh | 2010-09-25 | 1 | -0/+21 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Rails.application is set anyway, we don't need to set it manually, this was ↵ | Piotr Sarnacki | 2010-09-25 | 2 | -3/+1 |
| | | | | | | purpose of different behavior in tests and in application Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Generators fix: properly check if module should be created when creating a ↵ | Piotr Sarnacki | 2010-09-25 | 3 | -2/+11 |
| | | | | | | namespaced model Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | memoized protected methods should remain protected | Andrew Kaspick | 2010-09-24 | 2 | -0/+16 |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Delegate ActiveRecord::Base.offset to scoped methods (analogous to limit) ↵ | Marcelo Giorgi | 2010-09-24 | 2 | -1/+7 |
| | | | | | | [#5688 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | attr_accessor_with_default should raise an ArgumentError not a RuntimeError | Aaron Patterson | 2010-09-24 | 2 | -4/+3 |
| | |||||
* | Fix warning message when db/schema.rb doesn't exist [#5625 state:resolved] | Brian Candler | 2010-09-24 | 1 | -1/+1 |
| | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | got rid of the "ambiguous first argument; put parentheses or even spaces" ↵ | Jeff Kreeftmeijer | 2010-09-24 | 1 | -32/+32 |
| | | | | | | warnings in the scaffold_generator tests [#4872 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | Bump thor version requirement. | José Valim | 2010-09-24 | 2 | -3/+1 |
| |