From: SourceForge.net <no...@so...> - 2007-04-28 11:42:23
|
Bugs item #1687659, was opened at 2007-03-25 11:26 Message generated for change (Settings changed) made by jdempsey You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384719&aid=1687659&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: Equipment/Weaponprof Support >Group: To be Fixed for 6.0 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Tom Parker (thpr) Assigned to: Nobody/Anonymous (nobody) Summary: Eq Customizer ignores PLUS: Token restrictions after removal Initial Comment: Weapon Editor does not enforce PLUS when removing the + modifier. e.g. you can: Customize Longsword - Bane is not available Make it +1 - Bane is now available Add Bane(Dragons) Remove +1, leaving Bane - Except Bane should be illegal ---------------------------------------------------------------------- Comment By: Martijn Verburg (karianna) Date: 2007-04-26 00:09 Message: Logged In: YES user_id=252169 Originator: NO confirmed, code dealing with this is in Equipment.java, in particular the method: public void removeEqModifier(final EquipmentModifier eqMod, final boolean bPrimary) { .. .. if (false) removeUnqualified(bPrimary); // TODO: used? ) The removeUnqualified() method itself has a comment: // The problem is that you have entries // like the following for Adamantine: // PRETYPE:Weapon,Metal !PRETYPE:Masterwork // Which have nothing to do with the eqMod, so // they are always going to fail and be removed. // The entries in equip_enhancing are used by // the GUI to know what to display, not for // actual passesPreReq checks ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384719&aid=1687659&group_id=25576 |