Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | oops, missed a uniq | Aaron Patterson | 2010-09-29 | 1 | -1/+1 |
| | |||||
* | dry up action_methods | Aaron Patterson | 2010-09-29 | 1 | -4/+4 |
| | |||||
* | Merge remote branch 'miloops/warnings' | José Valim | 2010-09-29 | 2 | -7/+3 |
|\ | |||||
| * | We can't assign @view_context_class here, define super() in test instead if ↵ | Emilio Tagua | 2010-09-29 | 1 | -1/+1 |
| | | | | | | | | we want to avoid warnings. | ||||
| * | _ is not a valid scheme name character, \w includes it and also is redundant ↵ | Emilio Tagua | 2010-09-29 | 1 | -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 Tagua | 2010-09-29 | 1 | -5/+1 |
| | | |||||
| * | Don't try to interpolate string if there's no interpolation point at all. | Emilio Tagua | 2010-09-29 | 1 | -1/+1 |
| | | |||||
| * | Initialize @view_context_class and cache view_context_class value. | Emilio Tagua | 2010-09-29 | 1 | -1/+1 |
| | | |||||
* | | no need to call e.class | Aaron Patterson | 2010-09-29 | 1 | -1/+1 |
| | | |||||
* | | Fix a small typo (ht: masterkain) | José Valim | 2010-09-29 | 1 | -1/+1 |
|/ | |||||
* | Ensure that named routes do not overwrite previously defined routes. | José Valim | 2010-09-29 | 1 | -10/+13 |
| | |||||
* | Fix an error on 1.8.7. | José Valim | 2010-09-29 | 1 | -1/+1 |
| | |||||
* | @_etag is not used anywhere. | José Valim | 2010-09-29 | 1 | -2/+0 |
| | |||||
* | Avoid (@_var ||= nil) pattern by using initialize methods and ensuring ↵ | José Valim | 2010-09-29 | 9 | -33/+41 |
| | | | | everyone calls super as expected. | ||||
* | Merge remote branch 'miloops/warnings' | José Valim | 2010-09-29 | 17 | -23/+39 |
|\ | |||||
| * | Define @_layout if it is not defined. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | | |||||
| * | Initialize @path. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | | |||||
| * | Remove more warnings on AP. | Emilio Tagua | 2010-09-28 | 2 | -3/+4 |
| | | |||||
| * | Remove more warnings: no need to define attr_accessor if already exists. ↵ | Emilio Tagua | 2010-09-28 | 1 | -3/+5 |
| | | | | | | | | Initialize ivar. | ||||
| * | Use redefine_method instead define_method, it may be already defined. | Emilio Tagua | 2010-09-28 | 1 | -1/+1 |
| | | |||||
| * | Remove method if exists, avoid calling Array#first so many times. | Emilio Tagua | 2010-09-28 | 1 | -3/+5 |
| | | |||||
| * | Remove more warnings on variables. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | | |||||
| * | Initialize @compiled_at if it is not. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | | |||||
| * | Remove warings from rescues: don't define more than once debug_hash method ↵ | Emilio Tagua | 2010-09-28 | 1 | -2/+2 |
| | | | | | | | | and ask if @response if defined. | ||||
| * | Remove remaining warnings on _trace by adding parenthesis to gsub arguments. | Emilio Tagua | 2010-09-28 | 1 | -3/+3 |
| | | |||||
| * | Use parenthesis to avoid ambiguous first argument warning. | Emilio Tagua | 2010-09-28 | 1 | -1/+1 |
| | | |||||
| * | Ask if the instance variable is defined before asking for it, avoid *many* ↵ | Emilio Tagua | 2010-09-28 | 1 | -1/+5 |
| | | | | | | | | warnings. | ||||
| * | Avoid more uninitialized variable warnings. | Emilio Tagua | 2010-09-28 | 2 | -2/+3 |
| | | |||||
| * | Initialize @sub_templates | Emilio Tagua | 2010-09-28 | 1 | -0/+1 |
| | | |||||
| * | @selected may not be defined here, avoid warning. | Emilio Tagua | 2010-09-28 | 1 | -1/+1 |
| | | |||||
| * | Don't define _test_case method if already defined. | Emilio Tagua | 2010-09-28 | 1 | -2/+4 |
| | | |||||
| * | No need to call super here. Use yield instead block.call | Emilio Tagua | 2010-09-28 | 1 | -3/+2 |
| | | |||||
| * | Initialize @app if it doesn't exists. | Emilio Tagua | 2010-09-28 | 1 | -1/+1 |
| | | |||||
| * | Move uri parser to AS as URI.parser method to reuse it in AP and ARes. | Emilio Tagua | 2010-09-27 | 5 | -28/+5 |
| | | |||||
* | | convert inject to map + join | Aaron Patterson | 2010-09-28 | 1 | -5/+5 |
| | | |||||
* | | Move uri parser to AS as URI.parser method to reuse it in AP and ARes. | Emilio Tagua | 2010-09-28 | 5 | -28/+5 |
| | | |||||
* | | Fix tests on 1.9.2. | José Valim | 2010-09-28 | 2 | -5/+1 |
| | | |||||
* | | renderer calls object.to_json when rendering :json => object [#5655 ↵ | Diego Carrion | 2010-09-27 | 1 | -1/+1 |
| | | | | | | | | | | | | state:resolved] 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 | 3 | -8/+8 |
| | | | | 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> | ||||
* | 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 | 18 | -51/+87 |
|\ | | | | | | | | | 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 |
| | | |||||
| * | 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 |
| | |