|
From: SourceForge.net <no...@so...> - 2007-07-26 02:14:41
|
Bugs item #1760719, was opened at 2007-07-25 22:14 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=724978&aid=1760719&group_id=132696 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: User Interface Group: Pre-Alpha Status: Open Resolution: None Priority: 5 Private: No Submitted By: Wyrm Tools (wyrmtools) Assigned to: Wyrm Tools (wyrmtools) Summary: Layout fragments not updated with their effective element Initial Comment: When you update the name of an element (e.g. via an editor), all associated layout fragments should be updated in the UI to reflect the new name. This is no longer occurring, and thus is a regression. ENVIRONMENT: Wyrm Tools 0.12.0 REPRO STEPS: 1) Create a repository and import the d20 SRD. 2) In the Model Explorer, set the Source Layout to be Publisher/Type/Name. 3) Expand the Sources node completely. It should display a branch "Wizards of the Coast->Rulebook->System Reference Document". 4) Expand the Publication Types node to reveal the "Rulebook" element. 5) Open the publication type editor for the "Rulebook" element. 6) Change the name of the element (e.g. to "Rulebook2"). 7) Save the changes to the element. EXPECTED BEHAVIOR: Both the element under Publication Types and the fragment under Sources should reflect the changed name. ACTUAL BEHAVIOR: Only the element under Publication Types is updated. The fragment under Sources still displays the original name. It appears this broke in revision 1.10 of org.wyrmtools.internal.core.model.Model on 2005-10-23. The comment indicates elements that do not exist in the model do not have their events propagated through the model, otherwise detached elements firing events cause all sorts of problems. A quick fix seemed to be adding an exception to the !element.exists() expression, namely if( !element.exists() && element.getType() != IFragment.TYPE ) but it is not clear if this will cause any additional regressions. Not to mention it's just plain ugly. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=724978&aid=1760719&group_id=132696 |
