Opened 9 years ago
Closed 9 years ago
#3044 closed defect (fixed)
[PATCH] Always load map description in game setup screen
Reported by: | Itms | Owned by: | |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 18 |
Component: | UI & Simulation | Keywords: | |
Cc: | Patch: |
Description (last modified by )
When one changes a map description, if the map is remembered in the game setup settings, the description won't be reloaded.
This shouldn't be happening.
Attachments (1)
Change History (7)
comment:1 by , 9 years ago
Description: | modified (diff) |
---|
comment:2 by , 9 years ago
Keywords: | review patch added |
---|---|
Milestone: | Backlog → Alpha 18 |
comment:3 by , 9 years ago
comment:4 by , 9 years ago
I'm going to commit these fixes (along with #3033) but it will be eventually necessary to rewrite the gamesetup script, that is getting bloated and is still functionally limited.
Just a style comment: you have spaces missing lines 737 and 749.
by , 9 years ago
Attachment: | refreshMapData.patch added |
---|
Note: This patch goes beyond refreshing the map data by validating player civ's and AI-assignments as well
comment:5 by , 9 years ago
Summary: | Always load map description in game setup screen → [PATCH] Always load map description in game setup screen |
---|
comment:6 by , 9 years ago
Keywords: | review patch removed |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Fixed in r16346 (ticket number typo in the commit message)
Note:
See TracTickets
for help on using tickets.
As you're already proposing several fixes for the saved settings system, could you take a look at #3033? Thanks in advance :)