uralplan / psi-dev

Automatically exported from code.google.com/p/psi-dev
0 stars 0 forks source link

Дубль контакты-транспорты в нескольких группах, не исправить средствами Psi(+), только удалить оба #523

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
Шаги по воспроизведению проблемы:
What steps will reproduce the problem?
1. Добавить контакт-транспорт в какую-либо 
группу через другой клиент (например через 
tkabber добавить в группу Transports, что логично 
если вы ранее пользовались tkabber'ом).
2. Выйти из другого клиента (не обязательно), 
зайти под тем же аккурантом в Psi/Psi+. Видим 
транспорт сразу в группах Transports и Сервисы 
(вариант имени при русском интерфейсе, в 
английском тоже в двух группах).
3. Попытаться поменять принадлежность к 
группам, то есть убрать наличие транспорта 
в дополнительной группе Transports средствами 
Psi(+). Но это не возможно, это меню недоступно 
для контактов-транспортов.
4. Дополнительный необязательный шаг. По 
глупости попытаться удалить дубликат 
контакта в ростере через удаление 
контакта. В результате естественно этот 
контакт-транспорт удалится из всех групп, 
из ростера и вероятно деавторизуется, но 
пользователь может не ожидать такого 
поведения.

Каков ожидаемый результат?
What is the expected output?
Три варианта по степени уменьшения 
ожидаемости:
1) Psi(+) делает невозможным изменять 
принадлежность контакта-транспорта к 
группам, поэтому логично, что 
принадлежность к группам должна 
игнорироваться для контактов-транспортов 
и они должны показываться в единственном 
экземпляре в ростере в виртуальной группе 
Сервисы (для русского интерфейса).
2) Принудительно удалять принадлежность 
контактов-транспортов к каким-либо группам 
при загрузке ростера. Правда это изменит 
отображение ростера в других клиентах 
естественно.
3) По мнению участников официальной 
jabber-конференции tkabber'а очень странно, что 
Psi(+) не позволяет произвольно 
устанавливать группы для 
контактов-транспортов. В таком случае 
следует разрешить пользователю Psi(+) 
установку группы для контакта-транспорта и 
естественно поправить глюку с двойным 
отображением контакта-транспорта, когда он 
на самом деле состоит в одной-единственной 
группе Transports (Psi(+) требует чтобы контакт не 
состоял ни в одной группе, тогда он он 
отображается только в виртуальной группе 
Сервисы).

Что Вы видите вместо этого?
What do you see instead?
Я вижу контакт-транспорт одновременно в 
нескольких группах в ростере в Psi(+) и не 
могу это исправить средствами Psi(+). 
Единственные 2 способа изменить это 
средствами Psi(+):
1) послать специальные команды через 
xml-консоль, но простой пользователь из не 
знает;
2) удалить контакт из ростера и добавить 
заново, но для этого скорее всего нужна 
перерегистрация на транспорте, и транспорт 
нужно еще суметь найти вновь через "Обзор 
сервисов", потому что транспорт может быть 
на другом сервере, а пользователь забыл на 
каком, потому что регистрировал его очень 
давно.

Какую версию Psi+ / ОС Вы используете?
What version of the Psi+/OS are you using?
Версия и ОС не имеют значения в данном 
случае, но...
Psi+ 0.15-dev-20120727 / Ubuntu Linux 10.10.
Qt версии 4.7.4. Webkit нет.
DE: Gnome 2.32.0.

Дополнительная информация по проблеме:
Additional information:
Psi(+) показывает что контакт-транспорт 
принадлежит на одну группу больше, чем он 
принадлежит в tkabber'е за исключением случая, 
когда он добавлен через tkabber в группу 
Сервисы, совпадающую по имени с 
виртуальной группой Сервисы Psi(+). (В 
ткаббере можно добавлять контакты более 
чем в одну группу, но и одной достаточно для 
демонстрации глюка.)
P.S.: Небольшое уточнение: при первом тесте 
закрепления через ткаббер 
контакта-транспорта в совпадающем с именем 
виртуальной группы Сервисы получил в Psi(+) 
две группы Сервисы, при втором тесте одну 
группу Сервисы.

Original issue reported on code.google.com by W.prox...@gmail.com on 17 Oct 2012 at 11:04

GoogleCodeExporter commented 8 years ago

Original comment by wadea...@gmail.com on 18 Oct 2012 at 10:44

GoogleCodeExporter commented 8 years ago
Надо сначала решить какое поведение нужно 
достичь для клиента при переносе 
транспорта в другую группу, а уж потом 
патчить. Жду предложений тут и в конфе.

> P.S.: Небольшое уточнение: при первом тесте 
закрепления через ткаббер 
контакта-транспорта в совпадающем с именем 
виртуальной группы Сервисы получил в Psi(+) 
две группы Сервисы, при втором тесте одну 
группу Сервисы.

Возможно это связано с тем, что в одном 
случае Psi+ работал параллельно с ткаббером 
в момент переноса в другую группу, а в 
другом был отключен.

Original comment by li...@mail.ru on 3 Nov 2012 at 2:42

GoogleCodeExporter commented 8 years ago
Меня неожиданно перестало пускать на jabber.ru 
=) Ну да ладно.
Я предлагаю:
1) Psi(+) делает невозможным изменять 
принадлежность контакта-транспорта к 
группам, поэтому логично, что 
принадлежность к группам должна 
игнорироваться для контактов-транспортов 
и они должны показываться в единственном 
экземпляре в ростере в виртуальной группе 
Сервисы (для русского интерфейса).

Это больше всего согласовывается с 
политикой текущего поведения Psi+ и не 
изменит отображение ростера в других 
клиентах.

Original comment by W.prox...@gmail.com on 3 Nov 2012 at 2:54

GoogleCodeExporter commented 8 years ago
А вдруг аксакалы проекта согласятся на 
поведение по третьему сценарию? Надо 
сначало уточнить. На самом деле я сам не 
знаю как было изначально задумано. При 
первом варианте есть шанс нарваться на 
грабли при локализации этой группы на 
другие языки.

Original comment by li...@mail.ru on 3 Nov 2012 at 3:16

GoogleCodeExporter commented 8 years ago
Показывать транспорты только в спецгруппе. 
В этом случае и логичней будет поведение 
кнопки Скрыть сервисы 

Original comment by wadea...@gmail.com on 7 Nov 2012 at 6:51

GoogleCodeExporter commented 8 years ago
Исправлено: 
https://github.com/psi-plus/main/commit/f880d93509da2230a4f1660050a259576fc6059d

Original comment by li...@mail.ru on 16 Nov 2012 at 10:00

GoogleCodeExporter commented 8 years ago

Original comment by li...@mail.ru on 8 Feb 2013 at 10:36

GoogleCodeExporter commented 8 years ago
[22:27:08] <proxy_m> liuch: кстати 523 я тоже затестил 
наконец-то. там все нормально теперь 
работает.

Original comment by li...@mail.ru on 14 Jan 2014 at 6:31