cobub / razor

Cobub Razor - Open Source Mobile Analytics Solution
http://www.cobub.com
1.13k stars 431 forks source link

“事件列表”的业务逻辑有个小BUG,count和eventsk未清零 #71

Closed zzs1949 closed 11 years ago

zzs1949 commented 11 years ago

application/models/event/userevent.php

使用贵产品时,发现了BUG,某事件本该为无值,但由于在列表中位于他上方的项有值,导致此项错误的显示了count 代码改起来很简单,只需要在第76行,加上此下二项赋值即可。 $count=0;$eventsk=0;

代码如下,可以较清晰的看到,$count和$eventsk会在条件不成立时,保留之前的值(本该为0)

请知悉,并愿产品继续完善,支持ing……

【代码:】

for($i=0;$i<count($identifierarray);$i++) { if(count($eventlistarray)!=0) { for($j=0;$j<count($eventlistarray);$j++) { if($identifierarray[$i]['eventidentifier']==$eventlistarray[$j]['eventidentifier']) { $count=$eventlistarray[$j]['count']; $eventsk=$eventlistarray[$j]['event_sk']; break; } }
}
$eventobj=array( 'event_sk'=>$eventsk, 'eventidentifier'=>$identifierarray[$i]['eventidentifier'], 'eventname'=>$identifierarray[$i]['eventname'], 'count'=>$count );//此处为第75行 array_push($eventresult, $eventobj); }