I struggle to think of how it could be at all a technical issue. If I had coded the current system as it functions today, the glamour validation check would involve, among other things, if item.EquipableBy.contains(currentJob). All that would be needed to allow all non-AF gear to be universally glamourable would be to change that one check to if (item.EquipableBy.size >= 2 or item.EquipableBy.contains(currentJob)).
In layperson's terms, this checks if the item is AF by checking how many jobs/classes can equip it. If it's exactly one, then the gear is AF, and it needs to check if current job matches. If it's equipable by more than one, it's not AF and glamourable by anybody. Of course, this only works so long as we have classes, since the defining feature for DRG AF and NIN AF currently is that they cannot be equipped by LNC and ROG respectively. Were we to eliminate classes, an AF flag would have to be added to all AF gear because you wouldn't be able to tell DRG general gear from DRG AF, or NIN general gear from NIN AF, just by looking at the list of equipable jobs. So in that case the check would be if (not item.IsAF or item.EquipableBy.Contains(currentJob)).
But it should really be as simple as this.



Reply With Quote


