Skip to content
This repository was archived by the owner on Jan 2, 2019. It is now read-only.

Removed an error for zoom scale in favor of a default value. #1325

Closed
wants to merge 6 commits into from
Closed

Removed an error for zoom scale in favor of a default value. #1325

wants to merge 6 commits into from

Conversation

jleslie
Copy link

@jleslie jleslie commented Oct 5, 2017

This error prevents files from going through when in reality it might better to simply default the value on something like zoom scale. The end users of software made using PHPExcel will not know how to handle an error like this.

James Leslie and others added 6 commits October 5, 2017 11:34
…or prevents files from going through when in reality it might better to simply default the value on something like zoom scale. The end users of software made using PHPExcel will not know how to handle an error like this.
Preparing for Packagist
@PowerKiKi
Copy link
Member

Thank you for your contribution, however PHPExcel development for next version has moved under its new name PhpSpreadsheet. So please consider heading over to PhpSpreadsheet to contribute patches and features. Pull Requests will no longer be accepted for PHPExcel.

If you decide to submit a PR to PhpSpreadsheet, we should adopt a similar strategy than in PHPOffice/PhpSpreadsheet#186, that is keep the validation for normal usage, but allow a non-validated value only when reading files.

@PowerKiKi PowerKiKi closed this Oct 6, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants