Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | 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 | |
| | | | ||||||
| * | | 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 | |
| | | | ||||||
* | | | Call it compile_methods! and do the same on AM. | José Valim | 2010-09-27 | 4 | -6/+16 | |
| | | | ||||||
* | | | adding a test for slug behavior | Aaron Patterson | 2010-09-27 | 1 | -0/+4 | |
| | | | ||||||
* | | | Merge remote branch 'thedarkone/performance' | José Valim | 2010-09-27 | 16 | -76/+179 | |
|\ \ \ | ||||||
| * | | | Fix Namind#model_name. | thedarkone | 2010-09-27 | 1 | -2/+4 | |
| | | | | ||||||
| * | | | Assume compute_asset_host returns reasonable values. | thedarkone | 2010-09-27 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Convert unless/else into if/else. | thedarkone | 2010-09-27 | 1 | -4/+4 | |
| | | | | ||||||
| * | | | Performance: refactor convert_options_to_data_attributes. | thedarkone | 2010-09-27 | 1 | -10/+14 | |
| | | | | ||||||
| * | | | Remove dead code. | thedarkone | 2010-09-27 | 1 | -2/+2 | |
| | | | | ||||||
| * | | | Clean up url_for. | thedarkone | 2010-09-27 | 1 | -3/+1 | |
| | | | | ||||||
| * | | | Hash#empty? is faster than Enumerable#any? when used on a Hash. | thedarkone | 2010-09-27 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Do a single string interpolation. | thedarkone | 2010-09-27 | 1 | -1/+1 | |
| | | | | ||||||
| * | | | Simple .empty? test will do fine here (rails_asset_id returns nice strings). | thedarkone | 2010-09-27 | 1 | -1/+1 | |
| | | | |