silverstripe / silverstripe-linkfield

Silverstripe module for managing links
BSD 3-Clause "New" or "Revised" License
4 stars 14 forks source link

`null` being passed into `ArrayList` sometimes #289

Closed GuySartorelli closed 4 months ago

GuySartorelli commented 4 months ago

Found when running silverstripe/admin behat in kitchen sink: https://github.com/creative-commoners/recipe-kitchen-sink/actions/runs/9201969934/job/25311038804

MultiLinkField::dataValue() method sometimes returns null, and that gets put into an ArrayList in LinkIDs(), which causes an error.

PRs

GuySartorelli commented 4 months ago

PR got merged