| # |
Apr 26th 2017, 10:06 |
 birdy247 |
Neon1024 I feel like I am nearly there |
| # |
Apr 26th 2017, 09:45 |
dereuromark |
Is is sth that should be done by the framework - using the schema to detect non objects and transforming them into objects? |
| # |
Apr 26th 2017, 09:44 |
dereuromark |
I think the patching is the issue, I would expect it to convert date strings into Time() objects |
| # |
Apr 26th 2017, 09:33 |
dereuromark |
that looks like a valid enough datetime string that I would think our ORM/saving should allow this to go through without exceptions |
| # |
Apr 26th 2017, 09:33 |
dereuromark |
Guys, is there an easy fix to the MYSQL http://stackoverflow.com/questions/23581687/mysql-error-invalid-datetime-format-1292-incorrect-datetime-value-for-col issue? I get "SQLSTATE[22007]: Invalid datetime format: 1292 Incorrect datetime value: '2017-04-05T17:18:00+00:00' for column 'created' at row 1" exception when importing JSON export from the same table on a different database. |
| # |
Apr 26th 2017, 09:29 |
birdy247 |
ill give it a whirl |
| # |
Apr 26th 2017, 09:23 |
Neon1024 |
At least, that’s what I’d try ;) |
| # |
Apr 26th 2017, 09:23 |
Neon1024 |
So that you can let the framework do the associations for you :) |
| # |
Apr 26th 2017, 09:23 |
Neon1024 |
Yes, I just meant that you can do $user->set(‘partipations, $arrayOfRaceEntities); |
| # |
Apr 26th 2017, 09:17 |
birdy247 |
and when done, mark user 2 as "merged and removed" |
| # |
Apr 26th 2017, 09:16 |
birdy247 |
I gues the simplest way is to update all the associated records of user2, with an id of user_id = 1 |
| # |
Apr 26th 2017, 09:16 |
birdy247 |
in participants |
| # |
Apr 26th 2017, 09:16 |
birdy247 |
It has lots of other info |
| # |
Apr 26th 2017, 09:16 |
birdy247 |
So I cant just add the races to the new user |
| # |
Apr 26th 2017, 09:15 |
birdy247 |
Races hasMany Participants |
| # |
Apr 26th 2017, 09:15 |
birdy247 |
Neon1024 User hasMany Participants |
| # |
Apr 26th 2017, 09:14 |
Neon1024 |
What would I do with a valid login on an ‘archived’ user account? |
| # |
Apr 26th 2017, 09:13 |
Neon1024 |
I’d also then, after that, think about logins if there are two users are they trying to login with both accounts |
| # |
Apr 26th 2017, 09:13 |
Neon1024 |
Then I could feed back to users easily about which races have been ‘automerged’ into their account |
| # |
Apr 26th 2017, 09:13 |
Neon1024 |
As I’d have a status_id field, I’d just create a new status with that in |
| # |
Apr 26th 2017, 09:12 |
Neon1024 |
Also probably mark new races with a status of ‘merged’ or something, so I can tell which stuff changed |
| # |
Apr 26th 2017, 09:12 |
Neon1024 |
Probably with a cascade to mark all the old users data as inactive |
| # |
Apr 26th 2017, 09:12 |
Neon1024 |
Then I’d add archiving the old user |
| # |
Apr 26th 2017, 09:11 |
Neon1024 |
Then I’d recfactor it to allow more than one user to be merged |
| # |
Apr 26th 2017, 09:11 |
Neon1024 |
I’d write a shell to find a user, and then another user, and just add the races from user 2 into user 1 and re-save user 1 |
| # |
Apr 26th 2017, 09:03 |
birdy247 |
I want to merge 2 INTO 1, so user 1 now has 1,2,3,5,6 races |
| # |
Apr 26th 2017, 09:03 |
birdy247 |
in this case, user 1 and 2 are actually the same person, they registered twice with 2 different emails (doh) and user 1 entered races 1,2,3 and user 2 entered races 5,6 |
| # |
Apr 26th 2017, 09:02 |
birdy247 |
a User hasMany Participants |
| # |
Apr 26th 2017, 09:02 |
birdy247 |
Ok, bit more context |
| # |
Apr 26th 2017, 09:01 |
Neon1024 |
Will it |
| # |
Apr 26th 2017, 09:00 |
birdy247 |
the merger class will pass back what needs to be saved? |
| # |
Apr 26th 2017, 09:00 |
birdy247 |
so pass it some data, |
| # |
Apr 26th 2017, 09:00 |
birdy247 |
right |
| # |
Apr 26th 2017, 08:59 |
Neon1024 |
Give things their dependancies, dont’ have them reach out for them |
| # |
Apr 26th 2017, 08:59 |
birdy247 |
i.e. updating FK's |
| # |
Apr 26th 2017, 08:59 |
birdy247 |
to give context, the class will be "merging" duplicate user accounts |
| # |
Apr 26th 2017, 08:59 |
Neon1024 |
Erm, accidental caps |
| # |
Apr 26th 2017, 08:59 |
Neon1024 |
YES |
| # |
Apr 26th 2017, 08:59 |
birdy247 |
Should I be passing the entities to the class instead |
| # |
Apr 26th 2017, 08:59 |
Neon1024 |
It is |
| # |
Apr 26th 2017, 08:58 |
birdy247 |
It seems "wrong" that the class will be retrieving entities |