Bug 795599 - RFE: more concise output on successful push
Summary: RFE: more concise output on successful push
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: PressGang CCMS
Classification: Community
Component: CSProcessor
Version: 1.x
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
: ---
Assignee: Lee Newson
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-21 01:19 UTC by Joshua Wulf
Modified: 2014-10-19 23:00 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-05-29 02:06:47 UTC

Attachments (Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

Description Joshua Wulf 2012-02-21 01:19:13 UTC 
Currently it looks like this:

[jwulf@radhe temp]$ csprocessor push -cn Test_Book.spec 
CSProcessor client version: 0.16.3
Loading configuration from /home/jwulf/.config/csprocessor.ini
Web Service at http://csprocessor.cloud.lab.eng.bne.redhat.com:8080/ version: 0.21.2

INFO:  The Content Specification is valid.
WARN:  Note: All descriptions, tags, source urls and writers will be ignored for existing Topics.
INFO:  The Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655


Really all it needs to be is this:

[jwulf@radhe temp]$ csprocessor push -cn Test_Book.spec 
CSProcessor client 0.16.3
Configuration file: /home/jwulf/.config/csprocessor.ini
Server: http://csprocessor.cloud.lab.eng.bne.redhat.com:8080/ version: 0.21.2

Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655


Even then, the first part is verbose. Perhaps the default should be:
[jwulf@radhe temp]$ csprocessor push -cn Test_Book.spec 

Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655


and with -v for verbose:

[jwulf@radhe temp]$ csprocessor push -cnv Test_Book.spec 
CSProcessor client 0.16.3
Configuration file: /home/jwulf/.config/csprocessor.ini
Server: http://csprocessor.cloud.lab.eng.bne.redhat.com:8080/ version: 0.21.2

Content Specification saved successfully.
Content Specification ID: 6226
Revision: 39655

In which case, a verbose switch in the csprocessor.ini would be a good idea.
Comment 1 Lee Newson 2012-02-21 06:55:06 UTC 
The only part I agree with here is removing the "INFO:  The Content Specification is valid." message. The others ensure that the user knows what server/configuration they are connecting to. I guess the CSProcessor client message could be removed. 

The "WARN:  Note: All descriptions, tags, source urls and writers will be ignored
for existing Topics." is key to ensure that the user knows that those parts will be ignored in their content specification.

Anyways I'll try and catch you tomorrow to talk about it.
Comment 2 Lee Newson 2012-03-06 02:09:15 UTC 
Update:

The WARN message has been removed from version 0.22.0 since tags can be added to existing topics in that version.
Comment 3 Misty Stanley-Jones 2013-05-29 02:06:47 UTC 
Closing because of Lee's last comment.
Note You need to log in before you can comment on or make changes to this bug.