Bug 56888

Summary: [PATCH] XSSFDataValidation ignores "allow blank" read from sheet, assumes true
Product: POI Reporter: Chris Boyle <cmb-apache>
Component: XSSFAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: normal Keywords: PatchAvailable
Priority: P2    
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Attachments: Patch to not overwrite allow blank value

Description Chris Boyle 2014-08-26 12:10:36 UTC
Created attachment 31943 [details]
Patch to not overwrite allow blank value

XSSFDataValidation.XSSFDataValidation(CellRangeAddressList, CTDataValidation) calls ctDdataValidation.setAllowBlank(true), overwriting the original value read from the sheet. That value is therefore unavailable.

Attached is a patch from a colleague, with tests, to call setAllowBlank(true) in the DataValidationHelper instead, so it still defaults to true for write, but allows reading the actual value from the sheet.
Comment 1 Dominik Stadler 2014-12-22 10:03:55 UTC
Thanks for the Patch, this is applied via r1647276 now.