darcs patch: add_field_names_to_set_assoc_in_mysqlstore

Jonathan Daugherty cygnus at janrain.com
Fri Feb 29 10:44:31 PST 2008


>  But the good news is, the very small, very non-intrusive patch below
>  fixes the problem. Because it specifies the fields it's inserting data
>  to, it doesn't matter if the database table has extra fields, as long
>  as it has at least these. (And then you can create an index on the
>  former compound primary key.)

Hi Wes,

I'm more than happy to accept this.  It's always a good practice to
list the columns in an insert anyway.  I'm sorry I haven't had more
time lately to be responsive.  I'm wondering a little about Apple
Mail's concept of an attachment at the moment.  Can you send it as an
attachment?  (The patch is currently appearing inline, which makes it
slightly error-prone to extract and apply.)

Thanks!

-- 
  Jonathan Daugherty



More information about the Dev mailing list