Bug 107636
Summary: | Adding Actions to Forms (CMS ContentType item) generates error | ||
---|---|---|---|
Product: | [Retired] Red Hat Enterprise CMS | Reporter: | Richard Su <rwsu> |
Component: | content types | Assignee: | ccm-bugs-list |
Status: | CLOSED RAWHIDE | QA Contact: | Jon Orris <jorris> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 6.0 | ||
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2003-12-10 20:09:08 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 108447, 109973 |
Description
Richard Su
2003-10-21 16:21:53 UTC
Probably a dup of 109973. Marking as blocking to double-check it's tested before 6.0.1 release. Still seeing the same error after I integrated changelist 38130 as referenced by bz109973 The change 38130 only removed redundant save() calls for WidgetForm. The same thing neeeds to be done for ProcessListenerForm. The following code block is what I believe the problem is: PersistentProcessListener widget = null; if (action == null) { widget = getProcessListener(); m_action.setSelectedKey(state, widget.setID()); } else { try { widget = getProcessListener(action); } catch (DataObjectNotFoundException ex) { throw new FormProcessException("cannot find persistent process listener " + action, ex); } } processWidgets(e, widget); widget.save(); if (action == null) addToForm(e, widget); The call to 'widget.save()' needs to be removed for sure. Dan thought the problem may also have been fixed with bz 37977 and 37978. I diffed my sourec with those changes, and they were consistent. I also commented out the widget.save() line, and commented out .save() in a few other places. Still seeing the same error. workaround for bug 105378 (which basically doesn't allow 1..1 and link attributes) fixes the problem. done at 38005 (dev) and 38678 (6.0.x). |