Skip to content

Commit fe4252e

Browse files
Alexey PortnovAlexey Portnov
authored andcommitted
Исправлен краш AmoCdrDaemon при пустом массиве records
isset() → !empty() в getCreateFileAndLink() для случая когда CDR не имеет файлов записей (file_exists не прошёл). Пустой массив проходил isset() и падал на доступе к [0]. Краш вызывал цепочку перезапусков демона и дублирование задач в AmoCRM.
1 parent 1c2121c commit fe4252e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

bin/AmoCdrDaemon.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -733,7 +733,7 @@ private function reduceCdr($calls):array
733733
private function getCreateFileAndLink(string $id, int $created_at):string
734734
{
735735
$link = '';
736-
if(isset($this->cdrRows[$id]['records'])){
736+
if(!empty($this->cdrRows[$id]['records'])){
737737
if(count($this->cdrRows[$id]['records']) === 1){
738738
$fileName = $this->cdrRows[$id]['records'][0];
739739
}else{

0 commit comments

Comments
 (0)