Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Remove namespace for isolated namespaced models in forms | Piotr Sarnacki | 2010-09-03 | 4 | -6/+101 |
| | |||||
* | Use new ActiveModel::Naming.route_key in polymorphic_routes | Piotr Sarnacki | 2010-09-03 | 2 | -12/+3 |
| | |||||
* | Prepared ActiveModel::Naming to handle cases for namespaced isolated engines | Piotr Sarnacki | 2010-09-03 | 3 | -4/+134 |
| | |||||
* | Add some more docs on polymorphic_url with routes proxy | Piotr Sarnacki | 2010-09-03 | 1 | -0/+10 |
| | |||||
* | Add short note on using url_for instead of directly calling named route in ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -0/+3 |
| | | | | polymorphic_url | ||||
* | Add possibility to explicitly call engine's routes through ↵ | Piotr Sarnacki | 2010-09-03 | 3 | -1/+58 |
| | | | | polymorphic_routes, for example: polymorphic_url([blog, @post]) | ||||
* | Updated tests to use scope(:module => :engine_name) instead of namespace and ↵ | Piotr Sarnacki | 2010-09-03 | 2 | -23/+28 |
| | | | | updated mounted engine tests to actually use the namespacing | ||||
* | Move RoutesProxy to separate file | Piotr Sarnacki | 2010-09-03 | 3 | -32/+36 |
| | |||||
* | Include all helpers from non-namespaced engines | Piotr Sarnacki | 2010-09-03 | 4 | -12/+38 |
| | |||||
* | Set only helpers_path on inherited hook in action_controller/railtie.rb and ↵ | Piotr Sarnacki | 2010-09-03 | 5 | -38/+17 |
| | | | | use helper(:all) just after that | ||||
* | Added more tests for polymorphic_url with namespaced models and implemented ↵ | Piotr Sarnacki | 2010-09-03 | 2 | -5/+66 |
| | | | | missing use cases | ||||
* | This was used only to clear warning in ActionMailer tests, it shouldn't be ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -5/+0 |
| | | | | done like that | ||||
* | Moved ActionMailer and ActionController railties options to inherited hook | Piotr Sarnacki | 2010-09-03 | 5 | -22/+61 |
| | | | | | | | | | | | This change is needed, because we must take namespace into account and if controller's/mailer's class is namespaced, engine's paths should be set instead of application's ones. The nice side effect of this is removing unneeded logic in ActionController::Base.inherited - now the helpers_path should be set correctly even for engine's controllers, so helper(:all) will always include correct helpers. | ||||
* | Modified polymorphic_url to check for model's namespace | Piotr Sarnacki | 2010-09-03 | 2 | -4/+34 |
| | | | | | | | | | | | This change allows using namespaced models with polymorphic_url, in the way that you would use them without namespace. Let's say that you have Blog::Post model in namespaced Engine. When you use polymorphic_path with Blog::Post instances, like in form_for(@post), it will look for blog_posts_path named url helper. As we are inside Blog::Engine, it's annoying to always use the prefix. With this commit, blog_ prefix will be removed and posts_path will be called. | ||||
* | Get rid of static_paths method and instead configure paths for ↵ | Piotr Sarnacki | 2010-09-03 | 6 | -17/+29 |
| | | | | ActionDispatch::Static in initializers | ||||
* | We don't need to clear action_methods on inherited hook as they are cleaned ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -5/+0 |
| | | | | on method_added hook | ||||
* | ActionMailer should not depend on ActionDispatch | Piotr Sarnacki | 2010-09-03 | 3 | -3/+2 |
| | |||||
* | Refactor RoutesProxy to avoid using _with_routes in helpers | Piotr Sarnacki | 2010-09-03 | 3 | -4/+10 |
| | |||||
* | Documented mounted helpers | Piotr Sarnacki | 2010-09-03 | 1 | -0/+29 |
| | |||||
* | Document engine_name | Piotr Sarnacki | 2010-09-03 | 1 | -0/+16 |
| | |||||
* | Added documentation for namespaced Engine | Piotr Sarnacki | 2010-09-03 | 1 | -2/+34 |
| | |||||
* | Add table_name_prefix to Engine's namespace automatically | Piotr Sarnacki | 2010-09-03 | 3 | -1/+16 |
| | |||||
* | Rename isolated_engine_for to namespace | Piotr Sarnacki | 2010-09-03 | 2 | -5/+5 |
| | |||||
* | Refactor ActionMailer to not use hide_actions | Piotr Sarnacki | 2010-09-03 | 7 | -69/+40 |
| | |||||
* | Move ActionController::Railties::RoutesHelpers and ↵ | Piotr Sarnacki | 2010-09-03 | 4 | -22/+5 |
| | | | | ActionMailer::Railties::RoutesHelper to AbstractController::Railties::RoutesHelpers | ||||
* | Include routes helpers only for inherited classes in ActionMailer | Piotr Sarnacki | 2010-09-03 | 5 | -27/+67 |
| | |||||
* | Fix generating urls with mounted helpers in view context | Piotr Sarnacki | 2010-09-03 | 3 | -3/+5 |
| | | | | | | | | | | There were actually 2 problems with this one: * script_name was added to options as a string and then it was used in RouteSet#url_for with usage of <<, which was changing the original script_name * the second issue was with _with_routes method. It was called in RoutesProxy to modify _routes in view_context, but url_helpers in views is just delegating it to controller, so another _with_routes call is needed there | ||||
* | Fix copying migrations to empty directory | Piotr Sarnacki | 2010-09-03 | 3 | -1/+15 |
| | |||||
* | Include application's helpers and router helpers by default, but include ↵ | Piotr Sarnacki | 2010-09-03 | 8 | -4/+182 |
| | | | | engine's ones for controllers inside isolated namespace | ||||
* | For view_context we need to initialize RoutesProxy in context of controller, ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -2/+4 |
| | | | | not view, quick fix, I need to dig into it later | ||||
* | mounted helpers should be included in ActionMailer | Piotr Sarnacki | 2010-09-03 | 2 | -1/+3 |
| | |||||
* | This is not needed | Piotr Sarnacki | 2010-09-03 | 1 | -26/+0 |
| | |||||
* | Engine's assets are now served with ActionDispatch::Static | Piotr Sarnacki | 2010-09-03 | 3 | -6/+46 |
| | |||||
* | Modified ActionDispatch::Static to allow passing multiple roots | Piotr Sarnacki | 2010-09-03 | 6 | -21/+104 |
| | |||||
* | Set asset_path to engine_name by default | Piotr Sarnacki | 2010-09-03 | 2 | -14/+31 |
| | |||||
* | Always convert railtie_name to string | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 |
| | |||||
* | rake db:copy_migrations task, which copies railties migrations into ↵ | Piotr Sarnacki | 2010-09-03 | 2 | -0/+71 |
| | | | | application's db/migrate directory | ||||
* | Implemented ActiveRecord::Migrations#copy based on James Adam's idea | Piotr Sarnacki | 2010-09-03 | 11 | -25/+265 |
| | | | | | | | | | | | | | | | ActiveRecord::Migration#copy allows to copy migrations from one place to another, changing migrations versions and adding scope to filename. For example: ActiveRecord::Migration.copy("db/migrate", :blog_engine => "vendor/gems/blog/db/migrate") will copy all migrations from vendor/gems/blog/db/migrate to db/migrate with such format: Versions of copied migrations will be reversioned to be appended after migrations that already exists in db/migrate | ||||
* | Added tests for railtie_name and aliased it in engine as engine_name | Piotr Sarnacki | 2010-09-03 | 2 | -0/+18 |
| | |||||
* | Add Rails::Railtie.railtie_name method to allow setting custom name for railtie | Piotr Sarnacki | 2010-09-03 | 4 | -2/+19 |
| | |||||
* | Added Rails.application.config.paths.db.migrate to remove hardcoded ↵ | Piotr Sarnacki | 2010-09-03 | 3 | -10/+15 |
| | | | | db/migrate paths | ||||
* | to_param shoul return a string | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 |
| | |||||
* | We don't need delegating polymorphic_url and polymorphic_path anymore | Piotr Sarnacki | 2010-09-03 | 1 | -10/+6 |
| | |||||
* | Added ability to set asset_path for engines | Piotr Sarnacki | 2010-09-03 | 9 | -13/+141 |
| | |||||
* | Ensure that Rails.application.initialize! is called only once | Piotr Sarnacki | 2010-09-03 | 2 | -0/+7 |
| | |||||
* | Added documentation on endpoint, middeware stack and routes for Engine | Piotr Sarnacki | 2010-09-03 | 1 | -0/+60 |
| | |||||
* | We don't need that initializer in tests | Piotr Sarnacki | 2010-09-03 | 1 | -4/+0 |
| | |||||
* | Ensure that url_helpers included after application's ones have higher priority | Piotr Sarnacki | 2010-09-03 | 1 | -0/+15 |
| | |||||
* | Add mounted_helpers to routes | Piotr Sarnacki | 2010-09-03 | 9 | -98/+192 |
| | | | | | | | | | | | | | | | | mounted_helpers are a bit similar to url_helpers. They're automatically included in controllers for Rails.application and each of mounted Engines. Mounted helper allows to call url_for and named helpers for given application. Given Blog::Engine mounted as blog_engine, there are 2 helpers defined: app and blog_engine. You can call routes for app and engine using those helpers: app.root_url app.url_for(:controller => "foo") blog_engine.posts_path blog_engine.url_for(@post) | ||||
* | Routes refactoring: | Piotr Sarnacki | 2010-09-03 | 5 | -60/+83 |
| | | | | | | | * added more tests for prefix generation * fixed bug with generating host for both prefix and url * refactored url_for method * organized tests for prefix generation |