MrLoick / any-balance-providers

Automatically exported from code.google.com/p/any-balance-providers
2 stars 3 forks source link

[ab-cell-mts-ua]Исправил распознавание счетчиков на сайте для тарифа MAX Energy #232

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Провайдер ab-cell-mts-ua
Исправил распознавание счетчиков на сайте 
для тарифа MAX Energy (они несколько отличаются 
от тарифа MAX Energy Allo):
 - traffic_maxenergy_mb
 - sms_net
 - min_net_maxenergy
Добавил счетчик "50 минут на все сети"
- min_all_50

Original issue reported on code.google.com by panchenk...@gmail.com on 12 Feb 2015 at 3:57

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by panchenk...@gmail.com on 12 Feb 2015 at 4:01

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by panchenk...@gmail.com on 12 Feb 2015 at 4:02

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by panchenk...@gmail.com on 12 Feb 2015 at 4:14

Attachments:

GoogleCodeExporter commented 9 years ago
http://anybalance.ru/catalog.php?id=ab-cell-mts-ua&inapp=0

Original comment by stark...@gmail.com on 16 Feb 2015 at 12:55

GoogleCodeExporter commented 9 years ago
0. Провайдер очень капризный, лучше 
распознавать более конкретно, потому, что 
может подцепить другие счетчики. МТС 
конечно изменил отображение в последнее 
время, но лучше не рисковать.
1. traffic_maxenergy_mb
Лучше исправить строчку, которая 
определяет мегабайты для MAX Energy Allo 
одновременно и сделать, что-то типа такого, 
что бы опрелделяло и для MAX Energy:
sumParam (html, result, 'traffic_maxenergy_mb', /<li>(?:15|30)00 Mb GPRS 
Internet для (?:MAX Energy Allo|MAX Energy), осталось (\d+) 
бесплатных Kб.[^<]*<\/li>/ig, null, parseTrafficMb, aggregate_sum);
2. Тоже самое и для sms_net
sumParam (html, result, 'sms_net', /<li>1500 SMS на МТС для (?:MAX 
Energy Allo|MAX Energy), осталось (\d+) бесплатных 
SMS<\/li>/ig, replaceTagsAndSpaces, parseBalance, aggregate_sum);
3. min_net_maxenergy Наверно да. Можно было бы 
объединить с Allo, но там из-за предыдущих 
косяков версии интернет-помощника 
пришлось делать два счетчика. А сейчас 
удалять один счетчик - плохо.
4. min_all_50 Если вводить новый счетчик, то 
назвать его нужно "Минуты MAX Energy по Украине". 
"На другие сети", это типа на городские, и 
будет путаница. А вообще лучше не вводить 
новый счетчик, а использовать старый. Я не 
помню, парсился ли он раньше, поэтому лучше 
в счетчик min_allo_other и переименовать его в 
манифесте на "Минуты по Украине линейки 
тарифов MAX Energy"

Как-то так.

Original comment by igoloviz...@gmail.com on 16 Feb 2015 at 1:39

GoogleCodeExporter commented 9 years ago
1,2 - исправлю
4 - это отдельный пакет, не имеет отношения к 
MAX Energy (или Max Allo)
http://www.mts.ua/ru/services/dlya-ekonomii/zvonki/50-minut-abonentam-po-ukraine

Original comment by panchenk...@gmail.com on 16 Feb 2015 at 1:48

GoogleCodeExporter commented 9 years ago
4 - это отдельный пакет, не имеет отношения к 
MAX Energy (или Max Allo)
http://www.mts.ua/ru/services/dlya-ekonomii/zvonki/50-minut-abonentam-po-ukraine

Original comment by panchenk...@gmail.com on 16 Feb 2015 at 7:10