Probably worth at least a comment as to why you're reaching into the internals rather than just calling ``module.data`` (you don't want default.) FWIW I added a helper a long time ago which would shorten this to ``module.xblock_kvs._data`` and in my current refactoring branch I have a method which would make this merely ``module.get_set_fields_by_scope(Scope.content)`` which gets any locally set fields for the given scope.