Do you have more than one character with the same name (such as Veteran)? That would cause rolls to reference the wrong sheet, which may not have the proper information in the attacks.
Are the tokens linked to the correct character?
Those are my first guesses because the sheet looks correct at a glance.
For the Token Actions not working, your code is not referencing the correct thing there. That is not how to reference repeating actions. You are trying to reference an Ability (on the Attributes and Abilities tab) named 'Shortsword', which there is none.

Actions on the D&D 5E by Roll20 'NPC' sheet are listed as 'repeating actions', and to roll the Shortsword from the Veteran character sheet, you'll need to reference the 4th repeating action, which is done as %{Veteraan|repeating_npcaction_$3_npc_action}
You may be interested in using my Universal Statblock Macromule to use a chat menu for NPC rolls instead of opening character sheets.