Projects tigase _server server-core Issues #485
Get list available commands (#485)
Closed
Eric Dziewa opened 9 years ago
Due Date
2015-09-29

Script finished with no errors but returned no results.

Daniel Wisnewski commented 9 years ago

Both Monitor and MUC scripts have same error.

Also any possibility to change command name to incorporate the associated vhost like:

MUC-Get available commands

Monitor-Get available commands

Web UI returns expected results.

Daniel Wisnewski commented 9 years ago

Same behavior in Scripts section as well.

Andrzej Wójcik (Tigase) commented 9 years ago

I fixed some issues related to how this works, but I see no real solution to make this command properly work as adhoc executed from XMPP client and from HTTP API.

Also there is no real value for this command as it was designed to be used by Sure.IM, before we update discovery (disco#items) to return custom values. But now disco#items returns proper values and it is now not needed, so maybe we should remove this command?

Daniel Wisnewski commented 9 years ago

Can confirm this works properly now, would remove command scripts also be part of that cleanup?

Artur Hefczyc commented 9 years ago

I think we could leave this script as kind of a script code example. If you have no objections, I suggest to rename the script to indicate that this is a script example and leave it as it is. It can be part of scripting guide.

Daniel Wisnewski commented 9 years ago

%andrzej.wojcik let's move this to the example scripts section as %kobit suggests.

Andrzej Wójcik (Tigase) commented 9 years ago

Moved to Example scripts group as suggested. Change will be part of next snapshot build

Daniel Wisnewski commented 9 years ago

Can confirm change, looks good. Closing Issue.

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