#26 √ resolved
Glenn Powell

[Minor] Should rename template file with .erb EXT

Reported by Glenn Powell | July 30th, 2008 @ 05:07 AM

The file:

/generators/paperclip/templates/paperclip_migration.rb

should be:

/generators/paperclip/templates/paperclip_migration.rb.erb

and then simply referenced and handled in the generator.rb the same as it is now.

This just allows any IDE's (namely Eclipse) to avoid reporting errors within the template file when it encounters ERB tags ( <%= %> )

P.S. - Great plugin

Comments and changes to this ticket

  • Jon Yurek

    Jon Yurek August 13th, 2008 @ 05:50 PM

    • → State changed from “new” to “resolved”

    Quite right. I changed this in master.

  • Alex Tomlins

    Alex Tomlins August 21st, 2008 @ 06:15 AM

    It would seem that this change causes the generator to fail with file not found. Output:

    exists db/migrate create db/migrate/20080821101113_add_attachments_photo_to_content.rb No such file or directory - RAILS_ROOT/vendor/plugins/paperclip/generators/paperclip/templates/paperclip_migration.rb

    any ideas?

    renaming the file back makes it work again.

    Rails 2.1.0 ruby 1.8.6 p114

    thanks, Alex

  • Glenn Powell

    Glenn Powell August 21st, 2008 @ 06:52 AM

    The generators/paperclip/paperclip_generator.rb file also needs to be changes. Not sure if this was done in the master.

    
    m.migration_template "paperclip_migration.rb"
    

    must be

    
    m.migration_template "paperclip_migration.rb.erb"
    

Please Login or create a free account to add a new comment.

You can update this ticket by sending an email to from your email client. (help)

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

People watching this ticket