Skip to content

Commit 00f6335

Browse files
committed
[Fix for browsermedia#703] Add cms_form_for
* Previous solution didn't work for any view that wasn't explicitly using content_block_form_for. Added cms_form_for that should work instead (some portlets were using it)
1 parent 1218bcc commit 00f6335

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed
Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
module Cms
22
class ApplicationController < ::ApplicationController
3-
default_form_builder = Cms::FormBuilder::ContentBlockFormBuilder
43
include Cms::AdminController
5-
64
end
75
end

app/helpers/cms/form_tag_helper.rb

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,13 @@ def content_block_form_for(object, *args, &block)
1111
simple_form_for(engine_aware_path(object), *(args << options.merge(builder: Cms::FormBuilder::ContentBlockFormBuilder)), &block)
1212
end
1313

14+
# Simple wrapper for Rails form_for that will use the CMS CustomFormBuilder.
15+
# Can be used by portlets or other random public facing views to render content.
16+
def cms_form_for(*args, &block)
17+
options = args.extract_options!
18+
options.merge!(:builder => Cms::FormBuilder::ContentBlockFormBuilder)
19+
form_for(*(args + [options]), &block)
20+
end
1421

1522
def forecasting_a_new_section?(form_object)
1623
Cms::Section.with_path(form_object.object.class.path).first.nil?

app/views/portlets/email_page/render.html.erb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
<div class="email-page-portlet">
2-
<%= form_for @email_message, :url => cms.portlet_handler_path(@portlet, "deliver"), :method=>'post' do |f| %>
2+
<%= cms_form_for @email_message, :url => cms.portlet_handler_path(@portlet, "deliver"), :method=>'post' do |f| %>
33
<%= hidden_field_tag :email_page_portlet_url, @email_page_portlet_url %>
44
<%= f.cms_error_messages %>
55
<p>

0 commit comments

Comments
 (0)