Изменен код обработки полученных данных от api wg. Добавлена проверка на наличие данных в ответе api, помимо проверки статуса ответа. Исправлен редкий баг при получении данных. Баг возникал при некорректной работе api.

Подскажите в каком из коммитов реализованы эти вот изменения.
Мне в сугубо образовательных целях.

[Exinaus]

Подскажите в каком из коммитов реализованы эти вот изменения.
Мне в сугубо образовательных целях...

https://github.com/Edrard/MSFC/commit/2 ... 97fe59743a
Там много изменений, и я уже точно не скажу где конкретно и при каких условиях была ошибка, вполне даже возможно что ошибка могла появится в виде комбинации кода в нескольких местах.

Там не в апдейтере дело. Это я похоже в дамп забыл добавить параметр, хотя системную версию изменил, и при чистой установке оно так происходит. А апдейтер его не подхватит, т.к. он проверяет блоками, по системной версии.
В общем, я вечером скину пару способов исправить это.

Да, получается именно так. Причины возникновения ошибки все равно идентичные.

P.S:. Не обратил внимание на чистую установку, там ведь не сработает апдейт.

[Exinaus]

Изменен код обработки полученных данных от api wg. Добавлена проверка на наличие данных в ответе api, помимо проверки статуса ответа. Исправлен редкий баг при получении данных. Баг возникал при некорректной работе api.

Подскажите в каком из коммитов реализованы эти вот изменения.
Мне в сугубо образовательных целях...

Вспомнил. И понял почему не смог описать где в коде было это исправление.
Потому что в данном случае это две ошибки, которые были исправлены.

Первая:

Изменен код обработки полученных данных от api wg. Добавлена проверка на наличие данных в ответе api, помимо проверки статуса ответа.

Скришот изменений: http://prntscr.com/4oq65m
Я выделил где именно была ошибка, и как изменился новый код. Суть ошибки в том, что при получении данных, данные из первого метода (account/info) проверялись только на статус, но если данные поступали с правильным статусом, но данных могло не быть (глюки апи из-за большой нагрузки), то возникала ошибка т.к. не хватало базовых данных по аккаунту. Я стрелкой указал где добавил проверку на empty().
В остальных трех методах такой проверки не надо, т.к. там такая ситуация возможна (у игрока нет танков/наград/рейтингов).

Вторая:

Исправлен редкий баг при получении данных. Баг возникал при некорректной работе api.

Ошибка была в единственной переменной в function/func_get.php: http://prntscr.com/4oq92l
В случае ошибки при получении данных, статус ошибки присваивался везде, а не для группы ссылок где была ошибка.

PS Хотя на счет второго случая могу и ошибаться, т.к. или я ошибся описывая причину ошибки (там скорее была ошибка в коде а не некорректная работа апи), или там еще где то была похожая ошибка.

вчера установил(установилось без проблем), сегодня не обновило может из за выхода нового патча?

 Читать дальше

