I'll post this here.. but I'm not sure if should be here or under image_module. The FID has changed on some of my files, presumably by changes to derivatives. Now previous working images put in place with img_assist no longer work since the FID is no longer correct. I havn't yet verified if it is image_module that changed the FID of the images. I'm going to test and see what changed the FIDs in the DB and I'll post here again.. unless somebody moves this to image_module.

Comments

pegmonkey’s picture

pegmonkey commented

I have verified that changing a derivative can change the FID and render prior created img_assist filter entries on a page useless.

matt westgate’s picture

matt westgate commented

I don't understand how this is a problem with img_assist then.

pegmonkey’s picture

pegmonkey commented

I see it as a problem with both.. so i posted in both places.. if img_assist is going to use FID.. and image_module is going to change FID.. then there's a communication problem.. I don't know who's problem it is.. so I'm simply letting both of you know so a solution can be ironed out.. I don't know what the solution is. I guess the question is should image_module be changing the FIDs? I guess they could be allowed to.. but I can't make that decision... and I have no clue who would. It's more of a planning problem at this point the way I see it. I have a workaround.. and that's to use html for the time being so it's not big deal to me.

solipsist’s picture

solipsist commented

I'd rather img_assist used nids instead of fids and perhaps used a junction table for linking fids to nids (could be updated when cron is run). As it is now, updating an image node will break every img_assist code that referred to the image previously stored for that node, it would be better with a more flexible system.

benshell’s picture

benshell commented
Version: 4.6.x-1.x-dev » 6.x-3.x-dev
Status: Active » Fixed

I am the new maintainer for img_assist, but I am only supporting img_assist 4.7+. However, the 4.7 version does use NIDs instead of FIDs.

benshell’s picture

benshell commented
Status: Fixed » Closed (fixed)