Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#1654 closed enhancement (fixed)

[PATCH] Adjust random map "Fortress" now that gates work

Reported by: FeXoR Owned by: O.Davoodi
Priority: Should Have Milestone: Alpha 12
Component: UI & Simulation Keywords:
Cc: O.Davoodi Patch:

Description

With working gates there is no need to have an open entry and gates in the walls surrounding the players Civil Centres. Changes:

  • Replaced the entries with gates (Now that they work)
  • Replaced the gates with long walls (Since it was just a style thing)
  • Replaced walls with long walls (To fit the ingame wall placement defaults)
  • Added towers between gates and walls (To fit the ingame wall placement defaults)
  • Removed two wall segments per sede (To readjust fortress size)

NOTE: Since it's Spahbod's map he should confirm that this changes are wanted.

Attachments (2)

2012-9-5_adjust_fortress.zip (3.5 KB ) - added by FeXoR 12 years ago.
ZIP file containing the changed file fortress.js
fortress.js.patch (1.8 KB ) - added by FeXoR 12 years ago.
SVN patch of the changes

Download all attachments as: .zip

Change History (6)

by FeXoR, 12 years ago

ZIP file containing the changed file fortress.js

by FeXoR, 12 years ago

Attachment: fortress.js.patch added

SVN patch of the changes

comment:1 by fabio, 12 years ago

Note: current approach is still useful for a testcase of #1566.

comment:2 by leper, 12 years ago

Keywords: patch added; fortress RMS RMGEN removed
Milestone: BacklogAlpha 12

I sort of suggested this fix and I'd say it should be committed as soon as Alpha 11 is released.

comment:3 by O.Davoodi, 12 years ago

Owner: set to O.Davoodi
Resolution: fixed
Status: newclosed

In 12640:

Committing FeXoR's new fortress arrangement for Fortress random map. Fixes #1654.

comment:4 by O.Davoodi, 12 years ago

Keywords: patch review removed
Note: See TracTickets for help on using tickets.