Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DataValidator Does Not Handle All Types of List #4206

Open
1 of 8 tasks
oleibman opened this issue Oct 30, 2024 · 0 comments
Open
1 of 8 tasks

DataValidator Does Not Handle All Types of List #4206

oleibman opened this issue Oct 30, 2024 · 0 comments

Comments

@oleibman
Copy link
Collaborator

This is:

- [x] a bug report
- [ ] a feature request
- [ ] **not** a usage question (ask them on https://stackoverflow.com/questions/tagged/phpspreadsheet or https://gitter.im/PHPOffice/PhpSpreadsheet)

What is the expected behavior?

DataValidator::isValid calls DataValidator::isValueInList when appropriate. The expected behavior is that true/false will be returned depending on the cell's value vis-à-vis the list..

What is the current behavior?

The method expects the list to be either a comma-delimited string or a cell range. With the merge of PR #4203, the list can also be the result of a spill operator, which is implemented as a call to the ANCHORARRAY function. The validator function will not recognize this, and will therefore always return "true" in this situation. It also appears that a named range might be usable as a validator list, and that also won't be recognized.

What features do you think are causing the issue

  • Reader
  • Writer
  • Styles
  • Data Validations
  • Formula Calculations
  • Charts
  • AutoFilter
  • Form Elements

Does an issue affect all spreadsheet file formats? If not, which formats are affected?

The spreadsheet will work properly. It is only validation within PhpSpreadsheet itself that might not yield the correct answer.

Which versions of PhpSpreadsheet and PHP are affected?

All.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

1 participant