search

Hendrik7 / psi-dev

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

История сообщений в конференциях #558

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
При заходе в конференцию, сервер может 
отправить некоторое количество последних 
сообщений. Если произошел реконнект, то 
клиент опять получит те же сообщения. Это 
очень неудобно, так как чат оказывается 
забит повторяющимися наборами сообщений.

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

Original issue reported on code.google.com by mity...@gmail.com on 4 Sep 2013 at 7:31

Attachments:

GoogleCodeExporter commented 9 years ago 
патч одобряю.
не могли бы вы его сделать против ванильной 
версии пси?

Original comment by rion4ik on 4 Sep 2013 at 9:42

GoogleCodeExporter commented 9 years ago 
А как решается проблема с тем, что на 
сервере и клиенте время может быть 
существенно различным?

Original comment by 0xD34DF...@gmail.com on 4 Sep 2013 at 12:53

GoogleCodeExporter commented 9 years ago 
таки нашел косяк в патче. он использует 
lastMsgTime_ из диалога. но это переменная была 
перенесена в чатвью.

0xD34DF00D, utc. ну а если всё равно отличается, то 
очевидно это проблемы пользователя

Original comment by rion4ik on 4 Sep 2013 at 1:56

GoogleCodeExporter commented 9 years ago 
Привязка к UTC спасет только от разницы в 
часовых поясах. Условный jabber.ru и условный 
gentoo.ru могут иметь (и наверняка будут иметь) 
расхождение в часах на несколько секунд 
или даже минут. Для активных конференций 
это может быть важно.

Энивей, спасибо.

Original comment by 0xD34DF...@gmail.com on 4 Sep 2013 at 1:59

GoogleCodeExporter commented 9 years ago 
Я не уверен что _lastMsgTime из ChatViewCommon подойдет, 
так как она обновляется и при смене темы. 
Возможно использование lastMsgTime_ из диалога 
было и не самым лучшим решением, но, по 
крайней мере, это работает =)

Вот патч для чистой psi.

Original comment by mity...@gmail.com on 4 Sep 2013 at 3:23

Attachments:

GoogleCodeExporter commented 9 years ago 
закоммитил

Original comment by rion4ik on 5 Sep 2013 at 5:31