Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | stop splatting so much. We don't need :star:args everywhere | Aaron Patterson | 2012-01-20 | 1 | -7/+6 |
| | |||||
* | Merge pull request #4497 from carlosantoniodasilva/action-controller-refactor | José Valim | 2012-01-17 | 1 | -1/+2 |
|\ | | | | | Action controller refactor - remove Compatibility module | ||||
| * | Remove deprecated default_charset= from AC::Base | Carlos Antonio da Silva | 2012-01-17 | 1 | -1/+2 |
| | | | | | | | | | | This should be set globally as a configuration, using `config.action_dispatch.default_charset` instead | ||||
* | | Deprecate RAILS_CACHE constant. | kennyj | 2012-01-18 | 1 | -3/+2 |
|/ | |||||
* | use Rack::BodyProxy in AD::Reloader | Sergey Nartimov | 2012-01-16 | 2 | -33/+3 |
| | |||||
* | Add begin/ensure block since we are returning. | José Valim | 2012-01-14 | 1 | -3/+5 |
| | |||||
* | Use a BodyProxy instead of including a Module that responds to close. | Santiago Pastorino | 2012-01-13 | 2 | -14/+31 |
| | | | | Closes #4441 if Active Record is disabled assets are delivered correctly | ||||
* | Delegate subdomain method to the URL module, removing duplication | Carlos Antonio da Silva | 2012-01-12 | 1 | -1/+1 |
| | |||||
* | Simplify named host logic and missing host condition check | Carlos Antonio da Silva | 2012-01-12 | 1 | -2/+2 |
| | |||||
* | Extract ip host regexp to a constant | Carlos Antonio da Silva | 2012-01-12 | 1 | -1/+3 |
| | |||||
* | Use MiniTest::Assertions instead of Test::Unit::Assertions | Rafael Mendonça França | 2012-01-06 | 1 | -2/+1 |
| | |||||
* | use assert_equal and assert_operator | Aaron Patterson | 2012-01-06 | 1 | -5/+8 |
| | |||||
* | use sprintf rather than build_message | Aaron Patterson | 2012-01-06 | 1 | -4/+6 |
| | |||||
* | Just use assert_equal | Aaron Patterson | 2012-01-06 | 1 | -3/+2 |
| | |||||
* | refactor assert_response | Aaron Patterson | 2012-01-06 | 1 | -7/+10 |
| | |||||
* | test response assertions | Aaron Patterson | 2012-01-06 | 1 | -8/+0 |
| | |||||
* | no need for assert_block | Aaron Patterson | 2012-01-06 | 1 | -3/+3 |
| | |||||
* | stop using build_message for creating a string | Aaron Patterson | 2012-01-06 | 1 | -1/+1 |
| | |||||
* | just use assert and refute rather than building messages | Aaron Patterson | 2012-01-06 | 1 | -6/+2 |
| | |||||
* | We should finalize the routes as soon as possible and not wait for the ↵ | José Valim | 2012-01-06 | 1 | -2/+0 |
| | | | | controller to be loaded. | ||||
* | Fix routing test to use assert_equal | Carlos Antonio da Silva | 2012-01-06 | 1 | -2/+2 |
| | |||||
* | Merge pull request #4117 from lest/remove-deprecated | José Valim | 2012-01-05 | 1 | -6/+0 |
|\ | | | | | remove status_code from AD::ShowExceptions and useless requires | ||||
| * | remove status_code from AD::ShowExceptions and useless requires | Sergey Nartimov | 2011-12-21 | 1 | -6/+0 |
| | | |||||
* | | Change more Array.wrap to Kernel#Array | Rafael Mendonça França | 2012-01-05 | 1 | -1/+1 |
| | | |||||
* | | Override respond_to? since we are also overriding method_missing. | José Valim | 2012-01-03 | 1 | -0/+4 |
| | | |||||
* | | Clean up routes inclusion and add some comments for the next soul that ↵ | José Valim | 2012-01-03 | 3 | -46/+49 |
| | | | | | | | | decides to adventure on this code. | ||||
* | | be explicit about where helpers are installed | Aaron Patterson | 2011-12-30 | 1 | -1/+1 |
| | | |||||
* | | just add the writer rather than adding both and removing one | Aaron Patterson | 2011-12-29 | 1 | -2/+1 |
| | | |||||
* | | stop using __send__ and just module eval in the extensions | Aaron Patterson | 2011-12-29 | 1 | -3/+2 |
| | | |||||
* | | we know the classes will be a list, so *tell* it to respond to each | Aaron Patterson | 2011-12-29 | 1 | -3/+3 |
| | | | | | | | | rather than casting | ||||
* | | don't need the begin / end | Aaron Patterson | 2011-12-29 | 1 | -21/+17 |
| | | |||||
* | | avoid extra method calls by just defining the delegate | Aaron Patterson | 2011-12-29 | 1 | -2/+3 |
| | | |||||
* | | don't use instance eval, just reference variables so we don't have to | Aaron Patterson | 2011-12-29 | 1 | -10/+11 |
| | | | | | | | | worry about "inspect" marshalling | ||||
* | | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2011-12-29 | 1 | -0/+7 |
|\ \ | |||||
| * | | Documented about using :path option for resources | Kevin Moore | 2011-12-28 | 1 | -0/+7 |
| | | | |||||
* | | | removing dead code. | Aaron Patterson | 2011-12-29 | 1 | -12/+3 |
| | | | |||||
* | | | decouple initialize from clear!. Initialize ivars in initialize, clear | Aaron Patterson | 2011-12-29 | 1 | -5/+5 |
| | | | | | | | | | | | | ivars in clear! | ||||
* | | | modules don't have any instance methods | Aaron Patterson | 2011-12-29 | 1 | -3/+1 |
| | | | |||||
* | | | simplify sweep now that discard and flashes are in sync | Aaron Patterson | 2011-12-28 | 1 | -8/+2 |
| | | | |||||
* | | | mutations on the underlying hash should also mutate the discard set | Aaron Patterson | 2011-12-28 | 1 | -3/+5 |
| | | | |||||
* | | | rename @used to something a bit more meaningful | Aaron Patterson | 2011-12-28 | 1 | -7/+7 |
| | | | |||||
* | | | mutations can't be done without the consent of our proxy object. This | Aaron Patterson | 2011-12-28 | 1 | -3/+0 |
| | | | | | | | | | | | | is one benefit of choosing composition over inheritance. | ||||
* | | | again, use Set#subtract | Aaron Patterson | 2011-12-28 | 1 | -1/+1 |
| | | | |||||
* | | | Use Set#subtract and Set#merge for keeping track of used / unused keys | Aaron Patterson | 2011-12-28 | 1 | -15/+4 |
| | | | |||||
* | | | fix method visibility | Aaron Patterson | 2011-12-28 | 1 | -15/+15 |
| | | | |||||
* | | | no need for bang bang :bomb: | Aaron Patterson | 2011-12-28 | 1 | -1/+1 |
| | | | |||||
* | | | Correctly display rack apps with dynamic constraints in RoutesInspector | Piotr Sarnacki | 2011-12-26 | 1 | -1/+1 |
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | If you used dynamic constraint like that: scope :constraint => MyConstraint.new do mount RackApp => "/foo" end routes were not displayed correctly when using `rake routes`. This commit fixes it. If you want nice display of dynamic constraints in `rake routes` output, please just override to_s method in your constraint's class. | ||||
* | | remove checks for encodings availability | Sergey Nartimov | 2011-12-25 | 2 | -2/+2 |
| | | |||||
* | | deprecate String#encoding_aware? and remove its usage | Sergey Nartimov | 2011-12-24 | 3 | -5/+3 |
| | | |||||
* | | Remove Rescue middleware that was never used by Rails. | José Valim | 2011-12-24 | 1 | -26/+0 |
| | |