Unhardcode internal terrain palette name from all over the codebase
This commit is contained in:
@@ -16,8 +16,6 @@ namespace OpenRA.Graphics
|
||||
{
|
||||
sealed class TerrainRenderer : IDisposable
|
||||
{
|
||||
const string TerrainPalette = "terrain";
|
||||
|
||||
readonly World world;
|
||||
readonly Dictionary<string, TerrainSpriteLayer> spriteLayers = new Dictionary<string, TerrainSpriteLayer>();
|
||||
readonly Theater theater;
|
||||
@@ -33,7 +31,7 @@ namespace OpenRA.Graphics
|
||||
|
||||
foreach (var template in world.TileSet.Templates)
|
||||
{
|
||||
var palette = template.Value.Palette ?? TerrainPalette;
|
||||
var palette = template.Value.Palette ?? TileSet.TerrainPaletteInternalName;
|
||||
spriteLayers.GetOrAdd(palette, pal =>
|
||||
new TerrainSpriteLayer(world, wr, theater.Sheet, BlendMode.Alpha, wr.Palette(palette), wr.World.Type != WorldType.Editor));
|
||||
}
|
||||
@@ -48,7 +46,7 @@ namespace OpenRA.Graphics
|
||||
public void UpdateCell(CPos cell)
|
||||
{
|
||||
var tile = mapTiles[cell];
|
||||
var palette = world.TileSet.Templates[tile.Type].Palette ?? TerrainPalette;
|
||||
var palette = world.TileSet.Templates[tile.Type].Palette ?? TileSet.TerrainPaletteInternalName;
|
||||
var sprite = theater.TileSprite(tile);
|
||||
foreach (var kv in spriteLayers)
|
||||
kv.Value.Update(cell, palette == kv.Key ? sprite : null);
|
||||
|
||||
@@ -167,6 +167,8 @@ namespace OpenRA
|
||||
|
||||
public class TileSet
|
||||
{
|
||||
public const string TerrainPaletteInternalName = "terrain";
|
||||
|
||||
public readonly string Name;
|
||||
public readonly string Id;
|
||||
public readonly int SheetSize = 512;
|
||||
|
||||
@@ -16,7 +16,7 @@ namespace OpenRA.Traits
|
||||
public class FixedColorPaletteInfo : ITraitInfo
|
||||
{
|
||||
[Desc("The name of the palette to base off.")]
|
||||
[PaletteReference] public readonly string Base = "terrain";
|
||||
[PaletteReference] public readonly string Base = TileSet.TerrainPaletteInternalName;
|
||||
|
||||
[Desc("The name of the resulting palette")]
|
||||
[PaletteDefinition] public readonly string Name = "resources";
|
||||
|
||||
@@ -17,7 +17,7 @@ namespace OpenRA.Traits
|
||||
{
|
||||
public readonly string Sequence = "resources";
|
||||
[SequenceReference("Sequence")] public readonly string[] Variants = { };
|
||||
[PaletteReference] public readonly string Palette = "terrain";
|
||||
[PaletteReference] public readonly string Palette = TileSet.TerrainPaletteInternalName;
|
||||
public readonly int ResourceType = 1;
|
||||
|
||||
public readonly int ValuePerUnit = 0;
|
||||
|
||||
@@ -18,7 +18,7 @@ namespace OpenRA.Mods.Common.Traits
|
||||
public class BibInfo : ITraitInfo, Requires<BuildingInfo>, IRenderActorPreviewSpritesInfo, Requires<RenderSpritesInfo>
|
||||
{
|
||||
[SequenceReference] public readonly string Sequence = "bib";
|
||||
[PaletteReference] public readonly string Palette = "terrain";
|
||||
[PaletteReference] public readonly string Palette = TileSet.TerrainPaletteInternalName;
|
||||
public readonly bool HasMinibib = false;
|
||||
|
||||
public object Create(ActorInitializer init) { return new Bib(init.Self, this); }
|
||||
|
||||
@@ -192,7 +192,7 @@ namespace OpenRA.Mods.Common.Traits
|
||||
{
|
||||
if (!initialized)
|
||||
{
|
||||
var palette = wr.Palette("terrain");
|
||||
var palette = wr.Palette(TileSet.TerrainPaletteInternalName);
|
||||
renderables = new Dictionary<ushort, IRenderable[]>();
|
||||
foreach (var t in info.Templates)
|
||||
renderables.Add(t.First, TemplateRenderables(wr, palette, t.First));
|
||||
|
||||
@@ -20,7 +20,7 @@ namespace OpenRA.Mods.Common.Traits
|
||||
public class PlaceBuildingInfo : ITraitInfo
|
||||
{
|
||||
[Desc("Palette to use for rendering the placement sprite.")]
|
||||
[PaletteReference] public readonly string Palette = "terrain";
|
||||
[PaletteReference] public readonly string Palette = TileSet.TerrainPaletteInternalName;
|
||||
|
||||
[Desc("Play NewOptionsNotification this many ticks after building placement.")]
|
||||
public readonly int NewOptionsNotificationDelay = 10;
|
||||
|
||||
@@ -149,7 +149,7 @@ namespace OpenRA.Mods.Common.Traits
|
||||
public IEnumerable<IRenderable> Render(WorldRenderer wr, World world)
|
||||
{
|
||||
var xy = wr.Viewport.ViewToWorld(Viewport.LastMousePos);
|
||||
var pal = wr.Palette("terrain");
|
||||
var pal = wr.Palette(TileSet.TerrainPaletteInternalName);
|
||||
|
||||
foreach (var t in world.Map.FindTilesInCircle(xy, range))
|
||||
yield return new SpriteRenderable(tile, wr.World.Map.CenterOfCell(t), WVec.Zero, -511, pal, 1f, true);
|
||||
|
||||
@@ -25,7 +25,7 @@ namespace OpenRA.Mods.Common.Traits
|
||||
{
|
||||
[PaletteReference]
|
||||
[Desc("Palette to use for rendering the placement sprite.")]
|
||||
public readonly string Palette = "terrain";
|
||||
public readonly string Palette = TileSet.TerrainPaletteInternalName;
|
||||
|
||||
[Desc("Sequence image where the selection overlay types are defined.")]
|
||||
public readonly string Image = "editor-overlay";
|
||||
|
||||
@@ -33,7 +33,7 @@ namespace OpenRA.Mods.Common.Traits
|
||||
|
||||
[PaletteReference] public readonly string SmokePalette = "effect";
|
||||
|
||||
[PaletteReference] public readonly string Palette = "terrain";
|
||||
[PaletteReference] public readonly string Palette = TileSet.TerrainPaletteInternalName;
|
||||
|
||||
public object Create(ActorInitializer init) { return new SmudgeLayer(init.Self, this); }
|
||||
}
|
||||
|
||||
@@ -18,7 +18,7 @@ namespace OpenRA.Mods.Common.Widgets
|
||||
public class TerrainTemplatePreviewWidget : Widget
|
||||
{
|
||||
public Func<float> GetScale = () => 1f;
|
||||
public string Palette = "terrain";
|
||||
public string Palette = TileSet.TerrainPaletteInternalName;
|
||||
|
||||
readonly WorldRenderer worldRenderer;
|
||||
readonly TileSet tileset;
|
||||
|
||||
@@ -18,7 +18,7 @@ namespace OpenRA.Mods.D2k.Traits
|
||||
public class BuildableTerrainLayerInfo : ITraitInfo
|
||||
{
|
||||
[Desc("Palette to render the layer sprites in.")]
|
||||
public readonly string Palette = "terrain";
|
||||
public readonly string Palette = TileSet.TerrainPaletteInternalName;
|
||||
|
||||
public object Create(ActorInitializer init) { return new BuildableTerrainLayer(init.Self, this); }
|
||||
}
|
||||
|
||||
@@ -121,7 +121,7 @@ namespace OpenRA.Mods.RA.Traits
|
||||
if (self.Owner != self.World.LocalPlayer || Minefield == null)
|
||||
yield break;
|
||||
|
||||
var pal = wr.Palette("terrain");
|
||||
var pal = wr.Palette(TileSet.TerrainPaletteInternalName);
|
||||
foreach (var c in Minefield)
|
||||
yield return new SpriteRenderable(tile, self.World.Map.CenterOfCell(c),
|
||||
WVec.Zero, -511, pal, 1f, true);
|
||||
@@ -181,7 +181,7 @@ namespace OpenRA.Mods.RA.Traits
|
||||
var minefield = GetMinefieldCells(minefieldStart, lastMousePos,
|
||||
minelayer.Info.TraitInfo<MinelayerInfo>().MinefieldDepth);
|
||||
|
||||
var pal = wr.Palette("terrain");
|
||||
var pal = wr.Palette(TileSet.TerrainPaletteInternalName);
|
||||
foreach (var c in minefield)
|
||||
{
|
||||
var tile = movement.CanEnterCell(c, null, false) ? tileOk : tileBlocked;
|
||||
|
||||
@@ -145,7 +145,7 @@ namespace OpenRA.Mods.RA.Traits
|
||||
{
|
||||
var xy = wr.Viewport.ViewToWorld(Viewport.LastMousePos);
|
||||
var tiles = world.Map.FindTilesInCircle(xy, range);
|
||||
var pal = wr.Palette("terrain");
|
||||
var pal = wr.Palette(TileSet.TerrainPaletteInternalName);
|
||||
foreach (var t in tiles)
|
||||
yield return new SpriteRenderable(tile, wr.World.Map.CenterOfCell(t), WVec.Zero, -511, pal, 1f, true);
|
||||
}
|
||||
@@ -224,7 +224,7 @@ namespace OpenRA.Mods.RA.Traits
|
||||
public IEnumerable<IRenderable> Render(WorldRenderer wr, World world)
|
||||
{
|
||||
var xy = wr.Viewport.ViewToWorld(Viewport.LastMousePos);
|
||||
var pal = wr.Palette("terrain");
|
||||
var pal = wr.Palette(TileSet.TerrainPaletteInternalName);
|
||||
|
||||
// Source tiles
|
||||
foreach (var t in world.Map.FindTilesInCircle(sourceLocation, range))
|
||||
|
||||
Reference in New Issue
Block a user