There was no reason it should be nested inside.
This is (also) to address the review from bdach about StorageManager initialising a default value that gets overwritten upon migration anyway.
Also changes the class accessor from internal to public