Opened 7 years ago

Last modified 7 years ago

#4440 closed defect

Issues in r19095 — at Version 14

Reported by: fatherbushido Owned by:
Priority: Must Have Milestone: Alpha 22
Component: Art & Animation Keywords: patch
Cc: enrique, Johnathan Good Patch:

Description (last modified by elexis)

In r19095:

  • Spaces replaced by tabs
  • When adding classes, they should be documented in Identity schema help. Moreover classes like Female Guerilla are imo useless.
  • That upgrade feature to switch between those two bouddica model will not work.
  • Don't change identation in Auras entry (use the inline one).

Also:

Validating actors...
../../../binaries/data/mods/public/art/actors/props/units/cape_animated_bow_chariot_rider.xml:15: parser error : Opening and ending tag mismatch: group line 14 and variant

</variant>
          ^

../../../binaries/data/mods/public/art/actors/props/units/cape_animated_bow_chariot_rider.xml:16: parser error : Opening and ending tag mismatch: actor line 2 and group

</group>
        ^

../../../binaries/data/mods/public/art/actors/props/units/cape_animated_bow_chariot_rider.xml:17: parser error : Extra content at the end of the document

Also

 <p class="error">ERROR: Failed to find matching prop point called "projectile" in model "art/meshes/props/new/capes/cape_hd_medium.dae" for actor "cape_animated_bow_hip"</p>
.:Stan`:. <p class="error">ERROR: Failed to find matching prop point called "weapon_R" in model "art/meshes/props/new/capes/cape_hd_medium.dae" for actor "cape_animated_jav"</p>

<group>
^

Change History (18)

comment:1 by Stan, 7 years ago

Component: UI & SimulationArt & Animation
Description: modified (diff)

by fatherbushido, 7 years ago

Attachment: cleaning.diff added

clean simulation xml files

comment:2 by fabio, 7 years ago

Description: modified (diff)

comment:3 by fatherbushido, 7 years ago

In 19103:

Fix identation issues, line endings and remove a not working entry in templates. Refs r19095. Refs #4440.

comment:4 by fatherbushido, 7 years ago

Also some easter eggs units are introduced (for example, the ptolemaic pikemen is buildable only in some captured fortress). (I am not against it ;-)).

Last edited 7 years ago by fatherbushido (previous) (diff)

comment:6 by elexis, 7 years ago

Was it actually intended to be an easteregg? I assume players might report this as a bug if they discover it.

comment:7 by Lionkanzen, 7 years ago

Sounds nice have citizen soldiers from captured fortress.

by Stan, 7 years ago

Attachment: fndn_dock_super.xml.patch added

Fix the broken foundation actor committed in r19095 for the super dock.

by fatherbushido, 7 years ago

Attachment: capechariot.diff added

comment:8 by fatherbushido, 7 years ago

stanislas69: thx for the patch. Can you look at that if it seems ok? http://trac.wildfiregames.com/attachment/ticket/4440/capechariot.diff

in reply to:  8 comment:9 by Stan, 7 years ago

Replying to fatherbushido:

stanislas69: thx for the patch. Can you look at that if it seems ok? http://trac.wildfiregames.com/attachment/ticket/4440/capechariot.diff

Patch is okay. for a full fix of the actor we could do something like this. see following attachment.

by Stan, 7 years ago

Attachment: capechariot.1.diff added

Merge the two above variant blocks as there is no variation plus fatherbushido's fix. Not sure why he hasn't added all the anim inheritance.

comment:10 by fatherbushido, 7 years ago

stanislas69: thx for the patches. Can you explain if there is something to see in atlas actor viewer to see that the patch fix something (despite the fact that now there is no error)?

comment:11 by Stan, 7 years ago

You should now be able to see the footsoldier vercingetorix in actors(all) and the textures for the corespondong actor spec being the reflection

in reply to:  11 comment:12 by fatherbushido, 7 years ago

Replying to stanislas69:

You should now be able to see the footsoldier vercingetorix in actors(all) and the textures for the corespondong actor spec being the reflection

I don't understand!

comment:13 by Stan, 7 years ago

I'm sorry :/ I actually was referencing another ticket with issues. So no nothing to check for sadly.

Note: See TracTickets for help on using tickets.