Skip to content

Support for individual border styles for each border side#13

Open
vpil wants to merge 1 commit into
maksimovic:masterfrom
vpil:master
Open

Support for individual border styles for each border side#13
vpil wants to merge 1 commit into
maksimovic:masterfrom
vpil:master

Conversation

@vpil

@vpil vpil commented Jun 5, 2026

Copy link
Copy Markdown

Should be backward compatible. Could be similarly extended to colors, but I did just what I needed for my project.

Comment thread src/XLSXWriter.php

$border_value['side'] = array_intersect(explode(",", $style['border']), $border_allowed);
if (isset($style['border-style']) && in_array($style['border-style'],$border_style_allowed))
if (isset($style['border-style']) && $bss = explode(",", $style['border-style']))

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

isset($style['border-style']) && is_string($style['border-style']) &&...

passing a non-string will make explode() to... explode with a type error

@maksimovic

Copy link
Copy Markdown
Owner

Thanks for the PR. Just add the minor check I proposed and it's good to go 🚀

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.

2 participants