Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Schema dumper #22

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Schema dumper #22

wants to merge 3 commits into from

Conversation

agios
Copy link

@agios agios commented Dec 28, 2012

It will add non-integer primary key entries to the schema dump

@pyromaniac
Copy link
Collaborator

It is foken osom! I could not even bring myself to do this patch. It's sad that AR dumper source is so old and unpatchable (

@paneq
Copy link

paneq commented Dec 31, 2012

It doesn't work for me.

Rails 3.2.9 , Postgresql 8.4

db/schema.rb:

ActiveRecord::Schema.define(:version => 20121224130418) do
  create_table "objects", :id => false, :force => true do |t|
    t.string   "uuid",        :limit => nil,                                :null => false
  end
end

As you can see it has t.string instead of t.uuid

@agios
Copy link
Author

agios commented Jan 4, 2013

That's odd, I currently use it on 3.2.9 myself. What gems do you have installed? Maybe another also overrides schema dumper?

@juni0r
Copy link

juni0r commented Jan 8, 2013

It didn't work for me either. The uuid columns are mistaken as string columns and it requires a peek at the sql column type to figure it out.

I've got a modified version here.

schema_dumper.rb

Let me know if it works (@paneq)

mcosand added a commit to mcosand/activeuuid that referenced this pull request Mar 5, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants