Projects tigase _server server-core Issues #503
Update Item Configuration (basic-conf) (#503)
Closed
Eric Dziewa opened 9 years ago
Due Date
2015-09-29

Does not work.

Daniel Wisnewski commented 9 years ago

current settings display, but changes are not saved.

Andrzej Wójcik (Tigase) commented 9 years ago

%wojtek - do you know this command and what it should do for basic-conf?

wojciech.kapcia@tigase.net commented 9 years ago

(if memory servers me right) This is not explicit for basic-conf - with this command it should be possible to update configuration of any component, however only single property at a time, and it would effectively call .setProperties() of the given component. But! Some components doesn't deal well with single-property-updates (repo initialization with only URI at one time, etc) therefore a couple of components simply have following code:

if (props.size() == 1) {
	return;
}

which results in lack of configuration change.

Andrzej Wójcik (Tigase) commented 9 years ago

As Wojtek mentioned not all components support setting properties in this way and it is not possible in current version to detect if particular component supports this or not.

This values are also not saved in etc/init.properties file as this command is to change properties at runtime.

Due to that I would assume that this command works as it should.

Daniel Wisnewski commented 9 years ago

I spent some time working through features, changes to basic-conf components are read by the server now. If appropriate I can see which components will work with this feature. Otherwise we can close this issue.

wojciech.kapcia@tigase.net commented 9 years ago

I think that with the changed handling of configuration being in the works it would be enough to simply note that this may or may not work with some components -- what do you thing %kobit ?

Artur Hefczyc commented 9 years ago

Agree, Bartosz works on a new configuration framework anyway, so we can close this ticket.

issue 1 of 1
Type
Bug
Priority
Normal
Assignee
RedmineID
3119
Version
tigase-server-7.1.0
Issue Votes (0)
Watchers (0)
Reference
tigase/_server/server-core#503
Please wait...
Page is in error, reload to recover