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

fix error in rendering multiple rowspan #878

Closed
wants to merge 5 commits into from

Conversation

vipmaa1
Copy link

@vipmaa1 vipmaa1 commented Feb 11, 2019

This is: fixing bug in rendering html file having multiple rowspan.
Error message

A non well formed numeric value encountered

- [X] a bugfix

Checklist:

Why this change is needed?

fixing bug in rendering html file having multiple rowspan, in normal case it's showing error "A non well formed numeric value encountered" because variable $attributeArray['rowspan'] convert to string and not able to run for loop without recasting it as integer

@stale
Copy link

stale bot commented Apr 18, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If this is still an issue for you, please try to help by debugging it further and sharing your results.
Thank you for your contributions.

@stale stale bot added the stale label Apr 18, 2019
@stale stale bot closed this Apr 25, 2019
@PowerKiKi PowerKiKi reopened this Jul 27, 2019
@stale stale bot removed the stale label Jul 27, 2019
@PowerKiKi PowerKiKi closed this in 785705b Jul 28, 2019
@PowerKiKi
Copy link
Member

Thank you for your PR and sorry for the long wait. It has been partially merged in 785705b

PowerKiKi added a commit that referenced this pull request Aug 17, 2019
1.9.0

### Added

- When <br> appears in a table cell, set the cell to wrap [#1071](#1071) and [#1070](#1070)
- Add MAXIFS, MINIFS, COUNTIFS and Remove MINIF, MAXIF [#1056](#1056)
- HLookup needs an ordered list even if range_lookup is set to false [#1055](#1055) and [#1076](#1076)
- Improve performance of IF function calls via ranch pruning to avoid resolution of every branches [#844](#844)
- MATCH function supports `*?~` Excel functionality, when match_type=0 [#1116](#1116)
- Allow HTML Reader to accept HTML as a string [#1136](#1136)

### Fixed

- Fix to AVERAGEIF() function when called with a third argument
- Eliminate duplicate fill none style entries [#1066](#1066)
- Fix number format masks containing literal (non-decimal point) dots [#1079](#1079)
- Fix number format masks containing named colours that were being misinterpreted as date formats; and add support for masks that fully replace the value with a full text string [#1009](#1009)
- Stricter-typed comparison testing in COUNTIF() and COUNTIFS() evaluation [#1046](#1046)
- COUPNUM should not return zero when settlement is in the last period [#1020](#1020) and [#1021](#1021)
- Fix handling of named ranges referencing sheets with spaces or "!" in their title
- Cover `getSheetByName()` with tests for name with quote and spaces [#739](#739)
- Best effort to support invalid colspan values in HTML reader - [#878](#878)
- Fixes incorrect rows deletion [#868](#868)
- MATCH function fix (value search by type, stop search when match_type=-1 and unordered element encountered) [#1116](#1116)
- Fix `getCalculatedValue()` error with more than two INDIRECT [#1115](#1115)
- Writer\Html did not hide columns [#985](#985)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants