Bug 1007118 - Conflicting PressGang and Publican conditions should produce an error/warning
Summary: Conflicting PressGang and Publican conditions should produce an error/warning
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: CSProcessor
Version: 1.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 1.3
Assignee: Lee Newson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-09-12 00:04 UTC by Lee Newson
Modified: 2013-12-03 22:16 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-12-03 22:16:22 UTC
Embargoed:

Attachments (Terms of Use)

Description Lee Newson 2013-09-12 00:04:41 UTC 
Currently if someone includes a Publican condition (one in the publican.cfg file) and in the Content Spec (ie [condtion=test]), then the later will be completely ignored which could result in unexpected build errors. As such if both are specified then an error or warning should be produced saying that only one way will work.
Comment 1 Lee Newson 2013-11-06 00:30:24 UTC 
Fixed in 1.3-SNAPSHOT build 201311061019

The validator will now print warnings when a condition is used on a node and in the publican.cfg metadata.
Comment 2 Lee Newson 2013-11-07 04:09:52 UTC 
This version is now live on the dev/test server.
Comment 3 Matthew Casperson 2013-11-25 00:17:00 UTC 
Confirmed that an warning is produced when there are conflicting conditions.

I'd suggest that the error be changed from:

A condition has been defined in publican.cfg and as such the condition will be ignored.

To

A condition has been defined in publican.cfg, and as such the condition defined against the topic or container will be ignored.
Comment 4 Lee Newson 2013-11-25 01:45:55 UTC 
Fixed in 1.3-SNAPSHOT build 201311251126

I've updated the warning message to what Matt mentioned above.

Note: This version is currently live on the development/test server.
Comment 5 Matthew Casperson 2013-11-25 21:16:41 UTC 
Confirmed new warning message.
Note You need to log in before you can comment on or make changes to this bug.