Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | | | | | | Remove unused in AV fixture-controllers | Łukasz Strzałkowski | 2013-08-25 | 1 | -29/+0 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Indent protected methods | Łukasz Strzałkowski | 2013-08-25 | 1 | -18/+18 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | [ci skip] | |||||
| * | | | | | | | | | Remove view_assigns from AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -7/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It wasn't adding anything, just invoking super | |||||
| * | | | | | | | | | Load AV::Layouts dynamicly via railties | Łukasz Strzałkowski | 2013-08-25 | 5 | -4/+5 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Code formatting & typo fixes | Łukasz Strzałkowski | 2013-08-25 | 3 | -37/+37 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Move anonymous class to the top, add documentation | Łukasz Strzałkowski | 2013-08-25 | 1 | -4/+9 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Remove abstract_controller load hooks | Łukasz Strzałkowski | 2013-08-25 | 2 | -6/+0 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Move protected_instance_variables & view_assigns to AbstractController | Łukasz Strzałkowski | 2013-08-25 | 2 | -16/+24 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Include AbsC::Rendering interface in ActionMailer | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+3 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | No need for .rb extension | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Remove 'api plugin' from docs | Łukasz Strzałkowski | 2013-08-25 | 1 | -3/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | AbstractController::Rendering is module which defines thie api | |||||
| * | | | | | | | | | Create AbstractController::Rendering interface | Łukasz Strzałkowski | 2013-08-25 | 11 | -20/+69 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This interface should be use when implementing renderers. | |||||
| * | | | | | | | | | Revert "Rename abstract_controller/rendering. to errors.rb" | Łukasz Strzałkowski | 2013-08-25 | 2 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 6fe91ec5008838338e54ab8570f7c95ee0cdd8e8. | |||||
| * | | | | | | | | | Rename abstract_controller/rendering. to errors.rb | Łukasz Strzałkowski | 2013-08-25 | 2 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since all rendering stuff was extracted to AV, the only thing that left was single class with error so file name wasn't relevant anymore | |||||
| * | | | | | | | | | Fix AP test suite after moving stuff to AV | Łukasz Strzałkowski | 2013-08-25 | 7 | -6/+9 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Do not include AV::Rendering | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+0 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It's already included in Layouts, makes no sense to doubly include it | |||||
| * | | | | | | | | | Integrate ActionMailer with ActionView | Łukasz Strzałkowski | 2013-08-25 | 2 | -2/+6 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Fix eager_autoload for layouts | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Name was singular, should be plural | |||||
| * | | | | | | | | | Fix AV tests, I18nProxy was moved to AV | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+1 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Revert "Require only path_set && lookup_context instead of whole base" | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 77bb03df2ca3687b290e9fe77d21007642f0329c. | |||||
| * | | | | | | | | | Hook up AV::Rendering on AV intialization | Łukasz Strzałkowski | 2013-08-25 | 3 | -3/+7 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Move layouts to AV | Łukasz Strzałkowski | 2013-08-25 | 5 | -2/+432 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Add loading 'action_view' as part of rails/all | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+1 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Move rendering from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 7 | -613/+189 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Require only path_set && lookup_context instead of whole base | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+2 | |
| | | | | | | | | | | ||||||
| * | | | | | | | | | Move view_paths from AP to AV | Łukasz Strzałkowski | 2013-08-25 | 6 | -2/+2 | |
| | |_|_|/ / / / / | |/| | | | | | | | ||||||
* | | | | | | | | | PG adapter deals with negative money values formatted with parenthesis. | Yves Senn | 2013-08-26 | 3 | -0/+19 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #11899. | |||||
* | | | | | | | | | Merge pull request #12014 from wangjohn/change_deprecation_horizon | Santiago Pastorino | 2013-08-25 | 2 | -2/+2 | |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Changing deprecation_horizon to be Rails 4.2 | |||||
| * | | | | | | | | | Changing deprecation_horizon to be Rails 4.2 | wangjohn | 2013-08-24 | 2 | -2/+2 | |
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, +ActiveRecord::Migrator.proper_table_name+ should actually have a deprecation horizon of Rails 4.2 (not 4.1). | |||||
* | | | | | | | | | Merge pull request #12009 from wangjohn/fixtures_fix | Santiago Pastorino | 2013-08-25 | 3 | -11/+14 | |
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | | | | Removing instances of string class_names in fixtures. | |||||
| * | | | | | | | | Removing instances of string class_names in fixtures. | wangjohn | 2013-08-24 | 3 | -11/+14 | |
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also, constantizing the default_fixture_model_name when it gets loaded in from the file. Later, when the class_name is passed to a new FixtureSet, a deprecation warning will occur if the class_name is a string. | |||||
* | | | | | | | | Merge pull request #12008 from wangjohn/skipping_deprecated_migration_tests | Santiago Pastorino | 2013-08-24 | 1 | -7/+21 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | Skip tests using deprecated +proper_table_name+ method. | |||||
| * | | | | | | | | Adding deprecation assertions for proper_table_name. | wangjohn | 2013-08-24 | 1 | -7/+21 | |
|/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | This prevents deprecation warnings from popping up. | |||||
* | | | | | | | | Merge pull request #12000 from ↵ | Santiago Pastorino | 2013-08-23 | 1 | -7/+1 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | wangjohn/deprecating_class_name_string_in_fixtures Deprecating passing strings as class name in fixtures. | |||||
| * | | | | | | | | Deprecating passing strings as class name in fixtures. | wangjohn | 2013-08-23 | 1 | -7/+1 | |
| | | | | | | | | | ||||||
* | | | | | | | | | Merge pull request #11984 from wangjohn/proper_table_name | Santiago Pastorino | 2013-08-23 | 2 | -19/+68 | |
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Making proper_table_name take in options. | |||||
| * | | | | | | | | | Making proper_table_name take in options. | wangjohn | 2013-08-22 | 2 | -19/+68 | |
| | |_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The options will specify the prefix and the suffix. Also, I'm moving the method to be an instance method on the +Migration+ instance. This makes more sense than being a class method on the +Migrator+ class because the only place that uses it is on a +Migration+ instance (in a method_missing hook). The logic for the Migrator shouldn't be doing any work to calculate the table name, it should be the Migration itself. Also made some small indentation fixes. | |||||
* | | | | | | | | | typo [ci skip] | Xavier Noria | 2013-08-23 | 1 | -1/+1 | |
| | | | | | | | | | ||||||
* | | | | | | | | | Merge pull request #11960 from kirs/xhr-errors | Rafael Mendonça França | 2013-08-23 | 15 | -13/+129 | |
|\ \ \ \ \ \ \ \ \ | |_|/ / / / / / / |/| | | | | | | | | Show exceptions in text format for xhr requests | |||||
| * | | | | | | | | Display exceptions in text format for xhr request | Kir Shatrov | 2013-08-22 | 15 | -13/+129 | |
| | | | | | | | | | ||||||
* | | | | | | | | | cleans the guides sources from fancy non-ASCII stuff | Xavier Noria | 2013-08-23 | 21 | -96/+96 | |
| | | | | | | | | | ||||||
* | | | | | | | | | Merge pull request #11996 from semaperepelitsa/master | Yves Senn | 2013-08-23 | 1 | -1/+1 | |
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|/ / / / |/| | | | | | | | | Docs: fix typo in file path, should include extension. [ci skip] | |||||
| * | | | | | | | | Fix typo in file path, should include extension. | Semyon Perepelitsa | 2013-08-23 | 1 | -1/+1 | |
|/ / / / / / / / | ||||||
* | | | | | | | | Merge pull request #11994 from rajcybage/fix_instance | Carlos Antonio da Silva | 2013-08-23 | 1 | -2/+1 | |
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | remove not used instance variable | |||||
| * | | | | | | | | remove unused instance variable | Rajarshi Das | 2013-08-23 | 1 | -2/+1 | |
| | | | | | | | | | ||||||
* | | | | | | | | | Merge pull request #11991 from prathamesh-sonpatki/note-edge-guides | Yves Senn | 2013-08-23 | 1 | -2/+4 | |
|\ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / |/| | | | | | | | | Add a note about edge guides in the feedback section [ci skip] | |||||
| * | | | | | | | | Add a note about edge guides in the feedback section[ci skip] | Prathamesh Sonpatki | 2013-08-23 | 1 | -2/+4 | |
|/ / / / / / / / | ||||||
* | | | | | | | | Merge pull request #11971 from freerunningtechnologies/dynamic_respond_to | Aaron Patterson | 2013-08-22 | 2 | -1/+6 | |
|\ \ \ \ \ \ \ \ | |_|/ / / / / / |/| | | | | | | | Performance regression in AR::Base.respond_to? | |||||
| * | | | | | | | Avoid compiling regexs in AR::Base.respond_to? | John Hawthorn | 2013-08-21 | 2 | -1/+6 | |
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | Caches the patterns of ActiveRecord::DynamicMatchers in a class instance variable. | |||||
* | | | | | | | Merge pull request #11976 from etehtsea/inf | Yves Senn | 2013-08-22 | 1 | -2/+2 | |
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | [PG adapter] Use infinity const |