fix: [internal] Initialize ZMQ just when necessary after setting change

pull/6597/head
Jakub Onderka 2020-11-16 20:37:27 +01:00
parent 459144011e
commit 14efe5681c
1 changed files with 3 additions and 4 deletions

View File

@ -3978,18 +3978,17 @@ class Server extends AppModel
public function zmqAfterHook($setting, $value)
{
$pubSubTool = $this->getPubSubTool();
// If we are trying to change the enable setting to false, we don't need to test anything, just kill the server and return true.
if ($setting == 'Plugin.ZeroMQ_enable') {
if ($setting === 'Plugin.ZeroMQ_enable') {
if ($value == false || $value == 0) {
$pubSubTool->killService();
$this->getPubSubTool()->killService();
return true;
}
} elseif (!Configure::read('Plugin.ZeroMQ_enable')) {
// If we are changing any other ZeroMQ settings but the feature is disabled, don't reload the service
return true;
}
$pubSubTool->reloadServer();
$this->getPubSubTool()->reloadServer();
return true;
}