Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Don't add sqlite3 to gemspec with -O on rails plugin new | Piotr Sarnacki | 2012-06-08 | 1 | -0/+8 | |
|/ | | | | (closes #6672) | |||||
* | Fixed the application_controller require_dependency path generated by the ↵ | Fred Wu | 2012-06-06 | 1 | -1/+3 | |
| | | | | app generator | |||||
* | Fixed the Gemfile when gemspec is skipped in the 'rails plugin new' command | Fred Wu | 2012-06-05 | 1 | -0/+26 | |
| | ||||||
* | Fix app_generator_test with the changes introduced at | Rafael Mendonça França | 2012-05-30 | 1 | -1/+1 | |
| | | | | f4d7af67ffc90f2542afa50c7579fc83ea4f45f2 | |||||
* | Remove unnecessary comment. | Uģis Ozols | 2012-05-23 | 1 | -1/+0 | |
| | ||||||
* | Use new hash syntax for generators gem method | Alexey Vakhov | 2012-05-22 | 2 | -2/+10 | |
| | | | | | | 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 controllers | Piotr Sarnacki | 2012-05-22 | 1 | -2/+2 | |
| | | | | | | | 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 generators to help with ambiguous `ApplicationController` issue | Piotr Sarnacki | 2012-05-20 | 1 | -5/+12 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 | |||||
* | Remove warning in namespaced generator test | Carlos Antonio da Silva | 2012-05-19 | 1 | -1/+1 | |
| | ||||||
* | Fix railties tests | Piotr Sarnacki | 2012-05-19 | 1 | -1/+1 | |
| | | | | | Minitest uses different signature for assert_no_match, so we have to swap arguments. | |||||
* | Revert changes related to `bundle install` fixes in `rails new` | Piotr Sarnacki | 2012-05-17 | 1 | -7/+2 | |
| | | | | | | | | | | | 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. | |||||
* | Fix tests in railties | Piotr Sarnacki | 2012-05-15 | 1 | -1/+1 | |
| | | | | | | | | Since `bundle install` was fixed in `rails plugin new`, it now requires `rails 4.0.0.beta` version in filesystem when running tests. Instead of providing it, we can run tested command with `--dev` option, to use rails from the local directory. | |||||
* | improve #6318 | Michal Papis | 2012-05-15 | 1 | -1/+6 | |
| | ||||||
* | Remove not used date variable and another test warning | Carlos Antonio da Silva | 2012-05-14 | 1 | -2/+1 | |
| | ||||||
* | Remove assertion on date from humans.txt tests | José Valim | 2012-05-14 | 1 | -1/+1 | |
| | ||||||
* | Refactor App Generator Test | Oscar Del Ben | 2012-05-13 | 1 | -22/+20 | |
| | ||||||
* | Failing test for #6251 | Piotr Sarnacki | 2012-05-10 | 1 | -0/+7 | |
| | ||||||
* | Adding test for humans.txt | Arun Agrawal | 2012-05-08 | 1 | -0/+6 | |
| | ||||||
* | Fix build for railties generators | Arun Agrawal | 2012-05-02 | 2 | -3/+3 | |
| | ||||||
* | build fix for SharedGeneratorTests | Arun Agrawal | 2012-05-02 | 1 | -2/+2 | |
| | ||||||
* | Build fix for plugin new generator change | Carlos Antonio da Silva | 2012-05-01 | 1 | -2/+2 | |
| | | | | Related to a06a84bf77082a7435973fa1b6c8254fb410f243 | |||||
* | Fix tests | Piotr Sarnacki | 2012-05-01 | 1 | -1/+1 | |
| | ||||||
* | Merge pull request #5530 from ↵ | José Valim | 2012-04-30 | 1 | -0/+7 | |
|\ | | | | | | | | | malclocke/add_option_to_skip_index.html_on_rails_generate Added a generator option to skip the public/index.html file | |||||
| * | Added a generator option to remove the public/index.html file when ↵ | Ivan Vanderbyl | 2012-03-21 | 1 | -0/+7 | |
| | | | | | | | | | | | | | | generating a new Rails application The option is: -i, [--skip-index-html] # Skip public/index.html file | |||||
* | | Namedspaced generator indent method test refactoring | Alexey Vakhov | 2012-04-26 | 1 | -3/+4 | |
| | | ||||||
* | | Merge pull request #5902 from avakhov/generator-no-indent-blank-lines | José Valim | 2012-04-25 | 1 | -0/+8 | |
|\ \ | | | | | | | Don't indent blank lines in named base generators | |||||
| * | | Don't indent blank lines in named base generators | Alexey Vakhov | 2012-04-25 | 1 | -0/+8 | |
| | | | ||||||
* | | | Remove lonely number sign | Alexey Vakhov | 2012-04-25 | 1 | -1/+1 | |
|/ / | ||||||
* | | Fix test | Santiago Pastorino | 2012-04-16 | 1 | -2/+2 | |
| | | ||||||
* | | fix scaffold_generator_test.rb and model_generator_test.rb | Arun Agrawal | 2012-04-16 | 2 | -8/+6 | |
| | | | | | | | | Broken after this 6a054b0038bac288a1f6e45feb5470f4ee492081 | |||||
* | | "rails new -h" shows message in rails directory | ask | 2012-04-09 | 1 | -0/+10 | |
| | | ||||||
* | | Usage file in generators shouldn't be fetched only based on source_root | Piotr Sarnacki | 2012-04-01 | 2 | -1/+5 | |
| | | | | | | | | | | | | | | | | 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`. | |||||
* | | Avoid another blank line in generated migration and remove assertion as per ↵ | Marcelo Silveira | 2012-03-21 | 1 | -1/+0 | |
| | | | | | | | | @spastorino request | |||||
* | | Merge pull request #5532 from mhfs/migration_blank_line | José Valim | 2012-03-21 | 1 | -0/+1 | |
|\ \ | | | | | | | Remove blank line from generated migration | |||||
| * | | Remove blank line from generated migration | Marcelo Silveira | 2012-03-20 | 1 | -0/+1 | |
| |/ | ||||||
* / | Generate Migration Thats Adds Removed Index | Travis Jeffery | 2012-03-21 | 1 | -0/+18 | |
|/ | | | | | When generating a migration that removes a field with an index, the down will add both the field and its index. | |||||
* | Build fix for app_generator_test.rb | Arun Agrawal | 2012-03-18 | 1 | -1/+1 | |
| | ||||||
* | Remove --http. | José Valim | 2012-03-14 | 3 | -76/+0 | |
| | ||||||
* | Do not generate app/views and sprockets entries on http app | Carlos Antonio da Silva | 2012-03-14 | 1 | -1/+16 | |
| | | | | [Carlos Antonio da Silva & Santiago Pastorino] | |||||
* | Generate special controller and functional test templates for http apps | Carlos Antonio da Silva | 2012-03-14 | 2 | -0/+43 | |
| | | | | | | | | | | | | The main goal is to not generate the format.html block in scaffold controller, and to generate a different functional test as we don't rely on redirects anymore, we should test for http responses. In addition to that, the :edit action is removed from the http controller and the edit route is not generated by default, as they usually do not make sense in this scenario. [Carlos Antonio da Silva & Santiago Pastorino] | |||||
* | Rename http-only app generator option to http | Carlos Antonio da Silva | 2012-03-14 | 1 | -5/+5 | |
| | | | | [Carlos Antonio da Silva & Santiago Pastorino] | |||||
* | Remove duplicated tests from shared generator | Carlos Antonio da Silva | 2012-03-14 | 2 | -16/+2 | |
| | | | | [Carlos Antonio da Silva & Santiago Pastorino] | |||||
* | Generate middleware http_only! config when running http only app generator | Carlos Antonio da Silva | 2012-03-14 | 1 | -0/+6 | |
| | | | | [Carlos Antonio da Silva & Santiago Pastorino] | |||||
* | Add http-only option to Rails app generator | Carlos Antonio da Silva | 2012-03-14 | 1 | -0/+10 | |
| | | | | | | | Change application controller template accordingly, to inherit from ActionController::HTTP and not generate protect_from_forgery call. [Carlos Antonio da Silva & Santiago Pastorino] | |||||
* | The sprockets railtie was moved to sprockets/rails/railtie | Guillermo Iguaran | 2012-03-13 | 1 | -1/+1 | |
| | ||||||
* | Refactor the test_unit scaffold generator to use the key_value method | Rafael Mendonça França | 2012-03-13 | 2 | -8/+8 | |
| | ||||||
* | Use the attributes hash explicitly | Rafael Mendonça França | 2012-03-13 | 2 | -6/+26 | |
| | ||||||
* | Do not use the attributes hash in the scaffold functional tests | Rafael Mendonça França | 2012-03-12 | 2 | -2/+9 | |
| | ||||||
* | Revert table_name= needs connection now, so just stub table_name | kennyj | 2012-03-07 | 1 | -1/+1 | |
| | ||||||
* | Only add the whitelist_attributes option if ActiveRecord is present | Rafael Mendonça França | 2012-03-04 | 1 | -0/+6 | |
| |