Bug 107944 - Publishing fails if a live item ends up in the publishing queue
Publishing fails if a live item ends up in the publishing queue
Product: Red Hat Enterprise CMS
Classification: Retired
Component: other (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: ccm-bugs-list
Jon Orris
Depends On:
Blocks: 100952
  Show dependency treegraph
Reported: 2003-10-24 15:42 EDT by Mike Bonnet
Modified: 2007-04-18 12:58 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2003-12-10 11:47:07 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
Proposed fix (1.06 KB, patch)
2003-10-24 15:46 EDT, Mike Bonnet
no flags Details | Diff

  None (edit)
Description Mike Bonnet 2003-10-24 15:42:59 EDT
From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030701

Description of problem:
cms.lifecycle.PublishLifecycleListener.java asserts that it only operates on
draft or pending versions.  However, there are some situations (involving
auto-publishing) in which a live item could end up in the publishing queue. 
This causes publishing to die with an assertion error.

There is no need for the assertion.  The promotePendingVersion() method (called
by the PublishLifecycleListener) is smart enough to do the right thing if it's
passed an already-live item.  See the attached patch for a solution to the problem.

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

How reproducible:

Steps to Reproduce:
1. Update an item from pending to live while it's in the publishing queue.

Actual Results:  Assertion error, item does not get published correctly (it
never becomes the "liveVersion" of it's draft item).

Expected Results:  No assertion error, item does get published correctly.

Additional info:
Comment 1 Mike Bonnet 2003-10-24 15:46:33 EDT
Created attachment 95468 [details]
Proposed fix

This removes the assertion that the item is draft or pending, and treats a live
version the same as a pending version.	ContentItem.promotePendingVersion()
does the right thing in either case.
Comment 2 Mike Bonnet 2003-10-24 15:50:11 EDT
Adding as blocking the 6.0.1 tracking bug, because it's required for a client
(they're already using the patch, and we'd *really* like to get them onto a
fully supported release).
Comment 3 Archit Shah 2003-12-02 15:23:18 EST
merged on 6.0.x (38222) and dev (38422)
Comment 4 Archit Shah 2003-12-02 15:25:48 EST
actually, it is change 38411 on dev

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