From: SourceForge.net <no...@so...> - 2009-03-26 16:07:27
|
Feature Requests item #2710646, was opened at 2009-03-24 19:54 Message generated for change (Comment added) made by thpr You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384722&aid=2710646&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: Skill Support Group: Unscheduled Status: Open Resolution: None >Priority: 2 Private: No Submitted By: Kevin W. Patterson (kwp) Assigned to: Nobody/Anonymous (nobody) >Summary: Handle Duplicate Professions/Crafts Initial Comment: As craft and profession skills may exist in many different campaigns or sources, ALLOWDUPES should include SKILL as an option, so such things duplicate entries for Profession (Sailor) or Profession (Hunter) do not generate errors. ---------------------------------------------------------------------- >Comment By: Tom Parker (thpr) Date: 2009-03-26 12:07 Message: After rethinking, Changing to priority 2. The underlying issue needs to be fixed in a way that is reasonable - and should be within the scope of PCGen. The thought behind the original prio 1 decision (ALLOWDUPES) is a problem, but there are other methods we can consider for how to handle this situation that doesn't expose the user to other error suppression the way the "override with newer sources" feature does. ---------------------------------------------------------------------- Comment By: Tom Parker (thpr) Date: 2009-03-25 22:55 Message: ALLOWDUPES is a temporary workaround to allow ambiguous data structures to exist. It is not a long-term token and we do not want to encourage its use. If two sources have conflicting skills, then the skills need different keys. If two sources have identical skills, then the skills should be pulled from a common source, or the "override with newer sources" preference (as much as I dislike it because it hides errors) should be activated. Alternately, a much more complicated loading system should be developed to handle this case (this is probably the best situation), but such it out of scope of our development plans at this time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384722&aid=2710646&group_id=25576 |
From: SourceForge.net <no...@so...> - 2010-06-18 13:01:56
|
Feature Requests item #2710646, was opened at 2009-03-24 19:54 Message generated for change (Comment added) made by papa_drb You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384722&aid=2710646&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: Skill Support Group: Unscheduled >Status: Deleted Resolution: None Priority: 2 Private: No Submitted By: Kevin W. Patterson (kwp) Assigned to: Nobody/Anonymous (nobody) Summary: Handle Duplicate Professions/Crafts Initial Comment: As craft and profession skills may exist in many different campaigns or sources, ALLOWDUPES should include SKILL as an option, so such things duplicate entries for Profession (Sailor) or Profession (Hunter) do not generate errors. ---------------------------------------------------------------------- >Comment By: David R. Bender (papa_drb) Date: 2010-06-18 09:01 Message: jira code-105 ---------------------------------------------------------------------- Comment By: Tom Parker (thpr) Date: 2009-03-26 12:07 Message: After rethinking, Changing to priority 2. The underlying issue needs to be fixed in a way that is reasonable - and should be within the scope of PCGen. The thought behind the original prio 1 decision (ALLOWDUPES) is a problem, but there are other methods we can consider for how to handle this situation that doesn't expose the user to other error suppression the way the "override with newer sources" feature does. ---------------------------------------------------------------------- Comment By: Tom Parker (thpr) Date: 2009-03-25 22:55 Message: ALLOWDUPES is a temporary workaround to allow ambiguous data structures to exist. It is not a long-term token and we do not want to encourage its use. If two sources have conflicting skills, then the skills need different keys. If two sources have identical skills, then the skills should be pulled from a common source, or the "override with newer sources" preference (as much as I dislike it because it hides errors) should be activated. Alternately, a much more complicated loading system should be developed to handle this case (this is probably the best situation), but such it out of scope of our development plans at this time. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384722&aid=2710646&group_id=25576 |