user_save() throws a PDOException when using Image profile fields [#1118074]

Trying to call user_save() won't work on my installation. Guessing why.

    $account = user_load( 10 );
    $edit = array( 'mail' => "exampla@example.com" );

This will cause this error:

PDOException: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '' for key 'uri': INSERT INTO {file_managed} (filesize, status, timestamp) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2); Array ( [:db_insert_placeholder_0] => 0 [:db_insert_placeholder_1] => 1 [:db_insert_placeholder_2] => 1302092562 ) in drupal_write_record() (line 6776 of /var/www/citius/includes/common.inc).

Any clues on how can I debug this? Thank you.

Comments

Comment #1

jorge.suarez CreditAttribution: jorge.suarez commented 6 April 2011 at 14:24

Sorry, that code is wrong. I forgot the user_save() call.

    $account = user_load( 10 );
    $edit = array( 'mail' => "exampla@example.com" );
    user_save($account, $edit);

Comment #2

jorge.suarez CreditAttribution: jorge.suarez commented 6 April 2011 at 16:00

As I suspected, this has something to do with profile fields. After receiving this error, all the profile fields of type Image are gone forever.

I'm reproducing now this error. Please take a look at this demo: http://www.panicoenelnucleo.com/drupaltest/drupal-7.0/

- Clean 7.0 install standard.
- Enabled PHP Filter.
- Disabled user picture. Added image profile field.
- Created static page http://www.panicoenelnucleo.com/drupaltest/drupal-7.0/node/1
- The content of this page:

    global $user;
    $account = user_load( $user->uid );
    $edit = array( 'mail' => "exampla@example.com" );
    user_save($account, $edit);

So, to reproduce this bug:

- Login as testuser1, password testpassword. You can also login as testuser2, testuser3 and testuser4.
- Visit http://www.panicoenelnucleo.com/drupaltest/drupal-7.0/node/1 (notice that e-mail is changed, no error is thrown)
- Add a picture in your profile.
- Visit again http://www.panicoenelnucleo.com/drupaltest/drupal-7.0/node/1
- Everything seems to be OK. Reload. Then you get these warnings:

    Warning: array_flip() [function.array-flip]: Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->load() (line 178 of /home/panico7/panicoenelnucleo.com/drupaltest/drupal-7.0/includes/entity.inc).
    Notice: Trying to get property of non-object in file_field_presave() (line 221 of /home/panico7/panicoenelnucleo.com/drupaltest/drupal-7.0/modules/file/file.field.inc).
    Notice: Undefined property: stdClass::$uri in file_save() (line 575 of /home/panico7/panicoenelnucleo.com/drupaltest/drupal-7.0/includes/file.inc).

- Reload again. Then you get the warnings again, then the fatal error:

    Warning: array_flip() [function.array-flip]: Can only flip STRING and INTEGER values! in DrupalDefaultEntityController->load() (line 178 of /home/panico7/panicoenelnucleo.com/drupaltest/drupal-7.0/includes/entity.inc).
    Notice: Trying to get property of non-object in file_field_presave() (line 221 of /home/panico7/panicoenelnucleo.com/drupaltest/drupal-7.0/modules/file/file.field.inc).
    Notice: Undefined property: stdClass::$uri in file_save() (line 575 of /home/panico7/panicoenelnucleo.com/drupaltest/drupal-7.0/includes/file.inc).
    PDOException: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry '' for key 'uri': INSERT INTO {file_managed} (filesize, status, timestamp) VALUES (:db_insert_placeholder_0, :db_insert_placeholder_1, :db_insert_placeholder_2); Array ( [:db_insert_placeholder_0] => 0 [:db_insert_placeholder_1] => 1 [:db_insert_placeholder_2] => 1302105551 ) in drupal_write_record() (line 6776 of /home/panico7/panicoenelnucleo.com/drupaltest/drupal-7.0/includes/common.inc).

Comment #3

jorge.suarez CreditAttribution: jorge.suarez commented 6 April 2011 at 16:27

Title:	user_save() throws a PDOException	» user_save() throws a PDOException when using Image profile fields
Issue tags:		+profile, +cck fields

Changed title

Comment #4

jorge.suarez CreditAttribution: jorge.suarez commented 28 April 2011 at 11:13

Continued testing with this demo using testuser4 and vardumped some values. Definitely, user_save() is messing up with the Image fields. Can someone else more experienced test this?

Comment #5

neutecake CreditAttribution: neutecake commented 25 May 2011 at 06:36

Same problem.

After calling the user_save my filefield data is been deleted.

Comment #6

neutecake CreditAttribution: neutecake commented 25 May 2011 at 08:01

I've found a solution:

$existing_user = user_load($userid);

$edited_user = (array) $existing_user;

$edited_user['field_match_type'] = array(LANGUAGE_NONE => array(array('value' => $form_state['values']['match_type'])));

$user = user_save($existing_user, $edited_user);

Comment #7

jorge.suarez CreditAttribution: jorge.suarez commented 26 May 2011 at 07:58

Can you explain your workaround for this problem? What's the issue exactly? Maybe we can fix this on Drupal Core instead of workaround the problem.

Thank you,

Comment #8

neutecake CreditAttribution: neutecake commented 26 May 2011 at 11:29

I think it isn't a bug.

The user_save function updates all fields from the user, if the field isn't in the arrah of the second argument ($edit) the field wil be emptied.

Priority:	Normal	» Minor
Issue summary:	View changes

Comment #13

13 November 2018 at 16:38

Version:

7.0-rc4

» 7.x-dev

Core issues are now filed against the dev versions where changes will be made. Document the specific release you are using in your issue comment. More information about choosing a version.