Remove an unnecessary save in the file attachment resources.
Review Request #13452 — Created Dec. 7, 2023 and submitted — Latest diff uploaded
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.