Skip to main content

Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

    •  
      request #39732 Psalm should warn about EasyDB::insertMany empty arrays
    Actions
    Infos
    #39732
    Joris MASSON (jmasson)
    2024-09-27 14:19
    2024-09-27 10:34
    41359
    Details
    Psalm should warn about EasyDB::insertMany empty arrays

    EasyDB::insertMany() function is handy, but it throws an exception when given an empty array. Most people don't know this, and since we already have a Psalm stub for it, we should modify the stub so that Psalm will warn us that empty arrays aren't allowed.

    There are already about 25 errors for this, we should add them to our baseline and fix them gradually.

    Dev tools
    Empty
    Empty
    • [ ] enhancement
    • [ ] internal improvement
    Empty
    Stage
    Joris MASSON (jmasson)
    Closed
    2024-09-27
    Attachments
    Artifact links

    Releases

    Fixed in

    Artifact ID Project Tracker Summary Status Last Update Date Submitted By Assigned to
    rel #37898 Tuleap Releases 16.0 Delivered 2024-10-09 15:47 Manuel Vacelet (vaceletm)
    Empty
    References
    Referencing request #39732

    Git commit

    tuleap/tuleap/stable

    chore: EasyDB::insertMany accepts only non-empty arrays c76a46430b
    User avatar Joris MASSON (jmasson) , 2024-09-27 10:37
    Display settings

    Follow-ups