[profiel] onbeoordeelde reviews geven geen 0

Pagina: 1
Acties:

Acties:
  • 0 Henk 'm!

  • 5pë©ïàál_Tèkén
  • Registratie: April 2008
  • Laatst online: 12:57
Afbeeldingslocatie: https://tweakers.net/i/vgeeElR-ja-yYP7ppLpNsm9o0oY=/800x/filters:strip_exif()/f/image/S5EXdHF7Msw2AJPniD43B84o.png?f=fotoalbum_large

https://tweakers.net/productreview/user/259786/

Mijn bovenste review heeft een +2 score, de rest van de reviews heeft geen score. Echter, zou daar dan niet een grijze nul moeten staan, zoals bij ongemodereerde reacties op de FP?
Zowel in- als uitgelogd met Firefox als met Opera.

[ Voor 4% gewijzigd door 5pë©ïàál_Tèkén op 09-05-2022 18:22 ]

Team anti-scalp - wauw..


Acties:
  • 0 Henk 'm!

  • XWB
  • Registratie: Januari 2002
  • Niet online

XWB

Devver
Fix komt er aan :)

March of the Eagles


Acties:
  • +1 Henk 'm!

  • XWB
  • Registratie: Januari 2002
  • Niet online

XWB

Devver
Opgelost :)

March of the Eagles


Acties:
  • 0 Henk 'm!

  • 5pë©ïàál_Tèkén
  • Registratie: April 2008
  • Laatst online: 12:57
@XWB Wat was het probleem, en hoe lang was dit probleempje er al/pas?

Team anti-scalp - wauw..


Acties:
  • +1 Henk 'm!

  • XWB
  • Registratie: Januari 2002
  • Niet online

XWB

Devver
Vanwege de aanstaande PHP 8.1 upgrade hebben we recent veel deprecation meldingen opgelost. Denk aan:

PHP:
1
2
3
4
5
6
function getRating($value)
{
    $value = round($value);
}

getRating(null);


Gooit voortaan deze melding:
PHP Deprecated: round(): Passing null to parameter #1 ($num) of type int|float is deprecated
Meer uitleg: https://wiki.php.net/rfc/...ll_to_scalar_internal_arg

Dat had iemand opgelost door een null waarde niet door de round() functie te halen, maar daar had dan "return 0" in plaats van "return null" moeten staan.

PHP:
1
2
3
4
5
6
7
8
9
10
11
function getRating(?float $value)
{
    if ($value === null) {
        return null; // ai
        return 0; // fix
    }

    $value = round($value);
}

getRating(null);


De pagina waar je naar linkt betreft een oud stuk code waar geen unit testing voor beschikbaar is, waardoor dit soort foutjes er sneller inslopen.

March of the Eagles


Acties:
  • +1 Henk 'm!

  • DaFeliX
  • Registratie: December 2002
  • Laatst online: 12:43

DaFeliX

Tnet Devver
XWB schreef op woensdag 11 mei 2022 @ 21:48:
[...]

De pagina waar je naar linkt betreft een oud stuk code waar geen unit testing voor beschikbaar is, waardoor dit soort foutjes er sneller inslopen.
voeg dan gelijk een return-type declaration toe ;)

Einstein: Mijn vrouw begrijpt me niet

Pagina: 1