Ticket #441 (closed defect: duplicate)

Opened 11 months ago

Last modified 10 months ago

merb_activerecord: db:test:prepare and others are broken

Reported by: todd.fish..@gmail.com Assigned to:
Priority: major Milestone: The Future
Component: Plugin: ActiveRecord Keywords:
Cc: rogelio@therubymug.com

Description

http://merb.devjavu.com/browser/plugins/merb_activerecord/lib/merb/orms/active_record/tasks/databases.rb#L137

137 config = ActiveRecord::Base.configurations[Merb.environment.to_sym] 138 case config[:adapter] 139 when "mysql", "oci", "oracle" 140 ActiveRecord::Base.establish_connection(config[Merb.environment])

It looks like sqlite, sqlserver, and postgresql would work, but probably should review them too. But, basically from the code snippet above reading config[Merb.environment] is wrong, since before the case statement the environment specific configuration is accessed.

Attachments

tasks.patch (12.2 kB) - added by todd.fish..@gmail.com on 01/11/08 13:45:37.

Change History

01/11/08 11:06:54 changed by rogelio.samo..@gmail.com

  • cc set to rogelio.samo..@gmail.com.

01/11/08 11:58:04 changed by rogel..@therubymug.com

  • cc changed from rogelio.samo..@gmail.com to rogel..@therubymug.com.

01/11/08 13:45:37 changed by todd.fish..@gmail.com

  • attachment tasks.patch added.

01/11/08 19:20:56 changed by iv..@gweezlebur.com

  • milestone changed from 0.5.1 to 0.5.2.

01/14/08 09:39:14 changed by iv..@gweezlebur.com

  • milestone changed from 0.5.2 to 0.5.3.

01/14/08 09:39:59 changed by iv..@gweezlebur.com

  • milestone changed from 0.5.3 to 0.5.2.

01/28/08 13:58:16 changed by iv..@gweezlebur.com

  • milestone changed from 0.5.2 to The Future.

02/11/08 13:25:12 changed by shayarne..@gmail.com

  • status changed from new to closed.
  • resolution set to duplicate.

moved to http://merb.lighthouseapp.com