just remove messages-mixer settings grabbing all together

This commit is contained in:
Stephen Birarda 2015-11-17 18:55:09 -08:00
parent c21fbc9a46
commit 2355ba70ca
2 changed files with 2 additions and 30 deletions

View file

@ -129,32 +129,6 @@ void MessagesMixer::run() {
auto nodeList = DependencyManager::get<NodeList>();
nodeList->addNodeTypeToInterestSet(NodeType::Agent);
// The messages-mixer currently does not have any settings, so it would be kind of insane to bail on an empty settings
// object. The below can be uncommented once messages-mixer settings are enabled.
// wait until we have the domain-server settings, otherwise we bail
// DomainHandler& domainHandler = nodeList->getDomainHandler();
//
// qDebug() << "Waiting for domain settings from domain-server.";
//
// // block until we get the settingsRequestComplete signal
// QEventLoop loop;
// connect(&domainHandler, &DomainHandler::settingsReceived, &loop, &QEventLoop::quit);
// connect(&domainHandler, &DomainHandler::settingsReceiveFail, &loop, &QEventLoop::quit);
// domainHandler.requestDomainSettings();
// loop.exec();
//
// if (domainHandler.getSettingsObject().isEmpty()) {
// qDebug() << "Failed to retreive settings object from domain-server. Bailing on assignment.";
// setFinished(true);
// return;
// }
//
// // parse the settings to pull out the values we need
// parseDomainServerSettings(domainHandler.getSettingsObject());
}
void MessagesMixer::parseDomainServerSettings(const QJsonObject& domainSettings) {
// TODO - if we want options, parse them here...
const QString MESSAGES_MIXER_SETTINGS_KEY = "messages_mixer";
// The messages-mixer currently does currently have any domain settings. If it did, they would be
// synchronously grabbed here.
}

View file

@ -35,8 +35,6 @@ private slots:
void handleMessagesUnsubscribe(QSharedPointer<NLPacketList> packetList, SharedNodePointer senderNode);
private:
void parseDomainServerSettings(const QJsonObject& domainSettings);
QHash<QString,QSet<QUuid>> _channelSubscribers;
};