Fix completion data generation function

[moonysolari]

The new generateCompletionData function in #30 returns the parent location ID in the SubLocationId field in its return value, which is not correct. It causes 'UI_LOCATION_PARENT_XXX_TEXT' to appear in the mission end screen, among other places. This PR restores the behavior that the sublocation ID should be in that field for most if not all purposes other than mastery. It also prevents location progression data from appearing in the bottom right for the sniper levels and the ICA facility, which is the correct behavior on official servers.

[LennardF1989]

Looks good, but I feel instead of adding exceptions in code, we should just provide proper mastery data in the JSON files. I'll make a draft for it before deciding :)

[LennardF1989]

Alright, here's what I did:

• I added HideProgression as an optional boolean for the Mastery JSON
• I added a Mastery JSON for Facility to set the MaxLevel to 4 and HideProgression to true
• I changed the default CompletionData to be MaxLevel 1 and HideProgression true if a Mastery JSON is not found, since all levels that should actually already have a Mastery JSON anyway.
• I respected the change you made to the masteryService by passing the subLocationId as we would have before the masteryService was a thing
• I added a fix to hide the Mastery-tab in the game if there are no drops associated with the Mastery JSON

The result is exactly as you hard-coded it (eg. levels with no Mastery data should just be MaxLevel 1 AND hidden, eg. the sniper levels), but with a lot more control and less code :)