Hello all files people. I am getting somewhere with my files stuff, but i'd like some gurus and/or original file.inc developers to comment on an issue: We have a lot of code in upload.module and some in file.inc to cater previews. In case of a preview, we try to serve the files from the temp folder, or we try to generate temporary URLS etc (since we have no nid yet!) This is a bit crufty IMHO. I want to change this, in my code to behave as follows: * A node (or comment or whatever) that has attachements that are valid are inserted into the files system all the way. we only leave the obid (the column in tghe files dir for object ids like nid) empty * invalid files are deleted immediately * Once the object is inserted and saved, we have an obid (nid, mostly) and we update the table. That way, on loads of previews, we should be able to just use the urls, apis and functions we use normally. So: Why did we choose for the current method of writing all the exceptions for previews? Did I miss something important that renders my idea impossible? Should I take care of something else? Ber --