Skip to content

fix frontend#696

Merged
Al-p-i merged 1 commit intorybalkinsd:masterfrom
dm1tryK:master_fix
Dec 11, 2017
Merged

fix frontend#696
Al-p-i merged 1 commit intorybalkinsd:masterfrom
dm1tryK:master_fix

Conversation

@dm1tryK
Copy link
Copy Markdown
Contributor

@dm1tryK dm1tryK commented Dec 10, 2017

3 фикса фронтенда. Возникали ошибки при работе в Firefox 56.0 и Chrome 62.0, в других не тестил, но скорее всего то же самое.

  1. Message.js, функция handleReplica.
    Поскольку js воспринимает часть приходящего json не как строку, а как объект, то попытка парсить из несуществующей строки в объект все ломает.
  2. Message.js, функция handleTile. В моем понимании, этот метод размещает статические объекты (стены и коробки как минимум), но делает это отличным от размещения игроков, огня и бомб способом. Как результат, стены и коробки ставятся в некорректные позиции.
  3. ProxyServer.js, функция getSessionIdFromMatchMaker. processData: false запрещает преобразование строки формата key1=value1&key2=value2 в строку {key1: 'value1', key2: 'value2'}. Казалось бы это то, что действительно надо сделать, но почему-то на бэкэнд в таком случае приходят некорректные данные. Возможно потому, что стоит изменить настройку contentType с application/x-www-form-urlencoded на что-то другое (так пишут в интернетах), но точно сказать не могу.

Copy link
Copy Markdown
Collaborator

@Al-p-i Al-p-i left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Молодец!

@Al-p-i Al-p-i merged commit 07487c0 into rybalkinsd:master Dec 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants