From: SourceForge.net <no...@so...> - 2005-10-02 19:50:01
|
Feature Requests item #1262317, was opened at 2005-08-17 17:09 Message generated for change (Comment added) made by nuance You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384722&aid=1262317&group_id=25576 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 8 Submitted By: Devon Jones (soulcatcher) Assigned to: Andrew Wilson (nuance) Summary: Ability Object: Add ability tags Initial Comment: Create a new set of tags, that are all exactly like the FEAT tags (ADD:ABILITY instead of ADD:FEAT, ABILITYAUTO instead of FEATAUTO, etc) except instead of just the name, it will need to take in category|name so that we are able to have separate name spaces for ability object. The person that does 1262313 should probably do this one This freq depends on 1262306 If you need any guidance, contact me. Devon ---------------------------------------------------------------------- >Comment By: Andrew Wilson (nuance) Date: 2005-10-02 19:50 Message: Logged In: YES user_id=41667 src/java/pcgen/core/levelability/LevelAbility.java 1.9 src/java/pcgen/core/levelability/LevelAbilityAbility.java 1.1 First cut at ADD:ABILITY This needs serious testing, I'm just commiting it as a baseline so I don't lose it. ---------------------------------------------------------------------- Comment By: Andrew Wilson (nuance) Date: 2005-10-02 18:26 Message: Logged In: YES user_id=41667 src/java/pcgen/core/AbilityUtilities.java 1.8 src/java/pcgen/core/Kit.java 1.86 src/java/pcgen/core/PObject.java 1.754 src/java/pcgen/core/PObjectUtilities.java 1.16 src/java/pcgen/core/PlayerCharacter.java 1.1557 src/java/pcgen/core/chooser/FeatChoiceManager.java 1.4 src/java/pcgen/core/levelability/LevelAbilityFeat.java 1.13 src/java/pcgen/core/levelability/LevelAbilitySpecial.java 1.8 src/java/pcgen/gui/tabs/InfoFeats.java 1.81 src/java/pcgen/io/PCGVer0Parser.java 1.186 src/test/pcgen/core/PlayerCharacterTest.java 1.23 Split modFeat into: 1) the bit that looks for an Ability by name and then makes a clone of it and 2) the bit that then processes the clone. Added a modAbility to AbilityUtilities.java instead of a name with the choices embeded, this takes an Ability object and a String (which is the choice with no further processing necessary). It clones the ability and then calls part 2 of modFeat (as above). Changed the order of the arguments in modFeat. ---------------------------------------------------------------------- Comment By: Andrew Wilson (nuance) Date: 2005-10-02 14:45 Message: Logged In: YES user_id=41667 src/java/pcgen/core/AbilityUtilities.java 1.7 src/java/pcgen/core/Kit.java 1.85 src/java/pcgen/core/PObject.java 1.753 src/java/pcgen/core/PObjectUtilities.java 1.15 src/java/pcgen/core/PlayerCharacter.java 1.1556 src/java/pcgen/core/chooser/FeatChoiceManager.java 1.3 src/java/pcgen/core/levelability/LevelAbilityFeat.java 1.12 src/java/pcgen/core/levelability/LevelAbilitySpecial.java 1.7 src/java/pcgen/gui/tabs/InfoFeats.java 1.80 src/java/pcgen/io/PCGVer0Parser.java 1.185 src/test/pcgen/core/PlayerCharacterTest.java 1.22 Encapsulate importing and feats in PlayerCharacter so that ModFeat can be made static. Convert modFeat to a static method that takes a PC argument Move modFeats to AbilityUtilities ---------------------------------------------------------------------- Comment By: Andrew Wilson (nuance) Date: 2005-10-02 02:59 Message: Logged In: YES user_id=41667 src/java/pcgen/core/AbilityUtilities.java 1.6 src/java/pcgen/core/PlayerCharacter.java 1.1555 src/java/pcgen/core/levelability/LevelAbilityFeat.java 1.11 src/java/pcgen/io/PCGVer2Parser.java 1.184 Moved method addVirtualFeat from PlayerCharacter.java and made it a static method of AbilityUtilities.java. Split addVirtualFeat into two methods one the first has the same interface as the current method, the second uses an Ability Object and a list of choices instead of an Ability name. ---------------------------------------------------------------------- Comment By: Andrew Wilson (nuance) Date: 2005-09-30 19:06 Message: Logged In: YES user_id=41667 src/java/pcgen/core/Ability.java 1.21 src/java/pcgen/core/AbilityUtilities.java 1.5 Add some utility methods to Ability and AbilitiyUtilities ---------------------------------------------------------------------- Comment By: Andrew Wilson (nuance) Date: 2005-09-11 14:34 Message: Logged In: YES user_id=41667 src/java/pcgen/core/Ability.java 1.11 src/java/pcgen/core/Kit.java 1.74 src/java/pcgen/core/KitUtilities.java 1.2 src/java/pcgen/core/kit/KitFeat.java delete src/java/pcgen/gui/KitSelector.java 1.54 src/java/pcgen/persistence/lst/KitLoader.java 1.31 Update Kit code to take account of the fact that Abilities Need Category as well as Key (or name) to specify them. ---------------------------------------------------------------------- Comment By: Andrew Wilson (nuance) Date: 2005-09-11 14:20 Message: Logged In: YES user_id=41667 src/java/pcgen/core/AbilityInfo.java 1.1 src/java/pcgen/core/kit/KitAbilities.java 1.1 Two new classes to deal with the differences between Feats and Abilities in Kits. We can't just refer to an Ability by it's Key anymore, we also need its category. AblityInfo is a small object that can be used to refer to an AbilityObject. ---------------------------------------------------------------------- Comment By: Andrew Wilson (nuance) Date: 2005-09-06 22:04 Message: Logged In: YES user_id=41667 src/java/pcgen/core/EquipmentChoice.java 1.4 Added parsing of ABILITY tag. ---------------------------------------------------------------------- Comment By: Andrew McDougall (tir-gwaith) Date: 2005-08-17 17:14 Message: Logged In: YES user_id=208239 FEATAUTO is a class.lst object only. We need to deprecate that in favor of a new AUTO:FEAT tag (following other AUTO:xxx tag syntax). In this case, we would want it AUTO:ABILITY ---------------------------------------------------------------------- Comment By: Devon Jones (soulcatcher) Date: 2005-08-17 17:12 Message: Logged In: YES user_id=107647 this also depends on 1262309 ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384722&aid=1262317&group_id=25576 |