Bug 109022 - It is no longer possible to add an existing content type to a section
Summary: It is no longer possible to add an existing content type to a section
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Red Hat Enterprise CMS
Classification: Retired
Component: content types
Version: nightly
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: ccm-bugs-list
QA Contact: Jon Orris
URL:
Whiteboard:
Depends On:
Blocks: 106597
TreeView+ depends on / blocked
 
Reported: 2003-11-04 14:41 UTC by Randy Graebner
Modified: 2007-04-18 16:59 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2003-11-05 16:19:59 UTC
Embargoed:


Attachments (Terms of Use)

Description Randy Graebner 2003-11-04 14:41:31 UTC
Description of problem:
When I go to the content-types tab and remove a content type, I expect
to be able to click on the "add a content type" link, select the type,
and then add it back to my section.  However, whenever I hit the "add"
button it actually cancells.  This is from the form have a
CancelListener that fires and the CancelListener forgets to check if
the cancel button was actually pressed.  The below patch will fix this
problem.

[randyg@graebner aplaws]$ diff -uw
cms/src/com/arsdigita/cms/ui/type/ContentTypeAdminPane.java
cms/src/com/arsdigita/cms/ui/type/ContentTypeAdminPane2.java
--- cms/src/com/arsdigita/cms/ui/type/ContentTypeAdminPane.java	Tue
Nov  4 09:40:10 2003
+++ cms/src/com/arsdigita/cms/ui/type/ContentTypeAdminPane2.java	Tue
Nov  4 09:39:58 2003
@@ -150,10 +150,12 @@
         public void submitted(FormSectionEvent event) 
             throws FormProcessException {
             PageState state = event.getPageState();
+            if (m_form.isCancelled(state)) {
             getBody().pop(state);
             throw new FormProcessException("cancelled");
         }
     }
+    }
 
     private void resetPane(PageState state) {
         getBody().reset(state);


Version-Release number of selected component (if applicable):
Nightly

How reproducible:
Always

Steps to Reproduce:
1.  try to add a content type
2.
3.
  
Actual results:
the content type should be added

Expected results:
The content type remains unmapped.

Additional info:

Comment 1 Randy Graebner 2003-11-04 15:17:37 UTC
This should be fixed in change 37666.

Comment 2 Scott Seago 2003-11-04 22:43:22 UTC
This works for me now.


Note You need to log in before you can comment on or make changes to this bug.