Absolutely. In this pass I just updated the name of the variable (it was called get, but it comes from request.POST...), but didn't add anything to the docstrings. I will do it in a second pass.

The whole pipeline of how this "data" objects moves through xmodule and capa needs to be reworked. I opened an https://edx-wiki.atlassian.net/browse/LMS-486. The idea is to make the refactoring/improvements for the pipeline as the same time as we provide a consistent way to store student file submissions.