It's a handy feature, but the following conceptual problems are unresolved:
do new attachments keep track of their source attachment? The converted FK (points to converted file) and is_converted (is True on original) flag are intended to work the other way around. Changing the meaning just for this feature doesn't work well with other checks on converted. Maybe we need to refactor the meaning everywhere?
do we trigger the unpacking automatically? I'm a bit afraid of recursive Zip files are zip bombs. Right now there is an admin action to trigger it manually and task is killed after three minutes.
It's a handy feature, but the following conceptual problems are unresolved:
converted
FK (points to converted file) andis_converted
(isTrue
on original) flag are intended to work the other way around. Changing the meaning just for this feature doesn't work well with other checks onconverted
. Maybe we need to refactor the meaning everywhere?