@@ -209,6 +209,11 @@ module ScaffoldingExtensions content end + # Is the action given an idempotent action? True for show and edit, false otherwise + def scaffold_idempotent_action?(action) + [:show, :edit].include?(action) + end + # Script tag with javascript included inside a CDATA section def scaffold_javascript_tag(javascript) "<script type='text/javascript'>\n//<![CDATA[\n#{javascript}\n//]]>\n</script>" lib/scaffolding_extensions/helper.rb @@ -1,6 +1,6 @@ <% @scaffold_title = "Choose #{hn = @scaffold_options[:singular_lc_human_name]} to #{(haction = (action = @scaffold_action).to_s.humanize).downcase}" %> -<%= scaffold_form(scaffold_url("#{action}#{@scaffold_suffix}"), :method=>(action == :destroy ? :post : :get)) %> +<%= scaffold_form(scaffold_url("#{action}#{@scaffold_suffix}"), :method=>scaffold_idempotent_action?(action) ? :get : :post) %> <% if @scaffold_class.scaffold_use_auto_complete %> <%= scaffold_text_field_tag_with_auto_complete('id', @scaffold_options[:singular_name]) %> <% else %> scaffolds/list.rhtml 692fc9f96fca7026aea19c9826e329b8c832b968 Jeremy Evans code@jeremyevans.net http://github.com/jeremyevans/scaffolding_extensions/commit/314939e4ccbca9147fd1f167c476cd6dcce518eb 314939e4ccbca9147fd1f167c476cd6dcce518eb 2009-05-27T16:42:54-07:00 2009-05-27T16:42:54-07:00 In list scaffold, assume only show and edit actions are idempotent Previously, it assumed that all actions but destroy were idempotent. Now, scaffolding extensions only supports show, edit, and destroy by default, but it's fairly easy to extend to support other actions, and it's safer to assume that those methods won't be idempotent. This adds the scaffold_idempotent_method? helper method which can be overridden if you want to add custom idempotent methods that use get instead of post. 894615097700db4b284dab8bbed312f7774ec245 Jeremy Evans code@jeremyevans.net