A conversão de campos personalizados foi afetada pela atualização recente?

Dev
1

Nos últimos dois dias, foram feitos vários relatórios de bugs sobre o plugin de eventos, todos relacionados a problemas de conversão de tipo em campos personalizados. O plugin de eventos não recebeu atualizações substanciais há alguns meses.

O arquivo plugin.rb converte event_start para inteiro:

Topic.register_custom_field_type('event_start', :integer)

O erro está sendo gerado aqui:

def has_event?
  self.custom_fields['event_start']&.nonzero?
end

O próprio erro segue este formato:

NoMethodError (undefined method `nonzero?' for [1563127206, 1563127206]:Array)

Pelo que entendo, register_custom_field_type deveria garantir que o campo personalizado sempre retorne o tipo definido (talvez eu esteja entendendo errado).

Analisando a preocupação has_custom_fields.rb, houve algumas alterações na última semana que podem ter afetado isso, em particular:

@eviltrout, o que você acha disso?

2

Caso este seja de fato o bug que está causando problemas, explico aqui as consequências (pode tornar seu site inutilizável):

3

Isso era um bug conhecido com campos personalizados.

Em condições muito específicas, ele salvaria o valor várias vezes, criando assim um Array, quando você só deseja um Integer.

Você pode corrigir isso garantindo que haja apenas 1 linha por campo personalizado no banco de dados.

4

Sim, isso apareceu algumas vezes no passado. Essa última onda parece coincidir com o trabalho recente no concern has_custom_fields.rb, então pode haver algo para revisar ali.

Isso não corrige o problema, mas ajuda a abordá-lo caso ele surja:

5

Minha recomendação geral aqui é que, como autor de um plugin, você sempre adicione índices em uma migração para impor corretamente a restrição. Na verdade, a maioria dos plugins que escrevemos atualmente evita campos personalizados, a menos que sejam absolutamente necessários, e prefere usar tabelas personalizadas, que são muito mais fáceis de analisar.

Neste caso específico, você precisa de um índice assim:

create unique index idxStartEvent on topic_custom_fields(topic_id) where name = 'start_event'

Não tenho certeza do que mais precisaríamos fazer no núcleo aqui. Já consideramos uma reformulação dos campos personalizados, mas estamos um pouco preocupados com isso. Uma coisa que estou considerando é simplesmente remover o suporte a arrays dos campos personalizados, pois eles têm causado enormes quantidades de problemas ao longo dos anos.

6

Desculpe por ressuscitar este tópico, mas uma das condições ocorre quando você usa um índice de símbolo, ou seja, custom_fields[:hello], ao atualizar o valor existente. Nesse caso, ele adiciona outro campo em vez de atualizar, resultando em um array. Na minha opinião, essa pode ser a única condição.

Isso deve corrigir o efeito colateral causado com certeza.