use d6;
> select * from image where nid = 170;
+-----+-----+------------+
| nid | fid | image_size |
+-----+-----+------------+
| 170 | 720 | preview    |
| 170 | 719 | thumbnail  |
| 170 | 211 | _original  |
+-----+-----+------------+

Applies field conversion :

use d7;
> select * from image where nid = 170;
+-----+------+------------+
| nid | fid  | image_size |
+-----+------+------------+
| 170 | 3197 | preview    |
| 170 | 3197 | thumbnail  |
| 170 | 3197 | _original  |
+-----+------+------------+

The fid update applies to every file what appears to be wrong.
That's happen in image_legacy.field_convert.inc image_legacy_field_convert_object_pre_load()

CommentFileSizeAuthor
#1 1768432-nid-dont-update-all.patch438 bytesdrzraf

Comments

drzraf’s picture

drzraf commented
StatusFileSize
new 1768432-nid-dont-update-all.patch438 bytes
drzraf’s picture

drzraf commented

Adding an issue :
in its current shape the module does not do the update such that it can be run only once.
It never delete {image} but the test to run the field conversion is done on the existence of this table.

drzraf’s picture

drzraf commented
Priority: Minor » Normal

ping ?

joachim’s picture

joachim commented
Status: Active » Fixed

Committed, thanks.

> in its current shape the module does not do the update such that it can be run only once.

There was probably a good reason for that, but I don't remember.

Status: Fixed » Closed (fixed)

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

Anonymous’s picture

(not verified) commented
Issue summary: View changes

precision