Jump to content

Living Conditions is not working for the Persian Stables


Zewlakov
 Share

Recommended Posts

Sounds incorrect. If a stables is a barracks, then it should also receive the Barracks class. If a stables isn't a barracks, then the string should be changed. Unfortunately we're in string freeze for alpha 22. Also the tech is too expensive unless you're playing a game with death match resources. Could also increase the effect of the tech, but then what do we have temples for?

We can remove it from the production queue of the stables, or keep it there and fix it shortly after the release, or fix the affects entry now and the string after the release in case noone forgets.

  • Like 2
Link to comment
Share on other sites

Persian Stables xml don't have the Barracks class, it's replaced with Stables.

<Entity parent="template_structure_military_barracks">
  [...]
  <VisibleClasses datatype="tokens">-Barracks Stables</VisibleClasses>

Maybe both classes sounds better.

Anyway, if we have temples and monks, I don't see the point of this tech. Could be removed from the stables and barracks.

(Sorry for my English)

  • Like 2
Link to comment
Share on other sites

Copio todo el código para que se entienda.
Para probarlo simplemente se puede poner MaxDistance en un edificio y al colocarlo tira un warning.

if (this.template.Distance.MaxDistance)
{
  var dist = +this.template.Distance.MaxDistance;
  var nearEnts = cmpRangeManager.ExecuteQuery(this.entity, 0, dist, [cmpPlayer.GetPlayerID()], IID_BuildRestrictions).filter(filter);
  if (!nearEnts.length)
  {
    var result = markForPluralTranslation(
    "%(name)s too far from a %(category)s, must be within %(distance)s meter",
    "%(name)s too far from a %(category)s, must be within %(distance)s meters",
    +this.template.Distance.MinDistance);                                             // <---- This line

    result.success = false;
    result.translateMessage = true;
    result.parameters = {
      "name": name,
      "category": cat,
      "distance": this.template.Distance.MaxDistance
    };
    result.translateParameters = ["name", "category"];
    return result;	// Fail
  }
}

 

  • Like 1
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...