DRCIT/snipe-it
1
0
Fork 0
mirror of https://github.com/snipe/snipe-it.git synced 2025-01-12 14:27:33 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

adding in error logging and code comments

Browse source
This commit is contained in:
akemidx 2023-11-08 13:44:46 -05:00
parent 9cc89911f7
commit 2051ac785d
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
app/Helpers/Helper.php
View file

@ -349,13 +349,18 @@ class Helper
$total_colors = count($colors);
if ($index >= $total_colors) {
\Log::error('Status label count is '.$index.' and exceeds the allowed count of 256.');
//patch fix for array key overflow (color count starts at 1, array starts at 0)
$index = $index - $total_colors - 1;
//constraints to keep result in 0-265 range. This should never be needed, but if something happens
//to create this many status labels and it DOES happen, this will keep it from failing at least.
if($index < 0) {
$index = 0;
}
elseif($index > 255) {
$index = 255;
elseif($index > 265) {
$index = 265;
}
}