It gives you the long-sword, and provides a city trophy. The trophy does have bonuses, but the other thing I notice is this
<GameModifier>
<ModType>City</ModType>
<Attribute>TrainingBonus_ApplyAbilityBonus</Attribute>
<StrVal>Constitution</StrVal>
<Multiplier>1.5</Multiplier>
<Provides>Units trained here gain the Constitution trait</Provides>
</GameModifier>
The multiplier is unnecessary at the very least; since the trait is giving you the constitution ability to your units.
More importantly the problem lies with <ModType>City</ModType>
It should be instead
<ModType>CityTrainingBonus</ModType>