Hi,
I'm currently using the 6.x-1.x-dev version with Drupal 6.16 and as admin when I delete a user from the Default relationships list the below error pops up.
"recoverable fatal error: Object of class stdClass could not be converted to string in /home/.../sites/all/modules/user_relationships/user_relationship_defaults/user_relationship_defaults.module on line 194."
Tried a complete wipe of all UR tables and reinstall of the module but still happens. If I remember correctly this didn't happen when using 6.x-1.0-rc3
If doesn't seem a serious error as I can still proceed past it but still a little disconcerting to see it.
Comment | File | Size | Author |
---|---|---|---|
#9 | ur-default-friend-delete-9.patch | 913 bytes | Berdir |
#8 | ur-default-friend-delete-8.patch | 1.1 KB | mrf |
#6 | ur-default-friend-delete.patch | 1.04 KB | mrf |
Comments
Comment #1
glitz CreditAttribution: glitz commentedany update on how this was fixed? This is happening to me as well, only when adding a friend (relationship)
Thanks!
Chris
Comment #2
mmachina CreditAttribution: mmachina commentedgetting the same error when deleting a default friend...
Comment #3
mrf CreditAttribution: mrf commentedComment #4
mrf CreditAttribution: mrf commentedI just:
Set up a default user.
Created 50 users who have a relationshiip with this user.
Deleted the default user.
No error messages on relationship or user pages, think this one must have been fixed along the way.
Comment #5
mrf CreditAttribution: mrf commentedAfter going back to the UR admin page I noticed some notices caused by my tests for this, but no fatal error.
Committed a fix to ur defaults hook_user that now deletes the default entry when a user is deleted and should prevent any future issues like the above from occurring.
http://drupalcode.org/project/user_relationships.git/blobdiff/0c49e3f0df...
Comment #6
mrf CreditAttribution: mrf commentedAnd here's a version of this for d7.
Comment #7
BerdirThat condition makes no sense :) fetchAssoc() returns an array then you're using it as an object.
Just do the db_delete(), if there is something to delete, it will delete and if the condition doesn't match anything, nothing happens...
Comment #8
mrf CreditAttribution: mrf commentedAnd an actual tested working version of the patch. Failed mental transition between 6 and 7 :)
Comment #9
BerdirI'm not sure why it needs to be so complicated?
Why not simply do it as I did in my patch?
Comment #10
mrf CreditAttribution: mrf commentedAhhhh the magic of dbtng I stared at that one for a while trying to figure out what you meant by "just do db_delete()" and fell back to my version after wiping out all my defaults :)
Comment #11
BerdirCommited, not sure what the testbot i waiting for.
Giving this one back to you, you might wanna simplify your code...
Comment #12
mrf CreditAttribution: mrf commentedSimplified version committed to 6.x