Empresa 1.14.1 Amostras causando 35 s mais tempo de carregamento nas páginas de categoria

23

Implementamos o novo recurso Amostras embutidas em nossa versão mais recente do site. Quando habilitamos as amostras nas páginas de categorias, o tempo de carregamento da página varia de 2 segundos a 38 segundos ou mais.

Gostaria de saber se alguém já teve esse problema e se sim, poderia nos dar uma indicação de possíveis soluções?

Tentamos o EE 1.14.1 e CE 1.9.1 com 36 produtos configuráveis ​​com amostras aplicadas no tema rwd padrão e nenhum outro módulo ativo.

Esse problema não pode ser resolvido com o cache, pois sempre que um usuário pesquisa ou filtra uma categoria, a página é interrompida novamente.

Dave Bevington
fonte
Eu não posso reproduzir isso. Por favor, dê-nos mais instruções sobre o tipo de plug-ins instalados, tema etc. Por favor, siga o processo de depuração do Magento desativando seu tema, desativando módulos locais e tente novamente.
22715 philwinkle
Os atributos que estamos usando são amostras de cores e tamanhos não superiores a 8 por item e, na maioria dos casos, não superiores a 4. Isso está sendo executado em uma instalação em branco do magento CE 1.9.1 com dados de amostra carregados e 10 produtos configuráveis ​​com amostras personalizadas adicionado. Definitivamente associado às amostras, quanto mais adicionamos, mais lento o site fica. Observe que o cache está desativado para testar isso, pois os usuários podem filtrar a pesquisa e não podemos ter um tempo de carregamento louco sempre que um usuário ajusta sua pesquisa. Obrigado pelo seu tempo :)
Dave Bevington

Respostas:

22

Direita. Detecto problema na função Mage_ConfigurableSwatches_Helper_Mediafallback :: attachConfigurableProductChildrenAttributeMapping.

Eu faço algumas mudanças nele. Isso aumenta o desempenho.

Experimentar:

  1. Copie /app/code/core/Mage/ConfigurableSwatches/Helper/Mediafallback.phppara /app/code/local/Mage/ConfigurableSwatches/Helper/Mediafallback.php.

  2. No /app/code/local/Mage/ConfigurableSwatches/Helper/Mediafallback.phparquivo, mova este código (ll.88-91)

     // normalize to all lower case before we start using them
     $optionLabels = array_map(function ($value) {
      return array_map('Mage_ConfigurableSwatches_Helper_Data::normalizeKey', $value);
     }, $optionLabels);
    

    até antes do foreachloop.

Este é o método alterado:

 /**
 * Set child_attribute_label_mapping on products with attribute label -> product mapping
 * Depends on following product data:
 * - product must have children products attached
 *
 * @param array $parentProducts
 * @param $storeId
 * @return void
 */
