Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Переделать Состав+ на новый состав на матч #9

Closed
tconsts opened this issue Sep 7, 2016 · 9 comments

Comments

@tconsts
Copy link
Owner

tconsts commented Sep 7, 2016

проблема в том что текущие данные скрипт берет из старого источника (определенный файл вида .php), там данные берутся из пхп и парсятся загоняя их в js
необходимо переделать на новый источник (данные вида json), урл откуда брать можно взять из localStorage, ключ вида sostavurl1432 где 1432 id команды
урл туда записывается когда открывается страница выставления состава
ну и плюс надо не только их получить, но и преобразоваться в нужный вид естественно

@ba11ak ba11ak self-assigned this Sep 7, 2016
@ba11ak ba11ak added the bug label Sep 7, 2016
@ba11ak
Copy link
Collaborator

ba11ak commented Sep 7, 2016

Зашел. Разобраться сложно, но понятно с чего начинать )

@ba11ak
Copy link
Collaborator

ba11ak commented Sep 7, 2016

Сейчас берет из http://pefl.ru/fieldnew3.php
надо чтобы брал из
http://pefl.ru/jsonsostav.php?j=795&u=9724&r=281625&m=get&z=7e3e968223e55ead0d1544cdf019f8f2
пока что неясно откуда брать эти параметры
Как вариант - скачать страничку состава на матч и там получить и json url.

@tconsts
Copy link
Owner Author

tconsts commented Sep 8, 2016

яж написал что сейчас этот урл откладывается в localStorage - там урл и взять надо
ну и формат другой - я частично начал это делать, но не закончил

ba11ak pushed a commit that referenced this issue Sep 13, 2016
-- Если будет время позже, можно будет все переделать под новый состав
@ba11ak ba11ak added the FIXED label Sep 13, 2016
@ba11ak
Copy link
Collaborator

ba11ak commented Sep 13, 2016

Сделал quickfix - теперь отображаются правильные изначальные позиции.
Для того чтобы полностью переделать надо по-больше времени. Хорошо что хоть понял где надо было фиксить.

@ba11ak
Copy link
Collaborator

ba11ak commented Sep 13, 2016

Конечно если ссылка на старый состав будет удалена то ничего уже не загрузится, но пока что не вижу смысла девить лишь для того чтобы был чистый код, есть фишки по-важнее )

@tconsts
Copy link
Owner Author

tconsts commented Sep 14, 2016

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

@tconsts tconsts removed the FIXED label Sep 14, 2016
@tconsts
Copy link
Owner Author

tconsts commented Sep 14, 2016

хм, и еще...
у нас по коду идет несколько вызовов GetData('positions')
к вызову добавлен obj тока при одном вывозе, в остальных без него(будет undefined)
а в самой функции GetData этот obj пробрасывается в FillHeaders(obj)
а в FillHeaders obj не проверяется и сразу идет for (j in obj.sostav1)

PS для примера получаем ексепшин при смене кофоф

@ba11ak
Copy link
Collaborator

ba11ak commented Sep 14, 2016

Ага, понял. Я проверил только основной сценарий, не знаю какие там еще бывают )

@tconsts
Copy link
Owner Author

tconsts commented Sep 15, 2016

опубликовал в магазинах, вошло в версию 2.0.5

@tconsts tconsts closed this as completed Sep 15, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants