Opened 8 years ago

Closed 8 years ago

#1830 closed enhancement (fixed)

Extension point for external files should include a DbControl also for unsaved new file items

Reported by: Nicklas Nordborg Owned by: everyone
Priority: minor Milestone: BASE 3.3
Component: core Version:
Keywords: Cc:

Description

Consider the following similar code snippets:

File f = File.getNew(dc, directory);
f.setUrl(url, true);
f.saveItem(dc);

and

File f = File.getNew(dc, directory);
f.saveItem(dc);
f.setUrl(url, true);

and

File f = File.getById(dc, fileId);
f.setUrl(url, true);

The only difference is that the File.setUrl() method is called before or after saving the file. If there is a registered extension for the given url, the extension system will pass on the DbControl? in the second and third cases but not in the first. I think the DbControl? should always be passed on.

Change History (1)

comment:1 Changed 8 years ago by Nicklas Nordborg

Resolution: fixed
Status: newclosed

(In [6513]) Fixes #1830: Extension point for external files should include a DbControl? also for unsaved new file items

Note: See TracTickets for help on using tickets.