Projects tigase _server server-core Issues #815
DynamicRoster doesn't seem to work (#815)
wojciech.kapcia@tigase.net opened 8 years ago
Due Date
2017-05-02
'sess-man' {
    'dynamic-roster-classes' = 'tigase.xmpp.impl.roster.DynamicRosterTest'
}

Relevant option from 7.1.0:

sess-man/plugins-conf/dynamic-roster-classes=tigase.xmpp.impl.roster.DynamicRosterTest
Andrzej Wójcik (Tigase) commented 8 years ago

I adjusted implementation to work with Tigase Kernel and updated configuration converter.

Now it should be used in following way:

'sess-man' {
    'dynamic-rosters' () {
        DynamicRosterTest (class: tigase.xmpp.impl.roster.DynamicRosterTest) {}
        DynamicRosterTest123 (class: tigase.xmpp.impl.roster.DynamicRosterTest123) {}
    }
}

so that now we have dynamic-roster bean (instance of tigase.xmpp.impl.roster.DynamicRoster@) for which you register new beans which implements @DynamicRosterIfc

wojciech.kapcia@tigase.net commented 8 years ago

Andrzej Wójcik wrote:

I adjusted implementation to work with Tigase Kernel and updated configuration converter.

Now it should be used in following way:

[...]

so that now we have dynamic-roster bean (instance of tigase.xmpp.impl.roster.DynamicRoster@) for which you register new beans which implements @DynamicRosterIfc

I've tested above config and dynamic roster elements are not included.

instance in tigase.xmpp.impl.roster.DynamicRoster#getDynamicRosters is still @null@.

Andrzej Wójcik (Tigase) commented 8 years ago

You are right, I forgot to set instance to this within bean initialization method. Took me a while to understand that this part was missing.

wojciech.kapcia@tigase.net commented 8 years ago

Works:

Dynamic Roster 1 (c2s) 	success 	0 sec 	1 	57 ms 	Dynamic roster storage test 	Dynamic Roster 1 (c2s)
Dynamic Roster 2 (c2s) 	success 	0 sec 	1 	170 ms 	Dynamic roster merge test 	Dynamic Roster 2 (c2s)

Referenced from commit 1 year ago
issue 1 of 1
Type
Bug
Priority
Normal
Assignee
RedmineID
5429
Version
tigase-server-8.0.0
Spent time
13h 45m
Issue Votes (0)
Watchers (0)
Reference
tigase/_server/server-core#815
Please wait...
Page is in error, reload to recover