Page MenuHomeWildfire Games

Move map data to settings.js
Needs ReviewPublic

Authored by nani on Sun, Feb 10, 4:18 AM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

Map type path data and filter should be in settings.js
Needed for D1703: Map browser for gamesetup

Test Plan

Keep same behaviour as before.

Diff Detail

Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

nani created this revision.Sun, Feb 10, 4:18 AM
nani added a parent revision: D1703: Map browser for gamesetup.
nani edited the summary of this revision. (Show Details)Sun, Feb 10, 4:26 AM
nani updated this revision to Diff 7479.Sun, Feb 10, 8:38 PM
elexis added a subscriber: elexis.Sat, Feb 16, 9:32 PM
  • MapFilter: Wondering if this should remain in gamesetup/, since the purpose of settings.js to contain data that is used by multiple GUI pages. But it seems to pass that test, because similarly to the other data in settings.js, this could be used for example in the replay menu or lobby to filter games by maps too (at least if the map exists on the local computer). As mentioned in D1703, it would be better to specify the mapfilters JSON (using MatchClasses syntax) rather than JS. But I have a patch for that somewhere, and it could be considered independent.
  • MapType: Good idea, it never occured to me!