I had done it to avoid having lots of repeat hardcoded strings, that way in the future if one of the strings was to be changed it would only have to be changed once rather than every time the key was used.  That being said I can see if we changed 'filename' to something else we probably wouldn't want it referenced by FILENAME, thus making this change unnecessary.  I can revert this back if the hardcoded strings are the better option.