The file attachment resources were unconditionally saving the file

attachment's extra_data, causing a write when one otherwise wouldn't

occur.
This happened because we weren't checking the result of

import_extra_data(), assuming we always had to save the results. We

now only do this if we know a change was made.

Unit tests pass.