Joined tables problem

[alexturetsky]

I have two tables - User and Devices:

class User < ActiveRecord::Base
  include ActiveUUID::Model
  attribute :id, :uuid
  has_many :devices
end
class Device < ActiveRecord::Base
  include ActiveUUID::Model
  attribute :id, :uuid
  belongs_to :user
end

uuid is registered as ActiveUUID::Type::BinaryUUID
when I try to create these in DB

user = User.new
user.save!
device = Device.new
device.user = user
device.save!

I get a SQL error:

INSERT INTO `users` (`id`) VALUES (x'f33f3538371c4ba78715819dd972e4bf')
ActiveRecord::ValueTooLong: Mysql2::Error: Data too long for column 'user_id' at row 1: INSERT INTO `devices` (`id`, `user_id`) VALUES (x'2ae9bccb22774b35add0d8673bce282e', x'66333366333533382d333731632d346261372d383731352d383139646439373265346266')

if I decode the hex string x'66333366333533382d333731632d346261372d383731352d383139646439373265346266' I get 'f33f3538-371c-4ba7-8715-819dd972e4bf' - and it looks like id of the user

[ronaldtse]

@alexturetsky thank you for the report!

@skalee @ribose-jeffreylau do you have time to look at this? Thanks!

[alexturetsky]

@ronaldtse thanks a lot for looking into this!

[ribose-jeffreylau]

Hi @alexturetsky
Do you happen to have a minimal example repo that we can test on?
If not, can you try adding the following to the Device class?

  attribute :user_id, :uuid

Your feedback is appreciated!

[alexturetsky]

hi @ribose-jeffreylau ,
adding attribute to the Device class helped:

irb(main):007:0> device.user = user
irb(main):008:0> device.save!
   (0.6ms)  BEGIN
  Device Create (0.6ms)  INSERT INTO `devices` (`id`,  `user_id`) VALUES (x'a8dfab4b187c459bae9da6035bfcaae8', x'd440d8c6c8444e68a8c5ff8b21ce7631')

Do you think I should update all my models with explicit attributes referencing joined tables?

[ribose-jeffreylau]

Hi @alexturetsky ,
Yes, unlike the original activeuuid gem, all columns that have type UUID should be explicitly stated in each model.

[ronaldtse]

Thanks @ribose-jeffreylau , can we add this as a warning on the README?

[ribose-jeffreylau]

@ronaldtse Yes, perhaps under Adding UUIDs to models.

[alexturetsky]

Thank you, all!

[ribose-jeffreylau]

Thanks @alexturetsky !

[alexturetsky]

just an fyi, interesting enough, without explicitly declaring all UUID columns, migrations ran fine (it correctly created all tables with all foreign keys and types)

CREATE TABLE `devices` (
  `id` binary(16) NOT NULL,
  `user_id` binary(16) DEFAULT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;
CREATE TABLE `users` (
  `id` binary(16) NOT NULL,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8;

adding uuid attributes to the model did not break it =)