public function attachConfigurableProductChildrenAttributeMapping(array $parentProducts, $storeId)
{
    $listSwatchAttr = Mage::helper('configurableswatches/productlist')->getSwatchAttribute();

    $parentProductIds = array();
    /* @var $parentProduct Mage_Catalog_Model_Product */
    foreach ($parentProducts as $parentProduct) {
        $parentProductIds[] = $parentProduct->getId();
    }

    $configAttributes = Mage::getResourceModel('configurableswatches/catalog_product_attribute_super_collection')
        ->addParentProductsFilter($parentProductIds)
        ->attachEavAttributes()
        ->setStoreId($storeId)
    ;

    $optionLabels = array();
    foreach ($configAttributes as $attribute) {
        $optionLabels += $attribute->getOptionLabels();
    }

    // normalize to all lower case before we start using them
    $optionLabels = array_map(function ($value) {
        return array_map('Mage_ConfigurableSwatches_Helper_Data::normalizeKey', $value);
    }, $optionLabels);

    foreach ($parentProducts as $parentProduct) {
        $mapping = array();
        $listSwatchValues = array();

        /* @var $attribute Mage_Catalog_Model_Product_Type_Configurable_Attribute */
        foreach ($configAttributes as $attribute) {
            /* @var $childProduct Mage_Catalog_Model_Product */
            if (!is_array($parentProduct->getChildrenProducts())) {
                continue;
            }

            foreach ($parentProduct->getChildrenProducts() as $childProduct) {

                // product has no value for attribute, we can't process it
                if (!$childProduct->hasData($attribute->getAttributeCode())) {
                    continue;
                }
                $optionId = $childProduct->getData($attribute->getAttributeCode());

                // if we don't have a default label, skip it
                if (!isset($optionLabels[$optionId][0])) {
                    continue;
                }

                // using default value as key unless store-specific label is present
                $optionLabel = $optionLabels[$optionId][0];
                if (isset($optionLabels[$optionId][$storeId])) {
                    $optionLabel = $optionLabels[$optionId][$storeId];
                }

                // initialize arrays if not present
                if (!isset($mapping[$optionLabel])) {
                    $mapping[$optionLabel] = array(
                        'product_ids' => array(),
                    );
                }
                $mapping[$optionLabel]['product_ids'][] = $childProduct->getId();
                $mapping[$optionLabel]['label'] = $optionLabel;
                $mapping[$optionLabel]['default_label'] = $optionLabels[$optionId][0];
                $mapping[$optionLabel]['labels'] = $optionLabels[$optionId];

                if ($attribute->getAttributeId() == $listSwatchAttr->getAttributeId()
                    && !in_array($mapping[$optionLabel]['label'], $listSwatchValues)
                ) {
                    $listSwatchValues[$optionId] = $mapping[$optionLabel]['label'];
                }
            } // end looping child products
        } // end looping attributes


        foreach ($mapping as $key => $value) {
            $mapping[$key]['product_ids'] = array_unique($mapping[$key]['product_ids']);
        }

        $parentProduct->setChildAttributeLabelMapping($mapping)
            ->setListSwatchAttrValues($listSwatchValues);
    } // end looping parent products
}
Andrey M.
fonte
Eu estava tendo o mesmo problema com as amostras ativadas nas páginas da lista e isso ajudou a acelerar consideravelmente as coisas, então obrigado!
Marlon criativa
Eu encontrei o mesmo problema. Para resolver, o carregamento da página demorou de 2,5 minutos a 7 segundos.
Andrew Kett
Essas amostras realmente diminuem as categorias, especialmente quando você tem muitos produtos desconfiguráveis. A solução do Андрей М. reduza o carregamento de 10 a 3 segundos em uma categoria cheia de produtos configuráveis! Obrigado!
User1895954
+1! Obrigado por compartilhar isso. Nós estamos usando um monte de configurável com várias opções de cada um e só não poderia usar amostras de cores mais ...
Marc
+1! Resposta absolutamente brilhante, o tempo de carregamento mudou de 28 segundos para 3 segundos! Obrigado!!
KI
4

Maneira adicional de melhorar as amostras configuráveis ​​de desempenho quando você tem muitas opções de atributos.

Por exemplo, se você possui 2000 opções e mostra 36 produtos na lista de catálogos, nesse caso, o método Mage_ConfigurableSwatches_Model_Resource_Catalog_Product_Attribute_Super_Collection::_loadOptionLabels()se unirá aos rótulos de cada opção de super_atributos e você obterá 2000 * 36 = 72000 linhas.

Eu reescrevi esse método e ele carrega apenas 2000 linhas em vez 72000

<?php
/**
 * Load attribute option labels for current store and default (fallback)
 *
 * @return $this
 */
protected function _loadOptionLabels()
{
    if ($this->count()) {
        $labels = $this->_getOptionLabels();
        foreach ($this->getItems() as $item) {
            $item->setOptionLabels($labels);
        }
    }
    return $this;
}

/**
 * Get Option Labels
 *
 * @return array
 */
protected function _getOptionLabels()
{
    $attributeIds = $this->_getAttributeIds();

    $select = $this->getConnection()->select();
    $select->from(array('options' => $this->getTable('eav/attribute_option')))
        ->join(
            array('labels' => $this->getTable('eav/attribute_option_value')),
            'labels.option_id = options.option_id',
            array(
                'label' => 'labels.value',
                'store_id' => 'labels.store_id',
            )
        )
        ->where('options.attribute_id IN (?)', $attributeIds)
        ->where(
            'labels.store_id IN (?)',
            array(Mage_Catalog_Model_Abstract::DEFAULT_STORE_ID, $this->getStoreId())
        );

    $resultSet = $this->getConnection()->query($select);
    $labels = array();
    while ($option = $resultSet->fetch()) {
        $labels[$option['option_id']][$option['store_id']] = $option['label'];
    }
    return $labels;
}

/**
 * Get Attribute IDs
 *
 * @return array
 */
protected function _getAttributeIds()
{
    $attributeIds = array();
    foreach ($this->getItems() as $item) {
        $attributeIds[] = $item->getAttributeId();
    }
    $attributeIds = array_unique($attributeIds);

    return $attributeIds;
}
Yaroslav Voronoy
fonte