| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
| |
Thanks Junie AI
|
| |
|
|
|
|
|
|
|
|
| |
There was a mess of functions which mostly, but not quite, did the same
things. Get rid of a couple of them and fix call sites to use
alternative ones that also fit the job.
While at it, add phpdoc and comments to the remaining functions, trying
to clarify what they were designed for.
Lastly, the return type of functions that retrieve a location id has
been changed from false|int to ?int (nullable types are just nicer).
|
| | |
|
| | |
|
| |
|
|
| |
Mostly redundant checks, logic errors, dead code, etc.
|
| |
|
|
|
|
| |
Hide the button entirely if there are no movable machines in the
displayed list. Disable the button as long as the user selected at least
one unmovable machine.
|
| |
|
|
|
|
|
| |
* Auto-convert to utf8mb4_unicode_520_ci
* Use foreach instead of while to loop over results
* Drop useless statement caching
* Keep emulated prepares, as we sometimes loop over nested queries
|
| | |
|
| | |
|
|
|
* Casts are faster than settype.
* Check if assoc location cache is already populated.
* getLocationRootChain can use the parents array from
that location instead of building the array from scratch.
|