Bug 907897

Summary: pretty.print(null) fails
Product: [Other] RHQ Project Reporter: Filip Brychta <fbrychta>
Component: CLIAssignee: Nobody <nobody>
Status: VERIFIED --- QA Contact:
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 4.5CC: hrupp, jmartine
Target Milestone: ---   
Target Release: RHQ 4.14   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
: 1324828 (view as bug list) Environment:
Last Closed: 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:
Bug Depends On:    
Bug Blocks: 1324828    

Description Filip Brychta 2013-02-05 13:47:19 UTC
Description of problem:
This is very annoying in non-interactive mode (script is terminated). Null is a valid value returned by some methods, so pretty.print should print it without terminating the script.

Version-Release number of selected component (if applicable):
Version: 4.6.0-SNAPSHOT
Build Number: a74896a

How reproducible:
always

Steps to Reproduce:
1. ./rhq-cli.sh -s $yourRHQserverIP -p rhqadmin -u rhqadmin
2. pretty.print(null)

  
Actual results:
org.mozilla.javascript.EvaluatorException: The choice of Java method org.rhq.bindings.output.TabularWriter.print matching JavaScript argument types (null) is ambiguous; candidate methods are: 
    void print(java.lang.Object)
    void print(java.util.Map)
    void print(java.util.Collection)
    void print(org.rhq.core.domain.configuration.Configuration)
    void print(java.lang.Object[])
    void print(java.lang.String[][]) (<Unknown source>#1) in <Unknown source> at line number 1
pretty.print(null); 

Expected results:
null

Comment 1 John Sanda 2015-11-18 19:17:00 UTC
I saw that commit 8b540d7848a was pushed to fix this, but it resulted in a minor regression caught by some Jenkins test failures. I fixed the test failures with commit e1fa9edbe0.

Comment 2 Filip Brychta 2016-06-20 11:27:50 UTC
This is again broken, probably after https://github.com/rhq-project/rhq/commit/1074fac1280e34758fbfda8b1aaaebbdd184313b