Normally when the thumbnail field is updated the provider makes sure that the id is synced back to the base_id in the atom. This is done by implementing the update_atom hook. However on creation the system only calls the register_atom hook. So the image provider should implement both hooks and not just only the update, otherwise it is possible to break the atom at creation if the thumbnail image is replaced on the options form.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nagy.balint’s picture

Status: Active » Needs review
FileSize
1.04 KB

Patch attached.

jcisio’s picture

Status: Needs review » Fixed

Thanks. I've committed a modified version http://drupalcode.org/project/scald.git/commit/abfde0e. It's a bit longer but it keeps the private function sane because the logic of $mode should not be there.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.