Opened 12 years ago

Closed 12 years ago

#6928 closed defect (fixed)

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

Reported by: [email protected]… 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 12 years ago.

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by guest

Attachment: parser-empty-array.patch​ added

comment:1 Changed 12 years ago by bill

Cc: [email protected]… removed
Milestone: → 1.2
Reporter: changed from guest to [email protected]…
Status: new → assigned

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

comment:2 Changed 12 years ago by bill

Resolution: → fixed
Status: assigned → closed

(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.