aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/lib
Commit message (Collapse)AuthorAgeFilesLines
* oops, missed a uniqAaron Patterson2010-09-291-1/+1
|
* dry up action_methodsAaron Patterson2010-09-291-4/+4
|
* Merge remote branch 'miloops/warnings'José Valim2010-09-292-7/+3
|\
| * We can't assign @view_context_class here, define super() in test instead if ↵Emilio Tagua2010-09-291-1/+1
| | | | | | | | we want to avoid warnings.
| * _ is not a valid scheme name character, \w includes it and also is redundant ↵Emilio Tagua2010-09-291-1/+1
| | | | | | | | | | | | with \d. 'The scheme name consists of a letter followed by any combination of letters, digits, and the plus ("+"), period ("."), or hyphen ("-") characters; and is terminated by a colon (":").'
| * Remove redundant conditional.Emilio Tagua2010-09-291-5/+1
| |
| * Don't try to interpolate string if there's no interpolation point at all.Emilio Tagua2010-09-291-1/+1
| |
| * Initialize @view_context_class and cache view_context_class value.Emilio Tagua2010-09-291-1/+1
| |
* | no need to call e.classAaron Patterson2010-09-291-1/+1
| |
* | Fix a small typo (ht: masterkain)José Valim2010-09-291-1/+1
|/
* Ensure that named routes do not overwrite previously defined routes.José Valim2010-09-291-10/+13
|
* Fix an error on 1.8.7.José Valim2010-09-291-1/+1
|
* @_etag is not used anywhere.José Valim2010-09-291-2/+0
|
* Avoid (@_var ||= nil) pattern by using initialize methods and ensuring ↵José Valim2010-09-299-33/+41
| | | | everyone calls super as expected.
* Merge remote branch 'miloops/warnings'José Valim2010-09-2917-23/+39
|\
| * Define @_layout if it is not defined.Emilio Tagua2010-09-281-0/+1
| |
| * Initialize @path.Emilio Tagua2010-09-281-0/+1
| |
| * Remove more warnings on AP.Emilio Tagua2010-09-282-3/+4
| |
| * Remove more warnings: no need to define attr_accessor if already exists. ↵Emilio Tagua2010-09-281-3/+5
| | | | | | | | Initialize ivar.
| * Use redefine_method instead define_method, it may be already defined.Emilio Tagua2010-09-281-1/+1
| |
| * Remove method if exists, avoid calling Array#first so many times.Emilio Tagua2010-09-281-3/+5
| |
| * Remove more warnings on variables.Emilio Tagua2010-09-281-0/+1
| |
| * Initialize @compiled_at if it is not.Emilio Tagua2010-09-281-0/+1
| |
| * Remove warings from rescues: don't define more than once debug_hash method ↵Emilio Tagua2010-09-281-2/+2
| | | | | | | | and ask if @response if defined.
| * Remove remaining warnings on _trace by adding parenthesis to gsub arguments.Emilio Tagua2010-09-281-3/+3
| |
| * Use parenthesis to avoid ambiguous first argument warning.Emilio Tagua2010-09-281-1/+1
| |
| * Ask if the instance variable is defined before asking for it, avoid *many* ↵Emilio Tagua2010-09-281-1/+5
| | | | | | | | warnings.
| * Avoid more uninitialized variable warnings.Emilio Tagua2010-09-282-2/+3
| |
| * Initialize @sub_templatesEmilio Tagua2010-09-281-0/+1
| |
| * @selected may not be defined here, avoid warning.Emilio Tagua2010-09-281-1/+1
| |
| * Don't define _test_case method if already defined.Emilio Tagua2010-09-281-2/+4
| |
| * 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-275-28/+5
| |
* | convert inject to map + joinAaron Patterson2010-09-281-5/+5
| |
* | Move uri parser to AS as URI.parser method to reuse it in AP and ARes.Emilio Tagua2010-09-285-28/+5
| |
* | Fix tests on 1.9.2.José Valim2010-09-282-5/+1
| |
* | renderer calls object.to_json when rendering :json => object [#5655 ↵Diego Carrion2010-09-271-1/+1
| | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Properly initialize variables inside the initialize method.José Valim2010-09-271-3/+8
|/
* 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-273-8/+8
| | | | 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>
* Do not cache the script name outcome.José Valim2010-09-271-9/+11
|
* Merge remote branch 'miloops/warnings'José Valim2010-09-2718-51/+87
|\ | | | | | | | | 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
| |
| * 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
| |