Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[amqp] fix config options. #1

Merged
merged 1 commit into from
Jan 2, 2018
Merged

[amqp] fix config options. #1

merged 1 commit into from
Jan 2, 2018

Conversation

makasim
Copy link
Member

@makasim makasim commented Jan 2, 2018

@makasim makasim added the bug label Jan 2, 2018
@makasim makasim merged commit aee5cb0 into master Jan 2, 2018
@makasim makasim deleted the fix-config-options branch January 2, 2018 16:45
@cedricmagne
Copy link

cedricmagne commented Jan 2, 2018

Same error.
Therefore if we change config node we need to change setting node in setting.xml
I suggest to change array keys in Enqueue/Enqueue/Helper/Data.php line 133

/**
 * @return array
 */
public function getRabbitMqAmqpConfig()
{
    return ['rabbitmq_amqp' => [
        'host' => Mage::getStoreConfig('enqueue/rabbitmq_amqp/host'),
        'port' =>  (int) Mage::getStoreConfig('enqueue/rabbitmq_amqp/port'),
        'user' => Mage::getStoreConfig('enqueue/rabbitmq_amqp/login'),
        'pass' => Mage::getStoreConfig('enqueue/rabbitmq_amqp/password'),
        'vhost' => Mage::getStoreConfig('enqueue/rabbitmq_amqp/vhost'),
        'lazy' => (bool) Mage::getStoreConfig('enqueue/rabbitmq_amqp/lazy'),

    ]];
}

Thanks,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants