aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
...
* | Unused variable warning removed.Arun Agrawal2012-05-301-1/+1
| |
* | More assert_match warnings fixed.Arun Agrawal2012-05-304-12/+12
| |
* | Warnings removed for "assert_match /".Arun Agrawal2012-05-301-20/+20
| |
* | Add support runner hook.kennyj2012-05-296-0/+53
| |
* | Add a testcase for #5847.kennyj2012-05-291-0/+23
| |
* | update Gemfile template to 1.9 hash syntaxFrancesco Rodriguez2012-05-291-1/+1
| |
* | Fix railties_order when application object is passedPiotr Sarnacki2012-05-272-1/+5
|/ | | | | | | | | | | | | | | railites_order method, introduced in 40b19e0, had a bug that was causing loading application instance twice in initializers if railties_order already included application instance. So for example railties_order = [Foo::Engine, :main_app, Bar::Engine] would result in such railties array: [MyApp::Application, Foo::Engine, MyAppApplication, Bar::Engine] In order to fix it, we need to check for existence of application in both railties_order and railties arrays.
* Fixes build 4008.1Waseem Ahmad2012-05-251-2/+2
| | | | | | | | This fixex build 4008.1[1] because of the changes made in 4001835db00ce44cb75bca33ec02cd76b8ccc790 [1] http://travis-ci.org/#!/rails/rails/jobs/1429671
* Merge pull request #6481 from schneems/schneems/sextant-routesRafael Mendonça França2012-05-241-9/+0
|\ | | | | Remove Unnecessary Test from route_inspect_test
| * remove unnecessary test from route_inspect_testschneems2012-05-241-9/+0
| |
* | do not set the ENGINE_PATH to nilAaron Patterson2012-05-241-1/+1
| |
* | `name` should be public.Aaron Patterson2012-05-241-12/+12
|/
* Rails::InfoController tests passingschneems2012-05-243-6/+32
| | | | This includes new tests for /rails/info/routes
* /rails/info/routes path shows routing informationschneems2012-05-247-8/+72
| | | | Will show similar contents to the output of `$ rake routes` in the browser in development. This speeds the time required to generate routes, since the application is already initialized.
* Fixed backward incompatibility for engines.Philip Arndt2012-05-241-1/+1
| | | | | | | | | | | | - Many engines rely on being able to join directories to the Rails root: Rails.root.join('somedir') - This was now impossible because Rails.root returned a String: NoMethodError: undefined method `join' for "/code/myrailsapp":String - This was broken in 4001835db00ce44cb75bca33ec02cd76b8ccc790
* use File.join rather than depend on PathnameAaron Patterson2012-05-231-1/+1
|
* Revert "require the constants we use. ensure that root always returns a ↵Aaron Patterson2012-05-231-2/+1
| | | | | | Pathname" This reverts commit d77b576c0330d8b1c6189cb94814382ce32baab6.
* require the constants we use. ensure that root always returns a PathnameAaron Patterson2012-05-231-1/+2
|
* removing more pathnameismsAaron Patterson2012-05-232-6/+4
|
* use File.join to decrease dependencies on PathnameAaron Patterson2012-05-231-1/+1
|
* Merge branch 'master' of github.com:lifo/docrailsVijay Dev2012-05-231-2/+1
|\
| * Revert "Remove blank trailing comments"Vijay Dev2012-05-2310-0/+23
| | | | | | | | | | | | | | | | | | | | This reverts commit fa6d921e11363e9b8c4bc10f7aed0b9faffdc33a. Reason: Not a fan of such massive changes. We usually close such changes if made to Rails master as a pull request. Following the same principle here and reverting. [ci skip]
| * Remove obsolete comment from generator gem methodAlexey Vakhov2012-05-221-2/+1
| | | | | | | | | | This comment about :env option for gem method was actual for rails 2.3 without bundler. Now bundler uses grups for this goal.
| * Remove blank trailing commentsHenrik Hodne2012-05-2010-23/+0
| | | | | | | | | | | | | | For future reference, this is the regex I used: ^\s*#\s*\n(?!\s*#). Replace with the first match, and voilà! Note that the regex matches a little bit too much, so you probably want to `git add -i .` and go through every single diff to check if it actually should be changed.
* | Add license field to gemspecs, by Matt GriffinMatt Griffin2012-05-231-0/+1
| |
* | Remove unnecessary comment.Uģis Ozols2012-05-231-1/+0
| |
* | I guess we have to return a pathname object. o_OAaron Patterson2012-05-221-1/+1
| |
* | use File.realpath and avoid making Pathname objectsAaron Patterson2012-05-221-2/+1
| |
* | use RUBY_PLATFORM in case of cross compiled rubyAaron Patterson2012-05-221-1/+1
| |
* | initialize instance variablesAaron Patterson2012-05-221-2/+6
| |
* | initialize our instance variablesAaron Patterson2012-05-221-2/+14
| |
* | Move root method at TOP of routes fileGaurish Sharma2012-05-221-3/+4
| | | | | | | | | | | | | | | | Made the change as per the following text in routing guide:- "You should put the root route at the top of the file, because it is the most popular route and should be matched first." However, if root is best left at bottom. We will change to fix that text
* | Use new hash syntax for generators gem methodAlexey Vakhov2012-05-224-4/+12
| | | | | | | | | | | | The Gemfile of new application uses ruby 1.9 hashes. Gem method of generators should use them too. It prevents from mixing two kinds of syntax in one file.
* | Use require_dependency in generated controllersPiotr Sarnacki2012-05-223-4/+4
| | | | | | | | | | | | | | Using require in development mode will prevent required files from reloading, even if they're changed. In order to keep namespaced application_controller reloadable, we need to use require_dependency instead of require.
* | Fix `rails db -h` and cosmetic fixes in usage bannersAlexey Vakhov2012-05-224-3/+26
| | | | | | | | | | | | | | Ruby tries to use '-h' as short version of '--header' by default https://github.com/ruby/ruby/blob/trunk/lib/optparse.rb#L1372-1381. To force `rails db -h` prints an usage message we should add the `-h` options explicitly.
* | Use relative path to sqlite3 db in `rails db` commandAlexey Vakhov2012-05-222-7/+12
| | | | | | | | | | | | | | | | | | | | Rails uses sqlit3 db file with a path relative to the rails root. It allows to execute server not from rails root only. For example you can fire `./spec/dummy/script/rails s` to start dummy application server if you develop some engine gem. Now the `rails db` command uses relative paths also and you can explore your dummy db via `./spec/dummy/script/rails db` command.
* | using __method__ for the command method callsAaron Patterson2012-05-212-5/+64
| |
* | Fix generators to help with ambiguous `ApplicationController` issuePiotr Sarnacki2012-05-204-5/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In development mode, dependencies are loaded dynamically at runtime, using `const_missing`. Because of that, when one of the constants is already loaded and `const_missing` is not triggered, user can end up with unexpected results. Given such file in an Engine: ```ruby module Blog class PostsController < ApplicationController end end ``` If you load it first, before loading any application files, it will correctly load `Blog::ApplicationController`, because second line will hit `const_missing`. However if you load `ApplicationController` first, the constant will be loaded already, `const_missing` hook will not be fired and in result `PostsController` will inherit from `ApplicationController` instead of `Blog::ApplicationController`. Since it can't be fixed in `AS::Dependencies`, the easiest fix is to just explicitly load application controller. closes #6413
* | Merge pull request #3359 from mrreynolds/datamapper_naming_fixRafael Mendonça França2012-05-194-13/+13
|\ \ | |/ |/| Fixed DataMapper namings in symbols and constants.
| * Fixed DataMapper namings in symbols and constants.Robert Glaser2011-10-184-13/+13
| |
* | Add CHANGELOG entry for #6369 & #6391Andrew White2012-05-191-0/+2
| |
* | Remove delegated methods from generators test caseCarlos Antonio da Silva2012-05-191-1/+0
| | | | | | | | | | | | | | `class_attribute` already generates instance readers/writers by default, so there's no need to add delegation methods. Removes some "method redefined" warnings.
* | Remove warning in namespaced generator testCarlos Antonio da Silva2012-05-191-1/+1
| |
* | Fix railties testsPiotr Sarnacki2012-05-191-1/+1
| | | | | | | | | | Minitest uses different signature for assert_no_match, so we have to swap arguments.
* | Improve `rake routes` output for redirects - closes #6369.Andrew White2012-05-192-5/+8
| |
* | Fix inspecting route redirections, closes #6369Łukasz Strzałkowski2012-05-182-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | This commit fixes route inspection in `rake routes` Before: foo /foo(.:format) :controller#:action" After: foo /foo(.:format) Redirect (301)
* | More DRY route inspect testsŁukasz Strzałkowski2012-05-181-26/+18
| |
* | Revert changes related to `bundle install` fixes in `rails new`Piotr Sarnacki2012-05-172-17/+3
| | | | | | | | | | | | | | | | | | | | | | Reverted changes: f3482a9 Fix tests in railties 5904295 improve #6318 aed906a prevent using already loaded Gemfile for 'bundle install' In order to fix this, we need a fix in bundler related to GEM_PATH, which will allow to run tests properly. I will get this changes back when it happens.
* | Load all the env files available in config.paths (closes #6335)Piotr Sarnacki2012-05-163-3/+14
| |
* | Merge pull request #6337 from bcardarella/patch-1Aaron Patterson2012-05-161-1/+1
|\ \ | | | | | | Changing the indentation level of the block comment close for Sass