Код: Выделить всё2014-09-23 06:30////////////////////////////////////////////--->
2014-09-23 06:30: (Info) Loging Started (v. 310.7)
2014-09-23 06:30: (Info) Current run number 104
2014-09-23 06:30: (Info) Current db prefix: msfc_, clain ID: 6255
2014-09-23 06:30: (Err) Error while running cron task, in file: /var/www/shelofastov/data/www/rolex-wot.com/stats/function/func_cron.php on line: 86
2014-09-23 06:30: (Err) Error message: Duplicate entry 'T49' for key 'name_i18n'
2014-09-23 06:30: (Err) Error code: INSERT INTO `tanks` (`tank_id`, `nation_i18n`, `level`, `nation`, `is_premium`, `title`, `name_i18n`, `type`, `image`, `contour_image`, `image_small`) VALUES ('18209', 'США', '8', 'usa', '0',  'T49', 'T49', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-t49.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-t49.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-t49.png'), ('18689', 'СССР', '5', 'ussr', '0',  'KV-1s', 'КВ-1С', 'heavyTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-kv-1s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-kv-1s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-kv-1s.png'), ('58113', 'СССР', '6', 'ussr', '1',  'R108_T34_85M', 'R108 T34 85M', 'mediumTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-r108_t34_85m.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-r108_t34_85m.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-r108_t34_85m.png'), ('18433', 'СССР', '7', 'ussr', '0',  'R107_LTB', 'R107 LTB', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-r107_ltb.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-r107_ltb.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-r107_ltb.png'), ('16673', 'США', '6', 'usa', '0',  'T37', 'T37', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-t37.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-t37.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-t37.png'), ('18449', 'Германия', '8', 'germany', '0',  'G103_RU_251', 'G103 RU 251', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/germany-g103_ru_251.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/germany-g103_ru_251.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/germany-g103_ru_251.png'), ('65313', 'США', '1', 'usa', '0',  'M24_Chaffee_GT', 'M24 Chaffee GT', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-m24_chaffee_gt.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-m24_chaffee_gt.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-m24_chaffee_gt.png'), ('18177', 'СССР', '8', 'ussr', '0',  'R109_T54S', 'R109 T54S', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-r109_t54s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-r109_t54s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-r109_t54s.png'), ('17953', 'США', '7', 'usa', '0',  'M41_Bulldog', 'M41 Bulldog', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-m41_bulldog.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-m41_bulldog.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-m41_bulldog.png');
2014-09-23 06:40////////////////////////////////////////////--->
2014-09-23 06:40: (Info) Loging Started (v. 310.7)
2014-09-23 06:40: (Info) Current run number 104
2014-09-23 06:40: (Info) Current db prefix: msfc_, clain ID: 6255
2014-09-23 06:40: (Err) Error while running cron task, in file: /var/www/shelofastov/data/www/rolex-wot.com/stats/function/func_cron.php on line: 86
2014-09-23 06:40: (Err) Error message: Duplicate entry 'T49' for key 'name_i18n'
2014-09-23 06:40: (Err) Error code: INSERT INTO `tanks` (`tank_id`, `nation_i18n`, `level`, `nation`, `is_premium`, `title`, `name_i18n`, `type`, `image`, `contour_image`, `image_small`) VALUES ('18209', 'США', '8', 'usa', '0',  'T49', 'T49', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-t49.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-t49.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-t49.png'), ('18689', 'СССР', '5', 'ussr', '0',  'KV-1s', 'КВ-1С', 'heavyTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-kv-1s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-kv-1s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-kv-1s.png'), ('58113', 'СССР', '6', 'ussr', '1',  'R108_T34_85M', 'R108 T34 85M', 'mediumTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-r108_t34_85m.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-r108_t34_85m.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-r108_t34_85m.png'), ('18433', 'СССР', '7', 'ussr', '0',  'R107_LTB', 'R107 LTB', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-r107_ltb.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-r107_ltb.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-r107_ltb.png'), ('16673', 'США', '6', 'usa', '0',  'T37', 'T37', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-t37.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-t37.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-t37.png'), ('18449', 'Германия', '8', 'germany', '0',  'G103_RU_251', 'G103 RU 251', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/germany-g103_ru_251.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/germany-g103_ru_251.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/germany-g103_ru_251.png'), ('65313', 'США', '1', 'usa', '0',  'M24_Chaffee_GT', 'M24 Chaffee GT', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-m24_chaffee_gt.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-m24_chaffee_gt.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-m24_chaffee_gt.png'), ('18177', 'СССР', '8', 'ussr', '0',  'R109_T54S', 'R109 T54S', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/ussr-r109_t54s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/ussr-r109_t54s.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/ussr-r109_t54s.png'), ('17953', 'США', '7', 'usa', '0',  'M41_Bulldog', 'M41 Bulldog', 'lightTank', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/usa-m41_bulldog.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/contour/usa-m41_bulldog.png', 'https://worldoftanks.ru/static/2.14.0/encyclopedia/tankopedia/vehicle/small/usa-m41_bulldog.png');


[Exinaus]

В админке: "База данных" -> "Обновить информацию о технике"

ниче не делал уже такая ошибка посмотрите http://rolex-wot.com/stats/

[Exinaus]

ниче не делал уже такая ошибка посмотрите http://rolex-wot.com/stats/

Это из за раздвоения и переноса Т49. Постоянно такое, каждый раз новый сюрприз при таких перестановках танков. Предугадать что делать - просто нереально.
Самый простой способ все исправить:
В админке: "База данных" -> "Обновить информацию о технике"

А как с новыми должностями быть?

[Exinaus]

А как с новыми должностями быть?

Тут все не так просто. Данных о новых должностях пока нет в api: http://prntscr.com/4pl715
Как добавят - все исправим, я попрошу сделать релиз 3.1.2 - с исправлениями под этот патч.