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 chained border operations on cell ranges operating on last cell only #578

Merged
merged 1 commit into from
Jul 15, 2018
Merged

Fix chained border operations on cell ranges operating on last cell only #578

merged 1 commit into from
Jul 15, 2018

Conversation

einars
Copy link
Contributor

@einars einars commented Jul 2, 2018

This is:

  • a bugfix

Checklist:

Why this change is needed?

Fix issue where chained operations on cell ranges involving borders seemed to operate on last cell only (issue #428).

@einars
Copy link
Contributor Author

einars commented Jul 11, 2018

@PowerKiKi, would you care to take a look at this trivial fix? Cheers.

Copy link
Member

@PowerKiKi PowerKiKi left a comment

Choose a reason for hiding this comment

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

If you can fix the very minor comment and rebase, then I'll merge it

CHANGELOG.md Outdated
@@ -16,6 +16,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
- Xlsx reader crashed when reading a file with workbook protection - [#553](https://github.com/PHPOffice/PhpSpreadsheet/pull/553)
- Cell formats with escaped spaces were causing incorrect date formatting - [#557](https://github.com/PHPOffice/PhpSpreadsheet/issues/557)
- Could not open CSV file containing HTML fragment - [##564](https://github.com/PHPOffice/PhpSpreadsheet/issues/564)
- Chained operations on cell ranges involving borders operated on last cell only [##428](https://github.com/PHPOffice/PhpSpreadsheet/issues/428)
Copy link
Member

Choose a reason for hiding this comment

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

Should be a single #

@einars
Copy link
Contributor Author

einars commented Jul 11, 2018

ok, all should be fine now

@PowerKiKi PowerKiKi merged commit b05d07a into PHPOffice:develop Jul 15, 2018
Dfred pushed a commit to Dfred/PhpSpreadsheet that referenced this pull request Nov 20, 2018
@einars einars deleted the border-range-fix branch September 30, 2021 19:23
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.

2 participants