aboutsummaryrefslogtreecommitdiffstats
path: root/railties
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* 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.
* More info on commit messages in contributing guidePiotr Sarnacki2012-05-211-1/+33
| | | | | | | | | Add more info on how to write a good commit messages along with example showing nicely formatted commit message. Rails git history does not look too well when you try to figure out why particular changes were introduced. We can do much better than that and it's never too late to start.
* 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
* Changing the indentation level of the block comment close for SASS Brian Cardarella2012-05-151-1/+1
| | | If one wants to use use SASS for application.css.sass the comment block indentation is invalid.
* Default escape_html_entities_in_json to true in new appsJosé Valim2012-05-141-2/+2
|
* Update railties/lib/rails/generators/rails/app/templates/config/application.rbEgor Homakov2012-05-141-0/+3
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* :foreign_key option should be on has_many side of associationMikhail Dieterle2012-05-131-2/+2
|
* Update 'getting started' guides for new whitelist security implementation. ↵Erich Menge2012-05-121-1/+16
| | | | Closes #6286.
* Give more detailed instructions in script/rails in enginePiotr Sarnacki2012-05-041-0/+4
| | | | closes #4894
* improvements in "caching_with_rails" guide - backported from docrailskucaahbe2012-05-041-8/+10
| | | | | | Conflicts: railties/guides/source/caching_with_rails.textile
* Added test for assets:precompile for index filesDmitry Vorotilin2012-05-011-0/+9
|
* Failing test for #6034Piotr Sarnacki2012-04-301-0/+20
|
* Lazy load `default_form_builder` if it's passed as a stringPiotr Sarnacki2012-04-281-1/+1
| | | | closes #3341
* Allow Thor 0.15 and 1.0, to be released shortlytomhuda2012-04-281-1/+5
|
* Merge pull request #4528 from j-manu/log-tailer-fixAaron Patterson2012-04-271-3/+7
| | | | Fix for log tailer when the log file doesn't exist.
* Merge pull request #5919 from joevandyk/rake-dsl-fixJeremy Kemper2012-04-231-1/+1
|\ | | | | Only include Rake::DSL if it's defined (Rake >= 0.9)
| * Only include Rake::DSL if it's defined.Joe Van Dyk2012-04-211-1/+1
| | | | | | | | rake < 0.9 doesn't define Rake::DSL.
* | Updated other README to point 3-2-stableArun Agrawal2012-04-231-1/+1
|/
* Add hook for add_resource_routeSantiago Pastorino2012-04-184-7/+21
|
* Replace ruby-debug19 which doesnt work on 1.9.3 out of the box with a more ↵Aditya Sanghi2012-04-174-5/+5
| | | | maintained debugger gem
* "rails new -h" shows message in rails directoryask2012-04-132-3/+17
|
* Fix code example in generator test casePrem Sichanugrist2012-04-121-2/+2
|
* rewords the section about default_url_options in the Action Controller ↵Xavier Noria2012-04-091-3/+4
| | | | Overview guide
* Merge pull request #5784 from rafaelfranca/default_url-3-2Santiago Pastorino2012-04-081-2/+2
|\ | | | | [3-2-stable] Document that default_url_options must return a hash with symbolized keys
| * Document that default_url_options must return a hash with symbolizedRafael Mendonça França2012-04-091-0/+1
| | | | | | | | keys
| * default_url_options does not receive one argument anymoreRafael Mendonça França2012-04-081-2/+1
| |
* | Fix 'Everyday Git' linkAnil Wadghule2012-04-071-1/+1
| |
* | Fix Rails version in getting started guide.Anil Wadghule2012-04-061-4/+4
|/
* log which config format AR uses to connect withTerence Lee2012-04-031-0/+19
|
* test initializer logic for DATABASE_URL env varTerence Lee2012-04-031-0/+43
|
* Usage file in generators shouldn't be fetched only based on source_rootPiotr Sarnacki2012-04-023-5/+22
| | | | | | | | In case `source_roout` is not set, `default_source_root` is used, which includes also `templates` directory. If there is no `templates` directory, `default_source_root` is not available and USAGE will not be displayed. USAGE should be also checked based on default directory excluding `templates`.
* Merge branch '3-2-3' into 3-2-stableSantiago Pastorino2012-03-301-1/+1
|\
| * Bump up to 3.2.3Santiago Pastorino2012-03-301-1/+1
| |
| * Bumping to 3.2.3.rc2Santiago Pastorino2012-03-291-1/+1
| |
| * fix incorrect url in the deprecation message for vendor/plugins [ci skip]Vijay Dev2012-03-291-1/+1
| |
| * Merge pull request #5633 from drogus/embed-auth-token-in-remote-formsSantiago Pastorino2012-03-281-0/+2
| | | | | | | | Embed auth token in remote forms
* | fix incorrect url in the deprecation message for vendor/plugins [ci skip]Vijay Dev2012-03-291-1/+1
| |
* | Changed the config.assets.manifest to nil in the comments in the production ↵Anuj Dutta2012-03-281-1/+1
| | | | | | | | config template.
* | Added config.action_view.embed_authenticity_token_in_remote_formsPiotr Sarnacki2012-03-281-0/+2
| | | | | | | | | | | | | | | | There is a regression introduced in 16ee611fa, which breaks remote forms that should also work without javascript. This commit introduces config option that allows to configure this behavior defaulting to the old behavior (ie. include authenticity token in remote forms by default)
* | Add arel branch to the testsRafael Mendonça França2012-03-271-1/+1
| |
* | adding a branch to arelAaron Patterson2012-03-271-2/+2
|/
* Bumping to 3.2.3.rc1Santiago Pastorino2012-03-271-2/+2
|
* Fix 'Security#Mass Assignment' URL typoAbe Voelker2012-03-261-1/+1
|
* chdir before globbing so that we don't need to escape directory names.Aaron Patterson2012-03-231-1/+30
| | | | fixes #5521
* Fix GH #5411. When precompiling, params method is undefined.kennyj2012-03-221-0/+12
|
* [3-2-stable] Port of #5522 'Fix adding/removing field's index when ↵Marcelo Silveira2012-03-211-0/+18
| | | | generating migration'
* [3-2-stable] Remove blank line from generated migrationMarcelo Silveira2012-03-201-0/+1
|
* Build fix for app_generator_test.rbArun Agrawal2012-03-181-1/+1
|
* Fix GH #5435. db:structure:dump should be re-enable.kennyj2012-03-181-0/+12
|