Bug 829309 - [CLI] String assignment not possible
Summary: [CLI] String assignment not possible
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: RHQ Project
Classification: Other
Component: CLI
Version: 4.4
Hardware: Unspecified
OS: Unspecified
urgent
urgent
Target Milestone: ---
: RHQ 4.5.0
Assignee: John Sanda
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks: 824129 826604
TreeView+ depends on / blocked
 
Reported: 2012-06-06 12:49 UTC by Lukas Krejci
Modified: 2013-09-01 10:10 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-01 10:10:56 UTC
Embargoed:


Attachments (Terms of Use)

Description Lukas Krejci 2012-06-06 12:49:43 UTC
Description of problem:
In the interactive CLI, the following code breaks:

var a = "whatever";

With:

ReferenceError: "whatever" is not defined. 
var a = whatever ; 
^

This seems to me like we parsed away the quotes before we submit the line to the javascript interpreter.

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

How reproducible:
always

Steps to Reproduce:
1. on CLI commandline do:
var a = "whatever";
  
Actual results:
Error as above

Expected results:
assignment succeeds

Additional info:
This is not limited to assignments - any quoted literal (either single or double quoted) seems to be unqouted before interpreted.

Additionaly, this is NOT present in JON 3.1.0.CR2 (CR3 contains no change in CLI so I'd assume it is not present there either)

Comment 1 John Sanda 2012-06-06 13:41:26 UTC
This is a regression caused by the work I did for bug 826604. I will make this bug a blocker for bug 826604.

Comment 2 John Sanda 2012-06-06 18:34:24 UTC
I pushed a fix to master. The parsing changes only take effect for the script arguments.

commit hash: 5e8863403

Comment 3 Heiko W. Rupp 2013-09-01 10:10:56 UTC
Bulk closing of items that are on_qa and in old RHQ releases, which are out for a long time and where the issue has not been re-opened since.


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