Should we be updating both definition.data and metadata with the same data? This doesn't seem to make sense to me, looking at the implementation of update_item and update_metadata in common/lib/xmodule/xmodule/modulestore/mongo.py.