Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | Fixed about half of broken tests in routing_test | Łukasz Strzałkowski | 2010-09-05 | 1 | -125/+82 | |
| | | ||||||
| * | Fixed almost all resources tests | Piotr Sarnacki | 2010-09-05 | 1 | -159/+213 | |
| | | ||||||
| * | Fixed url_for test | Łukasz Strzałkowski | 2010-09-05 | 1 | -1/+1 | |
| | | | | | | | | Added route which matches all: controller, action, id and format | |||||
| * | Revert "Extended default route for match in tests - now it matches ↵ | Łukasz Strzałkowski | 2010-09-05 | 1 | -2/+2 | |
| | | | | | | | | | | | | controller, action, id and format if given." This reverts commit 05e9d0df0dea68ca4655aa8723c3ee69049fac78. | |||||
| * | Extended default route for match in tests - now it matches controller, ↵ | Łukasz Strzałkowski | 2010-09-05 | 1 | -2/+2 | |
| | | | | | | | | | | | | action, id and format if given. This fixes url_for tests | |||||
| * | Removed deprecated RouteSet API, still many tests fail | Piotr Sarnacki | 2010-09-05 | 31 | -590/+563 | |
| | | ||||||
* | | Make number_to_human and number_with_precision work with negatives | Santiago Pastorino | 2010-09-05 | 1 | -0/+2 | |
| | | ||||||
* | | Fix indentation errors | Santiago Pastorino | 2010-09-05 | 1 | -47/+47 | |
| | | ||||||
* | | Fix number_to_human(0) exception [#5532 state:resolved] | Ben Sharpe | 2010-09-05 | 1 | -0/+1 | |
|/ | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | Implemented RouteSet#default_scope, which allows to set the scope for the ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -0/+29 | |
| | | | | entire routes object | |||||
* | Remove namespace for isolated namespaced models in forms | Piotr Sarnacki | 2010-09-03 | 2 | -0/+32 | |
| | ||||||
* | Use new ActiveModel::Naming.route_key in polymorphic_routes | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 | |
| | ||||||
* | Add possibility to explicitly call engine's routes through ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -0/+8 | |
| | | | | polymorphic_routes, for example: polymorphic_url([blog, @post]) | |||||
* | Added more tests for polymorphic_url with namespaced models and implemented ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -0/+56 | |
| | | | | missing use cases | |||||
* | Modified ActionDispatch::Static to allow passing multiple roots | Piotr Sarnacki | 2010-09-03 | 5 | -11/+54 | |
| | ||||||
* | to_param shoul return a string | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 | |
| | ||||||
* | Added ability to set asset_path for engines | Piotr Sarnacki | 2010-09-03 | 1 | -0/+23 | |
| | ||||||
* | 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 | 2 | -66/+70 | |
| | | | | | | | | | | | | | | | | 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 | 1 | -46/+71 | |
| | | | | | | | * 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 | |||||
* | Use new url_for API instead of including routes.url_helpers | Piotr Sarnacki | 2010-09-03 | 1 | -3/+2 | |
| | ||||||
* | Extended url_for to handle specifying which router should be used. | Piotr Sarnacki | 2010-09-03 | 4 | -51/+94 | |
| | | | | | | | A few examples: url_for Blog::Engine, :posts_path url_for Blog::Engine, @post url_for Blog::Engine, :action => "main", :controller => "index" | |||||
* | Added some more tests for url generation between Engine and Application | Piotr Sarnacki | 2010-09-03 | 1 | -10/+28 | |
| | ||||||
* | Get rid of :skip_prefix options in routes | Piotr Sarnacki | 2010-09-03 | 1 | -1/+1 | |
| | ||||||
* | Refactored tests for prefix generation and added test for url generation in ↵ | Piotr Sarnacki | 2010-09-03 | 1 | -11/+32 | |
| | | | | regular class with default_url_options[:script_name] set | |||||
* | New way of generating urls for Application from Engine. | Piotr Sarnacki | 2010-09-03 | 2 | -1/+2 | |
| | | | | | | | | | It's based specifying application's script_name with: Rails.application.default_url_options = {:script_name => "/foo"} default_url_options method is delegated to routes. If router used to generate url differs from the router passed via env it always overwrites :script_name with this value. | |||||
* | Allow to generate Application routes inside Engine | Piotr Sarnacki | 2010-09-03 | 1 | -3/+22 | |
| | | | | | | This requires knowledge about original SCRIPT_NAME and the parent router. It should be pass through the env as ORIGIAL_SCRIPT_NAME and action_dispatch.parent_routes | |||||
* | Use env['action_dispatch.routes'] to determine if we should generate prefix ↵ | Piotr Sarnacki | 2010-09-03 | 2 | -1/+103 | |
| | | | | | | | | | | or not. This technique is here to allow using routes from Engine in Application and vice versa. When using Engine routes inside Application it should generate prefix based on mount point. When using Engine routes inside Engine it should use env['SCRIPT_NAME']. In any other case it should generate prefix as env should not be even available. | |||||
* | Revert "Setup explicit requires for files with exceptions. Removed them from ↵ | José Valim | 2010-09-02 | 7 | -7/+0 | |
| | | | | | | | | autoloading." Booting a new Rails application does not work after this commit [#5359 state:open] This reverts commit 38a421b34d0b414564e919f67d339fac067a56e6. | |||||
* | Setup explicit requires for files with exceptions. Removed them from ↵ | Łukasz Strzałkowski | 2010-09-02 | 7 | -0/+7 | |
| | | | | | | autoloading. Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Removing unnecessary code from render_test | Thiago Pradi | 2010-09-02 | 1 | -2/+0 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Removing unnecessary code | Thiago Pradi | 2010-09-02 | 1 | -7/+0 | |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Move implicit nested call before options handling so that nested constraints ↵ | Andrew White | 2010-09-01 | 1 | -0/+18 | |
| | | | | | | work [#5513 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Added :format option for form_for helper and spec for this [#5226 ↵ | Alex Chrome | 2010-09-01 | 1 | -2/+18 | |
| | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Bump rack-mount to 0.6.13 and add test case for named character classes ↵ | Andrew White | 2010-08-31 | 1 | -0/+12 | |
| | | | | | | [#5509 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | |||||
* | Remove NonConcattingString. | José Valim | 2010-08-29 | 1 | -2/+0 | |
| | ||||||
* | Remove deprecated support to <% form_for %> and several ↵ | José Valim | 2010-08-29 | 5 | -120/+12 | |
| | | | | ActionController::Base methods. | |||||
* | Ensure that inherited helper_methods are available after calling ↵ | Jesse Storimer | 2010-08-28 | 1 | -0/+31 | |
| | | | | | | clear_helpers [#5348 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | password_field renders with nil value by default | Santiago Pastorino | 2010-08-28 | 1 | -1/+4 | |
| | | | | | | | | This makes the use of passwords secure by default if you want to render the value of the password_field you have to do for instance f.password_field(:password, :value => @user.password) # => <input type=password id=user_password name=user[password] value=#{@user.password} /> | |||||
* | Make InstanceTagMethods#value_before_type_cast raise if the model don't ↵ | Santiago Pastorino | 2010-08-27 | 1 | -0/+13 | |
| | | | | | | respond to attr_before_type_cast or attr method [#3374] [#5471 state:committed] | |||||
* | Be sure to call helper :all just on direct children. (Tests by Jesse Storimer) | José Valim | 2010-08-26 | 3 | -0/+18 | |
| | ||||||
* | Add clear_helpers as a way to clean up all helpers added to this controller, ↵ | José Valim | 2010-08-26 | 2 | -2/+24 | |
| | | | | maintaing just the helper with the same name as the controller. | |||||
* | Use nested scope for routes defined at the :resources scope level (as in ↵ | Andrew White | 2010-08-24 | 1 | -0/+7 | |
| | | | | | | Rails 2.3) Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Allow format to be skipped. This is used internally by mount. | José Valim | 2010-08-24 | 1 | -1/+1 | |
| | ||||||
* | Finally fix the bug where symbols and strings were not having the same ↵ | José Valim | 2010-08-24 | 1 | -6/+38 | |
| | | | | | | | behavior in the router. If you were using symbols before for methods like match/get/post/put/delete, it is likely that this commit will break your routes. Everything should behave the same if you are using strings, if not, please open up a ticket. | |||||
* | Ensure shortcuts inside resources also generates helpers. | José Valim | 2010-08-24 | 1 | -4/+4 | |
| | ||||||
* | Fix how routes inside namespaces are generated. | José Valim | 2010-08-24 | 1 | -3/+3 | |
| | ||||||
* | Allow actions which start with A-Za-z_ only | Santiago Pastorino | 2010-08-24 | 1 | -1/+4 | |
| | ||||||
* | Reset symbolized path parameters when a test request is recycled [#5437 ↵ | Andrew White | 2010-08-24 | 1 | -0/+8 | |
| | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com> | |||||
* | Fix a bug where symbols and strings were not behaving the same in the ↵ | José Valim | 2010-08-24 | 1 | -3/+3 | |
| | | | | router. [#5431 state:resolved] |