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

Don't error when there are blank children elements. #487

Merged
merged 1 commit into from
Jan 18, 2019

Conversation

matzko
Copy link
Contributor

@matzko matzko commented Jan 18, 2019

Summary

In the scenario represented by the fixture added in test/files/blank_children.xlsx and in newly-added spec, the file fails to parse

Other Information

New spec fails without fix to SheetDoc

Thanks for contributing to Roo!

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 94.249% when pulling ed09cd1 on matzko:master into e7996b0 on roo-rb:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 94.249% when pulling ed09cd1 on matzko:master into e7996b0 on roo-rb:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.05%) to 94.249% when pulling ed09cd1 on matzko:master into e7996b0 on roo-rb:master.

Copy link
Member

@chopraanmol1 chopraanmol1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. LGTM.

@chopraanmol1 chopraanmol1 merged commit 2f94d63 into roo-rb:master Jan 18, 2019
@chopraanmol1
Copy link
Member

Thank you for submitting this PR.

@pwim
Copy link

pwim commented Jan 21, 2019

I just ran into this. @chopraanmol1 could you release a 2.8.1 containing this fix?

@chopraanmol1
Copy link
Member

@pwim
Copy link

pwim commented Jan 21, 2019

Awesome, thanks so much!

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this pull request Feb 6, 2019
##  [2.8.2] 2019-02-01
### Changed/Added
- Support range cell for Excelx's links [490](roo-rb/roo#490)
- Skip `extract_hyperlinks` if not required [488](roo-rb/roo#488)

### Fixed
- Fixed error for invalid link [492](roo-rb/roo#492)

##  [2.8.1] 2019-01-21
### Fixed
- Fixed error if excelx's cell have empty children [487](roo-rb/roo#487)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants