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 Itms)

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)

refreshMapData.patch (1.7 KB ) - added by Alex 9 years ago.
Note: This patch goes beyond refreshing the map data by validating player civ's and AI-assignments as well

Download all attachments as: .zip

Change History (7)

comment:1 by Itms, 9 years ago

Description: modified (diff)

comment:2 by Stan, 9 years ago

Keywords: review patch added
Milestone: BacklogAlpha 18

comment:3 by Itms, 9 years ago

As you're already proposing several fixes for the saved settings system, could you take a look at #3033? Thanks in advance :)

comment:4 by Itms, 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 Alex, 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 Stan, 9 years ago

Summary: Always load map description in game setup screen[PATCH] Always load map description in game setup screen

comment:6 by Itms, 9 years ago

Keywords: review patch removed
Resolution: fixed
Status: newclosed

Fixed in r16346 (ticket number typo in the commit message)

Note: See TracTickets for help on using tickets.