View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000657 | XMB1 | Bugs | public | 2021-04-11 21:01 | 2021-05-02 08:01 |
| Reporter | miqrogroove | Assigned To | miqrogroove | ||
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Target Version | 1.9.12.03 | Fixed in Version | 1.9.12.03 | ||
| Summary | 0000657: String-to-Number Comparisons in PHP 8 | ||||
| Description | One of the problems with XMB in PHP 8 is the unexpected result of comparing zero to an empty string. Before PHP 8, (0 == "") is true, but now in PHP 8 it is false. This subtle change is screwing up PHP's non-strict variable types in places like index.php where we have: if ($lastcat != $thing['cat_fid'] ... In this example the $lastcat is initialized to zero, and $thing['cat_fid'] is explicitly set to an empty string inside function getIndexForums(), which is unfortunate in light of the complete opposite results in different PHP versions now. | ||||
| Tags | No tags attached. | ||||
| MySQL Version | |||||
| PHP Version | 8.0 | ||||
| Web Server | |||||
| Browser | |||||
| Flags | |||||
| Original Reporter | |||||
| SVN Revision | 2989 | ||||
| Git Commit | |||||
| child of | 0000649 | closed | miqrogroove | PHP 8 Compatibility |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-04-11 21:01 | miqrogroove | New Issue | |
| 2021-04-11 21:02 | miqrogroove | Relationship added | child of 0000649 |
| 2021-04-11 21:02 | miqrogroove | Assigned To | => miqrogroove |
| 2021-04-11 21:02 | miqrogroove | Status | new => acknowledged |
| 2021-04-12 06:11 | miqrogroove | Status | acknowledged => assigned |
| 2021-04-13 17:39 | miqrogroove | Status | assigned => resolved |
| 2021-04-13 17:39 | miqrogroove | Resolution | open => fixed |
| 2021-04-13 17:39 | miqrogroove | Fixed in Version | => 1.9.12.03 |
| 2021-04-13 17:39 | miqrogroove | SVN Revision | => 2989 |
| 2021-05-02 08:01 | miqrogroove | Status | resolved => closed |