(Trac:429) В отчете "Статитстика по посещаемым сайтам" неверно расчитывается суммарный трафик

[GoogleCodeExporter]

(Trac:429) Originaly reported by iSa on 2010-01-12 19:09:39

  Суммарный трафик в отчете "Статитстика по посещаемым сайтам" расчитывается по формуле:
r_size + r_hit = r_sum,
что является неверным, так как r_size - 
суммарное кол-во байт, полученное 
пользователем, а r_hit - кол-во байт из r_size, 
взятые из кеша. Таким образом правильная 
формула должна выглядеть так:
r_size - r_hit = r_sum, при realtraffic=real
или
r_size = r_sum, при realtraffic=full.

В аттаче патч для исправления.

Original issue reported on code.google.com by bl...@nixdev.net on 5 Jun 2011 at 8:07

[GoogleCodeExporter]

Commented by blaze@nixdev.net on 2010-03-16 21:06:43:

Thanks. Fixed in [798]

Original comment by bl...@nixdev.net on 5 Jun 2011 at 8:07

[GoogleCodeExporter]

Commented by blaze@nixdev.net on 2010-07-30 08:30:18:

В версии из транка приходится в 
usersbuttom_3_site.php возвращать вышеуказанные 
изменения взад, иначе неправильно 
формируется статистика в общем списке 
пользователей.

В настройках трафик "Реальный (Полученный 
прокси-сервером)".

Original comment by bl...@nixdev.net on 5 Jun 2011 at 8:07

[GoogleCodeExporter]

Commented by blaze@nixdev.net on 2010-07-30 09:13:29:

Replying to [comment:2 hangover <root@…>]:
> В версии из транка приходится в 
usersbuttom_3_site.php возвращать вышеуказанные 
изменения взад, иначе неправильно 
формируется статистика в общем списке 
пользователей.
> 
> В настройках трафик "Реальный (Полученный 
прокси-сервером)".

А можно подробнее? Я посмотрел еще раз на 
исправление - оно корректное. Другой вопрос 
- что в некоторых других отчетах у нас 
возможно некорректно учитывается 
реальный/кешированный трафик.

Original comment by bl...@nixdev.net on 5 Jun 2011 at 8:07

[GoogleCodeExporter]

Commented by blaze@nixdev.net on 2010-07-30 09:29:05:

В версии из вчерашнего транка формирую 
отчёт "Статистика по посещаемым сайтам" по 
одному пользователю - всё правильно: 
"Размер"+"Из кэша"="Суммарный трафик"

Перехожу в общий список "Пользователи", 
выбираю период и формирую тот же самый 
отчёт - неправильно:
значение в столбце "Размер" равно значению 
"Суммарный трафик".

Вношу изменения в usersbuttom_3_site.php:

{{{
--- usersbuttom_3_site.php.old  2010-07-30 12:24:24.000000000 +0700
+++ usersbuttom_3_site.php  2010-07-30 12:26:40.000000000 +0700
@@ -203,10 +203,10 @@
            $URL["norm_url"][$count]=str_replace("/","",$row['url_domain']);
            $URL["url_size"][$count]=$row['url_size'];
            $URL["hit_size"][$count]=$row['hit_size'];
-           if($SAMSConf->realtraffic=="real")
-               $URL["sum_size"][$count]=$row['url_size']-$row['hit_size'];
-           else
-               $URL["sum_size"][$count]=$row['url_size'];
+//         if($SAMSConf->realtraffic=="real")
+               $URL["sum_size"][$count]=$row['url_size']+$row['hit_size'];
+//         else
+//             $URL["sum_size"][$count]=$row['url_size'];
            $count++;
        }
        }

}}}

Теперь отчёт отображается корректно: 
"Размер"+"Из кэша"="Суммарный трафик"

Original comment by bl...@nixdev.net on 5 Jun 2011 at 8:07

[GoogleCodeExporter]

Commented by blaze@nixdev.net on 2010-07-30 11:02:56:

Replying to [comment:4 hangover <root@…>]:
> В версии из вчерашнего транка формирую 
отчёт "Статистика по посещаемым сайтам" по 
одному пользователю - всё правильно: 
"Размер"+"Из кэша"="Суммарный трафик"
> 
> Перехожу в общий список "Пользователи", 
выбираю период и формирую тот же самый 
отчёт - неправильно:
> значение в столбце "Размер" равно значению 
"Суммарный трафик".
> 
> Теперь отчёт отображается корректно: 
"Размер"+"Из кэша"="Суммарный трафик"

Причина этого поведения описана в баге #497. 
На самом деле переменная $SAMSConf->realtraffic 
всегда пустая, т.к. не инициализируется в 
samsclass.php.

Поэтому текущая реализация отчета для 
пользователей правильная, а вот для 
конкретного пользователя - некорретно 
реализовано.

Тикет оставляю открытым чтобы исправить 
отчет для конкретного пользователя. Также 
необходимо исправить #497 чтобы все стало 
работать корретно.

Original comment by bl...@nixdev.net on 5 Jun 2011 at 8:07

[GoogleCodeExporter]

Commented by blaze@nixdev.net on 2010-07-30 11:06:40:

Fixed in [840], require fix #497 to work correctly.

Original comment by bl...@nixdev.net on 5 Jun 2011 at 8:08