Possible issue w/ becomes

[kyledecot]

In my migration I'm converting my posts table to use uuids instead of the default integers via something like the following:

...
create_table :posts_temp, id: :uuid do |t|
  t.string :title
  ...
end

post_temp_model = Class.new(ActiveRecord::Base) do
  self.table_name = :posts_temp
end

Post.all.each do |post|
  post_temp = post.dup.becomes(post_temp_model)
  binding.pry
  post_temp.title # => "foobar"
  post_temp.save # => true
  post_temp.reload
  post_temp.title # => nil
end
...

When running this and inspecting what post_temp.title is the value appears to be there. When calling post_temp.reload and inspecting the title again however it's nil. It seems as though the value is not being saved to the database.

Is this a bug or am I doing something incorrectly here?

[isaacsanders]

You are not assigning title.

[kyledecot]

@isaacsanders Sorry, bad code examples. I tried to update my "real" code with the classic Post example. I've updated it to reflect the proper values.

[isaacsanders]

No problem. Helps to know the example is sounds. Thanks!

On Jul 24, 2014, at 7:13 PM, Kyle Decot notifications@github.com wrote:

@isaacsanders Sorry, bad code examples. I tried to update my "real" code with the classic Post example. I've updated it to reflect the proper values.

—
Reply to this email directly or view it on GitHub.

[senny]

@kyledecot it would be easier if you provide your report in the form of an executable test case. You can use this script as a starting point.

[hmaurer]

Are you using postgresql? Here is the test, which currently doesn't pass because afaik SQLite has no built-in support for uuids: https://gist.github.com/a3gis/30bf9b5eee4da8bf3340

[kyledecot]

@A3gis Yep. The UUID part works find (id gets assigned). It seems like the attributes just get thrown away. @senny I'll get an example with that script up to you soon.

[hmaurer]

@kyledecot I gist(ed) an example sript above. The strange thing is that it works fine without the uuid on SQLite3. I'll try it on postgresql.

[bigxiang]

@A3gis I think your gist has a flaw. posts table should have id: :uuid too or the duplicated record couldn't return a correct uuid. It would raise an error when reloading. I can see reload is well on Postgresql after changing it. We probably need to wait for @kyledecot to give an example.

[hmaurer]

@bigxiang In the OP: "[...] I'm converting my posts table to use uuids instead of the default integers [...]", so I assume there was no uuid on the posts table?

[bigxiang]

@A3gis, ah I see what you mean. But I really can't execute the gist without uuid on my machine... , if I add it, all assertions are passed... 😞

[rails-bot]

This issue has been automatically marked as stale because it has not been commented on for at least
three months.

The resources of the Rails team are limited, and so we are asking for your help.

If you can still reproduce this error on the 4-1-stable, 4-0-stable branches or on master,
please reply with all of the information you have about it in order to keep the issue open.

Thank you for all your contributions.

[maclover7]

This passes for me on Rails master. I think the issue is that you are using SQLite for your database -- I don't think SQLite supports UUID. I think this should be closed.

require File.expand_path('../../load_paths', __FILE__)

require 'active_record'
require 'minitest/autorun'
require 'logger'

# This connection will do for database-independent bug reports.
ActiveRecord::Base.establish_connection(adapter: 'postgresql', database: 'testy')
ActiveRecord::Base.logger = Logger.new(STDOUT)

ActiveRecord::Schema.define do
  create_table :posts, id: :uuid do |t|
    t.string :title
  end

  create_table :posts_temp, id: :uuid do |t|
    t.string :title
  end
end

class Post < ActiveRecord::Base
end

class PostTemp < ActiveRecord::Base
  self.table_name = :posts_temp
end

class BugTest < Minitest::Test
  def test_stuff
    post = Post.create(title: 'foobar')
    post.save

    assert_equal post.title, 'foobar'

    post_temp = post.dup.becomes(PostTemp)

    assert_equal post_temp.title, 'foobar'
    assert_equal post_temp.save, true

    post_temp.reload

    assert_equal post_temp.title, 'foobar'
  end
end