PHP8.4 "voku\helper\UTF8::reduce_string_array(): Implicitly marking parameter $remove_short_values as nullable is deprecated"

[Athar42]

Steps to reproduce

Hi,

Just wanted to report an error which is triggered by this app, seems to be related to a dependancy (once again :D ):

voku\helper\UTF8::reduce_string_array(): Implicitly marking parameter $remove_short_values as nullable is deprecated, the explicit nullable type must be used instead at /home/web/apps/mail/vendor/voku/portable-utf8/src/voku/helper/UTF8.php#13827

Just like AMP case, I don't have more info, but still wanted to share it there so you know it.

Also, this issue seems to rely on apps/mail, but it seems to get triggered on update contacts and calendar as those two apps also got updated at the same time (see additional info)

Thanks.

Expected behavior

Shouldn't get an error in the events log.

Actual behavior

Got an error in the events log after updating the mail app from 5.0.6 to 5.0.7.

Mail app version

5.0.7

Nextcloud version

31.0.4

Mailserver or service

No response

Operating system

Debian 12.10

PHP engine version

Other

Nextcloud memory caching

No response

Web server

Apache (supported)

Database

MariaDB

Additional info

PHP8.4

`[PHP] Erreur: voku\helper\UTF8::reduce_string_array(): Implicitly marking parameter $remove_short_values as nullable is deprecated, the explicit nullable type must be used instead at /home/web/apps/mail/vendor/voku/portable-utf8/src/voku/helper/UTF8.php#13827
GET /index.php/settings/apps/update/calendar
de X.Y.Z.Q par Me à 13 mai 2025, 23:31:55

`[PHP] Erreur: voku\helper\UTF8::reduce_string_array(): Implicitly marking parameter $remove_short_values as nullable is deprecated, the explicit nullable type must be used instead at /home/web/apps/mail/vendor/voku/portable-utf8/src/voku/helper/UTF8.php#13827
GET /index.php/settings/apps/update/contacts
de X.Y.Z.Q par Me à 13 mai 2025, 23:31:54

[ChristophWurst]

Thank you for your report.

This is also an upstream issue, unfortunately: voku/portable-utf8#217.

[AndyXheli]

Hey @ChristophWurst

Looks like in voku/portable-utf8#222 fixed this.

The code on https://github.com/voku/portable-utf8 looks good below

    private static function reduce_string_array(
        array $strings,
        bool $remove_empty_values,
        ?int $remove_short_values = null
    ) {

In Nextcloud shows following

    private static function reduce_string_array(
        array $strings,
        bool $remove_empty_values,
        int $remove_short_values = null

Looks like this line of code needs to be updated from int $remove_short_values = null to ?int $remove_short_values = null

Also seeing this under suspicious_login

voku\helper\UTF8::reduce_string_array(): Implicitly marking parameter $remove_short_values as nullable is deprecated, the explicit nullable type must be used instead at /var/www/nextcloud/apps/suspicious_login/vendor/voku/portable-utf8/src/voku/helper/UTF8.php#13827

[ChristophWurst]

Looks like in voku/portable-utf8#222 fixed this.

not merged?

The code on https://github.com/voku/portable-utf8 looks good below

It looks like the changes were merged but not released. https://packagist.org/packages/voku/portable-utf8 last release was two years ago.