search

refinery / refinerycms-page-images

Adds an images tab to your pages so you can format a group of images really nicely on the frontend
104 stars 120 forks source link

fields_for error in admin/pages/tabs/_images_field (v1.0) #7

Closed clickworkorange closed 13 years ago

clickworkorange commented 13 years ago

When doing a fresh install of Refinery 0.9.9 and then adding page-images 1.0 I get this error:

refinerycms-page-images-1.0/app/views/admin/pages/tabs/_images_field.html.erb where line #3 raised: undefined local variable or method `f' for #<#Class:0xb89ca00:0xb893ec8>

Extracted source (around line #3):

1: <div class='wym_box field images_field'> 2: <ul id='page_images' class='clearfix'> 3: <%= f.fields_for :images do |imageform| %> 4: <li id='image<%= image_form.object.id %>'> 5: <%= image_form.hidden_field :id %> 6: <%= image_fu image_form.object, '135x135#c' %>

I have tried starting from scratch twice with the same result. page-images 0.9.9 works fine, however this lacks the caption functionality that I am after.

parndt commented 13 years ago

Fix silly mistake, not passing form reference to new partials. This engine needs specs/features. Closed by 57afb4d09890c86c59edea063e2d1ca9d049d807

clickworkorange commented 13 years ago

I can confirm that updating to 1.0.1 solved this issue. Thanks for the quick response!