Ticket #1288 (closed enhancement: duplicate)

Opened 14 months ago

Last modified 14 months ago

[PATCH] wall_builder.js update

Reported by: FeXoR Owned by:
Priority: Should Have Milestone:
Component: Atlas editor Keywords: delayed
Cc:

Description

A new version of the wall_builder.js random map script library. Tested in svn version 11426. Compatible to random map script 'fortress'.

Changes:

  • All classes/functions take more arguments (to enable 1 line placements)
  • Most arguments are optional now.
  • Wall style definition is MUCH shorter now (about 500 lines less!).
  • Linear wall placement added (More or less the code needed to add walls in game)
  • Circular wall placement added including random offset for organic looking fortresses (though not sufficient IMO)

As soon as it is in the repository I would like to add documentation to the rmgen wiki if that's ok.

Note: Non 'component' seam to fit to this ticket.

Attachments

wall_builder.js (28.9 KB) - added by FeXoR 14 months ago.

Change History

Changed 14 months ago by FeXoR

comment:1 Changed 14 months ago by FeXoR

I forgot to tell it fixes in issue on the random map 'fortress' that causes celtic huts to overlap. Due to the celtic wall's thickness they now overlap with the inner hill of the wall (it looks ok) but can clearly be noticed as 2 huts.

comment:2 Changed 14 months ago by FeXoR

  • Keywords delayed added; Review removed

NOTE: Some parts of the wall tool are not consistent with unit placement after the rmgen placement fix so don't add this yet...

Last edited 14 months ago by FeXoR (previous) (diff)

comment:3 follow-up: ↓ 4 Changed 14 months ago by FeXoR

This ticket is to be replaced by Ticket #1311. Sorry for the 'double ticket'. Thx, Leper, for pointing this out!

Last edited 14 months ago by FeXoR (previous) (diff)

comment:4 in reply to: ↑ 3 Changed 14 months ago by vts

  • Status changed from new to closed
  • Resolution set to duplicate

Replying to FeXoR:

This ticket is to be replaced by Ticket #1311. Sorry for the 'double ticket'. Thx, Leper, for pointing this out!

Closing this ticket then, with the closest resolution to 'superseded' I could find :)

comment:5 Changed 14 months ago by historic_bruno

  • Milestone Backlog deleted
Note: See TracTickets for help on using tickets.