Opened 11 years ago

Closed 11 years ago

#6928 closed defect (fixed)

[CLA / patch] fix parser so it handles empty arrays

Reported by: its.jeff.balogh@… Owned by: bill
Priority: high Milestone: 1.2
Component: Parser Version: 1.1.1
Keywords: Cc:
Blocked By: Blocking:

Description

Currently when the parser encounters an empty string ("") that is meant for an array element, it turns it into an array containing the empty string. This patch returns an empty array if the string evaluates to false, with a test.

I have a CLA on file (Jeff Balogh).

Attachments (1)

parser-empty-array.patch (1.4 KB) - added by guest 11 years ago.

Download all attachments as: .zip

Change History (3)

Changed 11 years ago by guest

Attachment: parser-empty-array.patch added

comment:1 Changed 11 years ago by bill

Cc: its.jeff.balogh@… removed
Milestone: 1.2
Reporter: changed from guest to its.jeff.balogh@…
Status: newassigned

Good catch, I'll check this in. Thanks (and thanks for the unit test update too).

comment:2 Changed 11 years ago by bill

Resolution: fixed
Status: assignedclosed

(In [13971]) Fixes #6928: fix parser so it handles empty arrays. Patch from Jeff Balogh (CLA on file)

Note: See TracTickets for help on using tickets.