Project

General

Profile

Actions

Bug #983

closed

Отсутствие проверки успешности смены ника для его подсветки.

Added by Mellon almost 12 years ago. Updated almost 12 years ago.

Status:
Invalid
Priority:
Normal
Assignee:
Category:
Plugins: Azoth
Target version:
-
Start date:
03/12/2013
Due date:
% Done:

100%

Estimated time:
Reported in:
master

Description

Как должно быть.
0. Имеем ник foo, сообщение подсвечивается при его упоминании.
1. /nick bar
2. Проверяем ответ от сервера. Если успешно, то
3. Меняем фильтр подсветки, чтоб подсвечивались сообщения содержащие bar

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

Actions #1

Updated by 0xd34df00d almost 12 years ago

  • Status changed from New to Invalid
  • % Done changed from 0 to 100

А вот фиг, уже есть проверка. Например, попытался сменить ник на ник другого участника — получил станзу

  <presence from="c_plus_plus@conference.jabber.ru/0xd34df00d" type="error" to="0xd34df00d@ibash.im/Azoth">
    <status>Я на связи!</status>
    <priority>5</priority>
    <x xmlns="vcard-temp:x:update">
      <photo>31b531601bdd5cc293bc7985ec6d690eff4e8114</photo>
    </x>
    <c node="http://leechcraft.org/azoth" xmlns="http://jabber.org/protocol/caps" ver="Ye206ImrUeKNroFjnnK36wypUBo=" hash="sha-1"/>
    <error type="cancel" code="409">
      <conflict xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"/>
      <text xmlns="urn:ietf:params:xml:ns:xmpp-stanzas">Этот псевдоним уже занят другим участником</text>
    </error>
  </presence>

и ник не сменился, хайлайт на старый, етц.

Actions #2

Updated by Mellon almost 12 years ago

станзами мерятся - ок.

→→→→→→ [13:51:52.339] →→→→→→
  <presence to="leechcraft@conference.jabber.ru/Z͉̮̦̬͎̞̥͍̋͊̔̍͆̏̑ͦa͔̞̎ͤͣ̓ͧl̪̜͌̅ͤ̌͑ͅg͇͎͙̲̩͍͚̟̩̀̓̑͒ͫ̃ͭ́o̲ͬͦ͗ͩ̇̎̚">
    <priority>5</priority>
    <x xmlns="vcard-temp:x:update">
      <photo>7158a24d46a3e9d336fd12e29464e3bc97abeba6</photo>
    </x>
    <c node="http://leechcraft.org/azoth" xmlns="http://jabber.org/protocol/caps" ver="C7SA71jUdquHVPpVoQJsupoOqhQ=" hash="sha-1"/>
  </presence>

←←←←←← [13:51:53.230] ←←←←←←
  <presence from="leechcraft@conference.jabber.ru/Z͉̮̦̬͎̞̥͍̋͊̔̍͆̏̑ͦa͔̞̎ͤͣ̓ͧl̪̜͌̅ͤ̌͑ͅg͇͎͙̲̩͍͚̟̩̀̓̑͒ͫ̃ͭ́o̲ͬͦ͗ͩ̇̎̚" type="error">
    <priority>5</priority>
    <x xmlns="vcard-temp:x:update">
      <photo>7158a24d46a3e9d336fd12e29464e3bc97abeba6</photo>
    </x>
    <c node="http://leechcraft.org/azoth" xmlns="http://jabber.org/protocol/caps" ver="C7SA71jUdquHVPpVoQJsupoOqhQ=" hash="sha-1"/>
    <error type="modify" code="400">
      <jid-malformed xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"/>
    </error>
  </presence>

http://dump.bitcheese.net/images/erogino/screenshot.png

Actions #3

Updated by Mellon almost 12 years ago

Вангую, что ты проверяешь только некоторые ошибки, и если они не цепляются, то типа всё норм. А вот и нифига. норм - это когда получено подтверждение успешности со стороны сервера

Actions #4

Updated by 0xd34df00d almost 12 years ago

Обнови qxmpp сначала же, ну. Там в другом проблема была.

Actions #5

Updated by Mellon almost 12 years ago

ок. починено. оба два бага. Благодарю

Actions

Also available in: Atom PDF