#5576 [PATCH] A few fixes for mountable engines - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#5576 ✓resolved

[PATCH] A few fixes for mountable engines

Reported by Piotr Sarnacki | September 7th, 2010 @ 11:05 PM

I have a few patches for mountable engines:
1. Refactoring of mounted_helpers
2. Changed app to main_app to not conflict with 'app' method in tests and probably in some other places
3. Fix warning in railties tests
4. asset_path should be set by default only for engine, otherwise assets will have weird paths like "/foo_application/bar_engine" by default

0001-do-not-require-passing-app-to-mounted-helpers-it-s-a.patch 2.9 KB
0002-change-app-to-main_app-in-mounted_helpers.patch 3.7 KB
0003-fix-warning-by-removing-map-from-routesdraw.patch 881 Bytes
0004-set-default-asset_path-only-for-engine.patch 830 Bytes

Comments and changes to this ticket

Piotr Sarnacki September 13th, 2010 @ 11:18 AM
It was already merged from my github fork, this ticket can be closed now.
Rohit Arondekar September 14th, 2010 @ 04:43 AM
- State changed from “new” to “resolved”
- Importance changed from “” to “Low”