• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

PHPOffice / PHPWord / 10850830346
97%

Build:
DEFAULT BRANCH: master
Ran 13 Sep 2024 02:37PM UTC
Jobs 1
Files 297
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

13 Sep 2024 02:19PM UTC coverage: 96.918% (+0.002%) from 96.916%
10850830346

push

github

web-flow
fix: Allow vAlign and vMerge on Style\Cell to be set to null (#2676)

* fix: Allow vAlign and vMerge on Style\Cell to be set to null (#2673)

vAlign and vMerge are initialized as `null` in every style until it is explicitly set.
Right now, it is not possible to unset them, after it has been set once.

I've added a null-check to skip the validation, based on the default
parameter value of `null`, which indicates to me that it once was intended to work like this.

I've also fixed the type-hints, which were wrong from the start.

* docs(changelog): Add note about supporting vAlign and vMerge to be unset

* test: Extend existing table test case to check if unsetting vAlign works

* test: Add test case to check if unsetting vMerge works

This should fix the reduction in test coverage because of the
new null checking code in the Setter

6 of 6 new or added lines in 1 file covered. (100.0%)

11886 of 12264 relevant lines covered (96.92%)

33.19 hits per line

Jobs
ID Job ID Ran Files Coverage
1 10850830346.1 13 Sep 2024 02:37PM UTC 297
96.92
GitHub Action Run
Source Files on build 10850830346
  • Tree
  • List 297
  • Changed 39
  • Source Changed 0
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Repo
  • f0ec497c on github
  • Prev Build on master (#10847418400)
  • Next Build on master (#10853857010)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2025 Coveralls, Inc