search

iliaal / php_excel

PHP Extension interface to the Excel writing/reading library
http://ilia.ws
534 stars 131 forks source link

Data Validation #197

Closed pbalan closed 3 years ago

pbalan commented 7 years ago

Hi,

I wanted to ask if you plan to add data validation feature to this library?

https://github.com/PHPOffice/PHPExcel/blob/develop/Documentation/markdown/Overview/08-Recipes.md#setting-data-validation-on-a-cell

johmue commented 7 years ago

Hi,

It's a bit more tricky since we do not add features to the library itself. We just maintain a wrapper for the library which exposes the features to PHP via an extension. If you have feature requests you have to contact the libxl support directly.

johmue commented 7 years ago

Addressed in PR #199

doppiogancio commented 7 years ago

The library libxl has "data validation". @johmue can you wrap also the followings methods?

void xlSheetAddDataValidation(SheetHandle handle, int type, int op, int rowFirst, int rowLast, int colFirst, int colLast, const wchar_t* value1, const wchar_t* value2)
void xlSheetAddDataValidationEx(SheetHandle handle, int type, int op, int rowFirst, int rowLast, int colFirst, int colLast, const wchar_t* value1, const wchar_t* value2, int allowBlank, int hideDropDown, int showInputMessage, int showErrorMessage, const wchar_t* promptTitle, const wchar_t* prompt, const wchar_t* errorTitle, const wchar_t* error, int errorStyle)
void xlSheetAddDataValidationDouble(SheetHandle handle, int type, int op, int rowFirst, int rowLast, int colFirst, int colLast, double value1, double value2)
void xlSheetAddDataValidationDoubleEx(SheetHandle handle, int type, int op, int rowFirst, int rowLast, int colFirst, int colLast, double value1, double value2, int allowBlank, int hideDropDown, int showInputMessage, int showErrorMessage, const wchar_t* promptTitle, const wchar_t* prompt, const wchar_t* errorTitle, const wchar_t* error, int errorStyle)
void xlSheetRemoveDataValidations(SheetHandle handle)

Thanks in advance Fabrizio

johmue commented 7 years ago

That is already in PR #199 waiting to be merged by @iliaal

doppiogancio commented 7 years ago

@johmue thanks very much for the fast answer

xianve commented 7 years ago

Any update on when this will be merged?