CategoryPermissionHandler::getPermissions() nicht Typ-sicher

  • Affected App
    WoltLab Suite Core

    Hallo zusammen,

    laut Doc soll die Methode CategoryPermissionHandler::getPermissions() einen Integer-Array zurückliefern, Die Zeile $permissions[$optionName] = $permissions[$optionName] || $optionValue; sorgt aber dafür, dass immer wieder Boolean im Array vorkomme. Das führt zu unerwartet Ergebnissen.

    VG,

    Daniel

  • Die Permissions sind streng genommen booleans, auch wenn diese auf Grund der Beschränkung durch MySQL als 0/1 ausgedrückt werden. Wirklich korrekt wäre die Dokumentation nur, wenn wir sie als bool[] deklarieren und gleichzeitig explizit zu bool casten. Bis zum Wechsel auf strict_types halte ich eine Änderung aber für wenig zielführend, da "inkonsequent".

    Alexander Ebert
    Senior Developer WoltLab® GmbH

  • Alexander Ebert November 2, 2019 at 10:35 PM

    Added the Label Won’t fix

Participate now!

Don’t have an account yet? Register yourself now and be a part of our community!