Bug 835288

Summary: csprocessor does not replace new topic notation with topic id on push
Product: [Community] PressGang CCMS Reporter: Stephen Gordon <sgordon>
Component: CSProcessorAssignee: Lee Newson <lnewson>
Status: CLOSED CANTFIX QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 1.xCC: jwulf
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: 0.24.9 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-04-26 05:01:49 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Content Specification none

Comment 2 Lee Newson 2012-06-25 22:31:24 UTC
Some info please? eg The content spec that failed, what version you were using, where there any abnormalities when pushing, etc...

Comment 3 Stephen Gordon 2012-06-26 01:53:29 UTC
All of these details, and the content specification, are marked private (that is why your comment is c#2). You might want to get your bugzilla permissions checked out to make sure they are correct ;). In the meantime I will email the details to you.

Comment 4 Stephen Gordon 2012-06-26 01:57:31 UTC
Created attachment 594329 [details]
Content Specification

Comment 5 Lee Newson 2012-06-26 02:20:31 UTC
Cheers, thanks Steve and I'll didn't even notice my comment was number so thanks for the heads up.

Comment 6 Lee Newson 2012-06-26 06:26:22 UTC
Fixed will attempt to release a version today. If not I'll get it done first thing tomorrow morning.

Cause:

Under certain circumstances the input string was returned for the replace function for new topics. The issue was due to a regular expression and returning the wrong item.

Consequence:

New topics weren't being created in the pushed content specification.

Fix:

Fix the regular expression and the return statement to return the updated output string.

Comment 7 Lee Newson 2012-06-26 07:41:59 UTC
Released as part of 0.24.9

Comment 8 Stephen Gordon 2012-06-27 20:15:22 UTC
Running cspclient-0.24.10-1.noarch and I tried to add a few new topics this way today and encountered the same behaviour (topics created in skynet, not reflected in the local specification). Has this been tested?

I'm adding topics to the tail end of content spec with ID 9054, I'm wondering if the length/complexity of the spec is still throwing off your regex?

Comment 9 Lee Newson 2012-06-27 22:35:05 UTC
Will test against a live server backup. I was testing against our developer server which has a different REST interface but I didn't see this as an issue.

Comment 10 Lee Newson 2012-06-27 23:40:33 UTC
I've tested it using the revision's that I found for topic 9054 against version 0.24.10 of the CSP and wasn't able to replicate it after multiple attempts. 

For a start, I'll add a warning message, if after post processing any new topics are found. That will at least stop duplicate topics being added. As for your issue I'll keep trying to see if I can replicate it.

Comment 12 Lee Newson 2013-04-26 05:01:49 UTC
Closing this as I couldn't replicate it and it has been reported happening again. If it does happen again, please re-open the bug.

Comment 13 Lee Newson 2013-04-26 05:02:59 UTC
Oops "has" should have been "hasn't" in the previous comment.