Errors found when doing upgrade from 1.7

Support Ticket

+status
+tags
 

Hello Ethan,

long time no see.  Life is sometimes a bi*beep*, sorry I wasn't here for long time.  I'm now upgrading my prehistoric version 1.7 of wagn and found some hitches, also found the solutions.  Did not have time to post it via github so at least here.

 

Found actually two issues:

 

1) error in 1.10.0 and even the newest when performing bundle exec rake RAILS_ENV=production wagn:install
 Remove the , at the end of the hash -> wagn:install does not run otherwise
/home/vedma/web/wagn/lib/util/card_builder.rb:39: syntax error, unexpected ')'

 

2) Error in version 1.10.0 (migration from 1.9.x) when running rake RAILS_ENV=production db:migrate

undefined method `merge'
/home/vedma/web/migrate_wagn/wagn-1.10.0/lib/wagn/model/templating.rb:76:in `expire_templatee_references'

Take care,

 

Patrik

To fix:

 

ad 1)

On line 39 edit:

:blocked => true,  -> the hash should be end like :blocked => true

 

ad 2)

# missing prenteses (in version 1.10.0) when doing rake RAILS_ENV=production db:migrate
old "incorrect" code:
Wql.new (wql == true ? {:name => name} :  wql).merge(:return => :id)
correct code:
Wql.new ((wql == true ? {:name => name} :  wql).merge(:return => :id))

(not possible to upgrade from 1.9.x to 1.10.2 due to
Your database is not ready to be migrated to 1.10.2.
Please first install version 1.10.0 and run `rake db:migrate`.

Sorry about this! We're working to minimize these hassles in the future

 

This is super helpful, thanks!

 

I'm pushing out v1.10.3 now, which addresses both of the above (in the card_builder case I actually went a bit further and got rid of that file altogether).

 

I've also retagged v1.10.0 and included the patch you mention.  Otherwise folks won't be able to migrate up from old versions.