aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* No need to call super here. Use yield instead block.callEmilio Tagua2010-09-281-3/+2
|
* Initialize @app if it doesn't exists.Emilio Tagua2010-09-281-1/+1
|
* Move uri parser to AS as URI.parser method to reuse it in AP and ARes.Emilio Tagua2010-09-278-43/+25
|
* Define @_routes inside method, makes more sense and will be initialized when ↵Emilio Tagua2010-09-272-2/+1
| | | | | | called anywhere. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Remove more warnings shadowing outer local variable.Emilio Tagua2010-09-275-17/+17
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Remove warning "URI.unescape is obsolete" from actionpack.Emilio Tagua2010-09-276-13/+16
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Merge remote branch 'gramos/ares-missing-prefix-value'José Valim2010-09-275-1/+41
|\
| * - added mock to test ActiveResource::MissingPrefixParam in finder_testGaston Ramos2010-09-271-0/+1
| |
| * - update exceptions documentationGaston Ramos2010-09-271-0/+11
| |
| * - check prefix options in collection_pathGaston Ramos2010-09-272-1/+8
| |
| * - better name for prefix param test caseGaston Ramos2010-09-271-1/+1
| |
| * - refactoring, move prefix_options check to a custom methodGaston Ramos2010-09-271-5/+9
| |
| * - elmenth_path raise an ActiveResource::MissingPrefixParam exception when ↵Gaston Ramos2010-09-274-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é Valim2010-09-271-9/+11
| |
* | Merge remote branch 'miloops/warnings'José Valim2010-09-2743-150/+180
|\ \ | | | | | | | | | | | | Conflicts: actionpack/lib/action_controller/metal/url_for.rb
| * | _routes must be inside @controller conditional.Emilio Tagua2010-09-271-1/+1
| | |
| * | Initialize @_routes if not defined yet, avoiding more warnings.Emilio Tagua2010-09-271-0/+1
| | |
| * | Rename duplicated test name.Emilio Tagua2010-09-271-1/+1
| | |
| * | Refactor method to avoid warnings and not run unnecessary code.Emilio Tagua2010-09-271-11/+11
| | |
| * | Avoid uninitialized variable warning.Emilio Tagua2010-09-271-1/+1
| | |
| * | Initialize @_request and @_response.Emilio Tagua2010-09-271-1/+3
| | |
| * | Remove method previous method if already defined.Emilio Tagua2010-09-271-0/+1
| | |
| * | Define @emitted_hidden_id if it doesn't exists and reuse it if it does.Emilio Tagua2010-09-271-1/+1
| | |
| * | Avoid uninitialized variable warning.Emilio Tagua2010-09-271-0/+1
| | |
| * | Initialize @cookies.Emilio Tagua2010-09-271-0/+1
| | |
| * | Remove more warnings shadowing outer local variable.Emilio Tagua2010-09-275-17/+17
| | |
| * | Don't shadow outer local variables.Emilio Tagua2010-09-272-5/+5
| | |
| * | Remove method if already exists to avoid warnings.Emilio Tagua2010-09-271-0/+1
| | |
| * | Remove duplicated test.Emilio Tagua2010-09-271-8/+0
| | |
| * | Avoid uninitialized variable warning, reuse @integration_session.Emilio Tagua2010-09-271-8/+14
| | |
| * | Ask is @controller is defined to avoid warning.Emilio Tagua2010-09-271-1/+1
| | |
| * | Initialize @_routes if it doesn't exists.Emilio Tagua2010-09-271-0/+1
| | |
| * | Remove warning "too many arguments for format string" when interpolating ↵Emilio Tagua2010-09-271-1/+1
| | | | | | | | | | | | with empty hash.
| * | Initialize @_etag.Emilio Tagua2010-09-271-0/+1
| | |
| * | Initialize @as before plural method is called.Emilio Tagua2010-09-271-0/+1
| | |
| * | Remove old method before redefining it.Emilio Tagua2010-09-271-0/+2
| | |
| * | Remove warning "URI.unescape is obsolete" from actionpack.Emilio Tagua2010-09-275-14/+34
| | |
| * | Fix indentation.Emilio Tagua2010-09-271-19/+18
| | |
| * | Use parentheses when using assert_match followed by a regexp to avoid warnings.Emilio Tagua2010-09-2717-58/+58
| | |
| * | Don't redefine existing attributes accessors.Emilio Tagua2010-09-271-1/+3
| | |
| * | Prevent shadowing outer local variable.Emilio Tagua2010-09-271-3/+3
| | |
* | | Call it compile_methods! and do the same on AM.José Valim2010-09-274-6/+16
| | |
* | | adding a test for slug behaviorAaron Patterson2010-09-271-0/+4
| | |
* | | Merge remote branch 'thedarkone/performance'José Valim2010-09-2716-76/+179
|\ \ \
| * | | Fix Namind#model_name.thedarkone2010-09-271-2/+4
| | | |
| * | | Assume compute_asset_host returns reasonable values.thedarkone2010-09-271-1/+1
| | | |
| * | | Convert unless/else into if/else.thedarkone2010-09-271-4/+4
| | | |
| * | | Performance: refactor convert_options_to_data_attributes.thedarkone2010-09-271-10/+14
| | | |
| * | | Remove dead code.thedarkone2010-09-271-2/+2
| | | |
| * | | Clean up url_for.thedarkone2010-09-271-3/+1
| | | |