Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #8032 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

[PATCH] Allow layouts with extension of .html.erb

Reported by: joshknowles Assigned to: core
Priority: normal Milestone: 1.x
Component: ActionPack Version: edge
Severity: normal Keywords:
Cc:

Description

The current regex for selecting a layout to be used expects that a file is named with a single extension. This patch allows for multiple extensions, such as .html.erb.

I was unable to understand the tests being used for layouts, thus I am submitting without for the time being while I research.

Attachments

layout_extension.diff (0.7 kB) - added by joshknowles on 04/09/07 05:26:47.
layouts_with_multiple_extensions_including_tests.diff (2.0 kB) - added by joshknowles on 04/09/07 05:40:30.

Change History

04/09/07 05:26:47 changed by joshknowles

  • attachment layout_extension.diff added.

04/09/07 05:28:41 changed by joshknowles

  • summary changed from [Patch] Allow layouts with extension of .html.erb to [PATCH] Allow layouts with extension of .html.erb.

04/09/07 05:40:30 changed by joshknowles

  • attachment layouts_with_multiple_extensions_including_tests.diff added.

04/09/07 05:42:40 changed by joshknowles

The latest attachment includes unit tests which will fail prior to patch being applied

04/30/07 20:31:57 changed by josh

  • status changed from new to closed.
  • resolution set to fixed.

Fixed [6516].