Pim-community-dev: Não é possível atualizar o produto com eixo variante

Criado em 28 ago. 2018 · 12Comentários · Fonte: akeneo/pim-community-dev

Desde a versão 2.3.5 não é possível atualizar uma variante do produto.

O problema está no método isEqual(ValueInterface $value) na classe AbstractValue (src / Pim / Component / Catalog / Model / AbstractValue.php). O primeiro igual a $this->getData() === $value->getData() na linha 60 só será verdadeiro se ambas forem as mesmas referências à mesma instância da mesma classe, o que nunca deve ser o caso. Portanto, neste caso, o operador de comparação deve ser usado em vez do operador de identidade.

O erro lógico foi introduzido na confirmação 3e652c2820639e6b7192292644273bb2d97d2b85.

akeneo-pim-system-info_2018-08-28_11_14.txt

bug

Fonte

simonkey

Comentários muito úteis

Na verdade, agora posso reproduzi-lo! Obrigado. Pode estar relacionado ao tipo de atributo usado como eixo variante. Atualmente, só consigo ver o erro quando o tipo é "métrico".

docteurklein em 7 set. 2018

👍2

Todos 12 comentários

Olá @simonkey ,

Agradecemos por levantar esse problema. Você pode nos dizer como podemos reproduzi-lo facilmente?
Tentei atualizar as variantes do produto em nosso ambiente de demonstração e está funcionando bem.
(Nosso env demo agora está na versão 2.3.6).

Obrigado!

LaureBrosseau em 6 set. 2018

Está acontecendo na demonstração. Podemos ver aqui:
http://demo.akeneo.com/#/enrich/product/1240

Já atualizei para o 2.3.6, mas isso não corrigiu.

fjlenatz em 6 set. 2018

Ei @fjlenatz ,

Obrigado pelo bom relatório. @LaureBro e eu (separadamente) tentamos reproduzir isso na demonstração, mas tudo parece funcionar como esperávamos.

Talvez não sigamos o mesmo cenário. Você pode fornecer um cenário passo a passo da demonstração com suas ações e expectativas?

Obrigado.

docteurklein em 7 set. 2018

Oi!

1) Crie um modelo de produto de TV LED e defina a variante de TV LED.
2) Crie um produto com essa variante.
3) Ele será criado, mas quando você tentar editá-lo ou salvá-lo novamente, verá o erro.

No produto que enviei antes que você pudesse ver. Outro exemplo:
http://demo.akeneo.com/#/enrich/product/1242

fjlenatz em 7 set. 2018

👍2

No conjunto de demonstração padrão é o modelo do produto TV SAMSUNG (identificador samtv) na família "TVs LED". Quando você tenta atualizar algo em uma das variantes existentes, o erro Variant axis "display_diagonal" não pode ser modificado, ... ocorre e o produto não pode ser salvo / atualizado.

simonkey em 7 set. 2018

👍1

Obrigado pelos detalhes :) Vou testar isso agora mesmo!

LaureBrosseau em 7 set. 2018

Na verdade, agora posso reproduzi-lo! Obrigado. Pode estar relacionado ao tipo de atributo usado como eixo variante. Atualmente, só consigo ver o erro quando o tipo é "métrico".

docteurklein em 7 set. 2018

👍2

Sim, parece que o erro ocorre apenas se "métrica" for usada.

simonkey em 7 set. 2018

👍1

Eu também confirmo, este é realmente um bug, obrigado por relatá-lo!
Você gostaria de contribuir e corrigir isso?
Aqui está nosso guia de contribuição: https://docs.akeneo.com/2.3/contribute_to_pim/contribution_guide.html#how -to-submit-a-patch-to-the-pim
Avise-me se quiser fazer uma RP e, em caso afirmativo, se precisar de ajuda.
Se você não quiser contribuir, não há problema, vamos adicionar esse problema em nosso rastreador de bugs e corrigi-lo o mais rápido possível.

LaureBrosseau em 7 set. 2018

Olá @simonkey , @fjlenatz ,

Este problema será tratado por nossa equipe de manutenção muito em breve, eu o removi do projeto Quer contribuir. Eu vou manter você informado.

LaureBrosseau em 12 set. 2018

Olá @simonkey e @fjlenatz ,

Aqui está o PR com a correção: https://github.com/akeneo/pim-community-dev/pull/8872
Ele será mesclado durante o dia e a correção estará disponível na próxima tag do 2.3 :)

Tenha um bom dia !