Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | Remove more warnings by initializing variables in test. | Emilio Tagua | 2010-09-28 | 1 | -0/+6 | |
| | | | ||||||
| * | | Remove more warnings: no need to define attr_accessor if already exists. ↵ | Emilio Tagua | 2010-09-28 | 2 | -3/+6 | |
| | | | | | | | | | | | | Initialize ivar. | |||||
| * | | Use redefine_method instead define_method, it may be already defined. | Emilio Tagua | 2010-09-28 | 1 | -1/+1 | |
| | | | ||||||
| * | | Remove duplicated class test. copy/paste fail? | Emilio Tagua | 2010-09-28 | 1 | -32/+0 | |
| | | | ||||||
| * | | 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 @trusted_proxies. | 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 | |
| | | | ||||||
| * | | undef method if already defined. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 | |
| | | | ||||||
| * | | Remove useless use of :: in void context. | Emilio Tagua | 2010-09-28 | 1 | -1/+0 | |
| | | | ||||||
| * | | Ask if the instance variable is defined before asking for it, avoid *many* ↵ | Emilio Tagua | 2010-09-28 | 1 | -1/+5 | |
| | | | | | | | | | | | | warnings. | |||||
| * | | Add parenthesis to avoid syntax warnings. | Emilio Tagua | 2010-09-28 | 2 | -2/+2 | |
| | | | ||||||
| * | | Avoid more uninitialized variable warnings. | Emilio Tagua | 2010-09-28 | 2 | -2/+3 | |
| | | | ||||||
| * | | Remove duplicated method. | Emilio Tagua | 2010-09-28 | 1 | -2/+0 | |
| | | | ||||||
| * | | Initialize @sub_templates | Emilio Tagua | 2010-09-28 | 1 | -0/+1 | |
| | | | ||||||
| * | | Initialize @_virtual_path path ivar. | Emilio Tagua | 2010-09-28 | 1 | -0/+1 | |
| | | | ||||||
| * | | Rename tests to avoid name collisions and warnings when running rake task. | Emilio Tagua | 2010-09-28 | 1 | -5/+5 | |
| | | | ||||||
| * | | Rename duplicated test, and give it a correct name. Remove nonsense line. | Emilio Tagua | 2010-09-28 | 1 | -2/+1 | |
| | | | ||||||
| * | | Remove duplicated test. | Emilio Tagua | 2010-09-28 | 1 | -20/+0 | |
| | | | ||||||
| * | | @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 | 8 | -43/+25 | |
| | | | ||||||
* | | | Add gem 'arel' from git commented out as an example of how to Bundle egde ↵ | Santiago Pastorino | 2010-09-29 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | Rails with Arel edge [#5723 state:committed] | |||||
* | | | removing more useless code! yay! | Aaron Patterson | 2010-09-28 | 1 | -6/+2 | |
| | | | ||||||
* | | | SqlLiteral is a string, so we can dry up these conditionals | Aaron Patterson | 2010-09-28 | 1 | -4/+3 | |
| | | | ||||||
* | | | shorten up or sql literal creation statements | Aaron Patterson | 2010-09-28 | 1 | -4/+2 | |
| | | | ||||||
* | | | we only care about arrays and strings | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | ||||||
* | | | dry up calls to arel.join() | Aaron Patterson | 2010-09-28 | 1 | -5/+4 | |
| | | | ||||||
* | | | removing unused lasgns | Aaron Patterson | 2010-09-28 | 1 | -4/+4 | |
| | | | ||||||
* | | | refactoring to remove crazy logic | Aaron Patterson | 2010-09-28 | 2 | -9/+6 | |
| | | | ||||||
* | | | where_values_hash always returns a hash | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | ||||||
* | | | performance improvement based on discussion at ↵ | Neeraj Singh | 2010-09-28 | 1 | -9/+3 | |
| | | | | | | | | | | | | | | | | | | http://github.com/rails/rails/commit/fbd1d306b95cc2efb6422e12d26d5818a3a42343 Credit goes to all the participants in the discussion | |||||
* | | | Revert "porting 515917f5d8678af6c57842ca5dfd7c18e67ff1fe to master" | Aaron Patterson | 2010-09-28 | 1 | -6/+0 | |
| | | | | | | | | | | | | This reverts commit bee447a5b9fe1d683c6cc69aefb7fc22c2a9d9af. | |||||
* | | | Revert "porting 066518295032a8e3f3468737337b8c8299442867 to master. Thanks ↵ | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | Marcelo Giorgi" This reverts commit 9eca11a4a564f44675cca951216e917b8f610eab. | |||||
* | | | removing a conditional that is not used | Aaron Patterson | 2010-09-28 | 1 | -2/+0 | |
| | | | ||||||
* | | | fisting the postgresql tests | Aaron Patterson | 2010-09-28 | 1 | -1/+2 | |
| | | | ||||||
* | | | Set attributes properly for model built from association with conditions ↵ | Marcelo Giorgi | 2010-09-28 | 8 | -10/+61 | |
| | | | | | | | | | | | | | | | | | | [#5562 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | | | convert inject to map + join | Aaron Patterson | 2010-09-28 | 1 | -5/+5 | |
| | | | ||||||
* | | | each works well too | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | ||||||
* | | | no need for splat and flatten | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | ||||||
* | | | avoid calling to_sql when we can | Aaron Patterson | 2010-09-28 | 2 | -4/+4 | |
| | | | ||||||
* | | | removing an inject + merge in favor of Hash#[] | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | ||||||
* | | | fixing indentation warning | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | ||||||
* | | | Test add_index and remove_index with a symbol name #4891 | Étienne Barrié | 2010-09-28 | 2 | -4/+10 | |
| | | | ||||||
* | | | porting 066518295032a8e3f3468737337b8c8299442867 to master. Thanks Marcelo ↵ | Aaron Patterson | 2010-09-28 | 1 | -1/+1 | |
| | | | | | | | | | | | | Giorgi |