Diferença entre catalog_product_save_after e catalog_product_save_commit_after?

8

Alguém pode explicar a diferença entre esses eventos. Apenas o rápido e sujo, por favor. Obrigado.

Eu tenho um método Observer assim:

public function detectProductChanges($observer)
    {
        $product = $observer->getProduct();
        $old = $product->getOrigData();
        $new = $product->getData();
        if ($product->hasDataChanges() && $old['status'] == 1 && $new['status'] == 2) {
            $this->_sendStatusMail($product);
        }
    }

Não está chegando ao sendStatusMail()

Estou entrando no evento:

        <events>
            <catalog_product_save_after>
                <observers>
                    <productchange>
                        <type>singleton</type>
                        <class>A_ProductNotification_Model_Observer</class>
                        <method>detectProductChanges</method>
                    </productchange>
                </observers>
            </catalog_product_save_after>
        </events>

Devo estar usando: catalog_product_save_commit_after

OBJETIVO:

Envie um email após o produto ser desativado.

private function _sendStatusMail($product)
    {
        if (!Mage::getStoreConfig('trans_email/ident_custom3/email')) return false;
        $emailTemplate = Mage::getModel('core/email_template');
        $emailTemplate->loadDefault('elec_productnotification_tpl');
        $emailTemplate->setTemplateSubject('Product has been disabled');
        $emailTemplate->setSenderEmail($salesData['email']);
        $emailTemplateVariables['style_number']   = $product->getElecStyle();
        $emailTemplateVariables['frame_color']    = $product->getAttributeText('frame_color');
        $emailTemplateVariables['size']           = $product->getAttributeText('size');
        $emailTemplateVariables['elec_color'] = $product->getAttributeText('elec_color');
        $emailTemplateVariables['store_name']   = Mage::getModel('core/store')->load($product->getStoreId())->getName();
        $emailTemplateVariables['product_name'] = Mage::getModel('catalog/product')->load($product->getId())->getName();
        $emailTemplateVariables['product_sku']  = $product->getSku();
        $emailTemplateVariables['dates']        = date("F jS Y h:i:sA", strtotime('-7 hours'));
        // Get General email address (Admin->Configuration->General->Store Email Addresses)
        $emails = explode(',', Mage::getStoreConfig('trans_email/ident_custom3/email'));
        foreach ($emails as $email) $emailTemplate->send($email, $product->getStoreId(), $emailTemplateVariables);
    }
}
este método
fonte
você deve usar o evento <catalog_product_status_update>
Nickool
É por isso que não está disparando? Simplesmente usando o evento errado? @Nickool
thismethod

Respostas:

14

O salvamento acontece em uma transação do MySQL e o save_afterevento é acionado antes que a transação seja confirmada, para que você possa fazer atualizações adicionais no banco de dados dentro da mesma transação.

O save_commit_afterevento é acionado após a transação ser confirmada, ou seja, quando as alterações foram gravadas no banco de dados.

Além disso, em save_commit_after, a _hasDataChangespropriedade já foi redefinida para false, portanto, sua verificação não funcionaria. Por outro lado, se não houvesse alterações, os dois eventos nem seriam acionados, porque Mage_Core_Model_Abstract :: save () não fará nada se não houver alterações nos dados:

if (!$this->_hasModelChanged()) {
    return $this;
}

Dito isto, não vejo por que seu código não deve funcionar.

Fabian Schmengler
fonte
Obrigado pela resposta. Quando adiciono um Mage :: log () no lugar de sendStatusMail (), recebo a mensagem de log corretamente. Mas não está enviando e-mails. Garanti que "Desativar comunicações por email" esteja definido como NÃO e que meu endereço de email esteja em meus endereços de email personalizados da loja de email. Alguma outra idéia de por que não está funcionando? @fschmengler
thismethod
Sem conhecer o método sendStatusMail, não. Provavelmente é material para outra pergunta. Ou o mesmo método funciona se chamado de um contexto diferente?
Fabian Schmengler
Atualizei minha pergunta original para mostrar o método sendStatusMail. Se você não se importa em ajudar ainda mais. Obrigado.
Thismethod
Alguma chance de você me dar sua opinião sobre meu método sendStatusMail ($ product)?
Thismethod
Eu não posso detectar eventuais erros lá, desculpe
Fabian Schmengler
0

fornecedor / magento / framework / Modelo / ResourceModel / Db / AbstractDb.php

public function save(\Magento\Framework\Model\AbstractModel $object)
{
    // ...

    $this->beginTransaction();

    try {
        // ...
        if ($object->isSaveAllowed()) {
            // ...
            $this->_beforeSave($object);
            // ...
            if ($this->isObjectNotNew($object)) {
                $this->updateObject($object);
            } else {
                $this->saveNewObject($object);
            }
            // ...
            $this->processAfterSaves($object);
        }
        $this->addCommitCallback([$object, 'afterCommitCallback'])->commit();
        // ...
    } catch (\Exception $e) {
        $this->rollBack();
        $object->setHasDataChanges(true);
        throw $e;
    }
    return $this;
}

Vamos dar uma olhada em salvar a entidade do produto.

-product_model save
|-product_resource save
|--begin transaction (0 lvl)
|---before product save events
|---creating new product or updating existing one
|---after product save events
|----one of event is saving another entity CatalogInventory Stock
|-----catalog_inventory_stock resource save
|------begin another transaction (1 lvl)
|-------before stock save events
|-------updating / creating stock item
|-------after product save events (here could be one more 
        dependable entity which could cause one more save
        operation and begin another transaction)
|------commit of 1st level !!! No callbacks executed
|--commit of 0 level ALL CALLBACKS ARE EXECUTED

Aqui está o código da função de confirmação:

/**
 * Commit resource transaction
 *
 * @return $this
 * @api
 */
public function commit()
{
    $this->getConnection()->commit();
    /**
     * Process after commit callbacks
     */
    if ($this->getConnection()->getTransactionLevel() === 0) {
        $callbacks = CallbackPool::get(spl_object_hash($this->getConnection()));
        try {
            foreach ($callbacks as $callback) {
                call_user_func($callback);
            }
        } catch (\Exception $e) {
            $this->getLogger()->critical($e);
        }
    }
    return $this;
}

Vamos dar uma olhada no nosso exemplo mais de perto.

  1. $this->getConnection()->commit();coloque valores no DB para o nosso 1º nível (é Estoque). Se algo ruim acontecer aqui, a exceção será lançada e todas as alterações serão revertidas.

  2. Em seguida, ele processa retornos de chamada. Como estamos atualmente no 1º nível, nenhum retorno de chamada será chamado. E estamos saindo do evento catalog_product_after_save para confirmar as alterações do produto (nível 0).

  3. $this->getConnection()->commit();coloque valores no DB para o nosso nível 0 (é o próprio produto). Se algo ruim acontecer aqui, a exceção também será lançada e todas as alterações também serão revertidas.

  4. Então, estamos migrando para a execução de retornos de chamada. Agora estamos no nível 0 e os retornos de chamada serão executados. Qualquer coisa ruim dentro dele call_user_func($callback);será alcançada e registrada. Nada será revertido se o retorno de chamada causar uma exceção

zhartaunik
fonte