Issue #242

Closed
softwarepublico/softwarepublico#242
Created by Jonathas Santos Rocha (Edited )

Não mostra todas as mensagens do fórum.

Descrição do Bug: No fórum “00 – Instalação do e-cidade” da comunidade (e-cidade) foram migrados 44 comentários, porém está sendo visualizado somente 11 comentários.

Ambiente: Produção. Passo-a-Passo: 1- Entre na página principal do Portal de Software Público

     2- Agora escrava o nome da comunidade (e-cidade) no catálogo de Software Público. 

         3-Depois de ter entrado na comunidade e-cidade, irá ter um diretório na parte inferior 

direito da sua tela, escrito HISTÓRICO DE FÓRUNS.

https://softwarepublico.gov.br/social/e-cidade/historico-de-foruns

4- Já dentro do Histórico de Fóruns basta você entrar no fórum (00 – Instalação do e-cidade).

https://softwarepublico.gov.br/social/e-cidade/historico-de-foruns/00.-instalacao-do-e-cidade

       5- Depois basta acessar o post Portal da Transparência. 

https://softwarepublico.gov.br/social/e-cidade/historico-de-foruns/00.-instalacao-do-e-cidade/portal-da-transparencia.

Sugestão de solução: Solicitamos que apareça os 44 comentários.

Captura de tela de 2015-09-15 13:17:43

@nayanne @marisa @obsneto, para conhecimento!

13 participants
  • A8ff67a0e69e7cfaeb1299e02082c339?s=40&d=identicon
    Arthur Esposte @arthurmde

    @thiago e @tallysmartins mapear a correção deste bug como urgente.

    @diguliu será que isso pode ter a ver com algum limite de respostas encadeadas?

    Choose File ...   File name...
    Cancel
  • A8ff67a0e69e7cfaeb1299e02082c339?s=40&d=identicon
    Arthur Esposte @arthurmde

    Reassigned to @thiago

    Choose File ...   File name...
    Cancel
  • 2fb770603e9880964ce0f8ac999d516f?s=40&d=identicon
    Rodrigo Souto @diguliu

    Pessoal, estou investigando esse problema e tanto testando manualmente quando pelo código, não parece ter nada limitando visualização de todos os comentários. Não tem nenhum mecanismos de paginação ou limite de aninhamento.

    Vejam o exemplo em anexo. Continuo investigando o problema.

    Choose File ...   File name...
    Cancel
  • 2fb770603e9880964ce0f8ac999d516f?s=40&d=identicon
    Rodrigo Souto @diguliu

    Outro exemplo com um nível de profundidade 30 (nenhum um pouco recomendável).

    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    @arthurmde como o @diguliu está investigando, a solução desse bug será resolvida na release 5.

    Choose File ...   File name...
    Cancel
  • 2fb770603e9880964ce0f8ac999d516f?s=40&d=identicon
    Rodrigo Souto @diguliu

    Descobri o problema. O restante dos comentários não está aparecendo porque existe isso aqui no corpo de um dos comentário:

    <!-- @page { margin: 2cm } P { margin-bottom: 0.21cm } - </style>
    

    Eu apliquei o patch de não escapar os htmls antes de rodar a migração para evitar que os comentários ficassem com aqueles duplo escapes feios e provavelmente isso fez com que esse problema antigo de html mal feito no corpo do comentário fizesse a página quebrar. A pouco tempo eu apliquei um patch no noosfero oficial acreditando que esse problema não acontecia mais no Rails 3 (eu testei com alguns htmls mal formados e aparentemente não estava mais acontecendo), mas pode ser que ainda aconteça. Irei investigar se isso está acontecendo no master, mas o que podemos fazer temporariamente é remover essa parte do corpo do comentário para pelo menos resolver esse problema pontual.

    Choose File ...   File name...
    Cancel
  • 2fb770603e9880964ce0f8ac999d516f?s=40&d=identicon
    Rodrigo Souto @diguliu

    Acabei de testar no master e o problema não está acontecendo. Então o problema não irá mais acontecer na próximas versões, porém os conteúdos que já foram criados antes podem ainda conter o problema, como é o caso desse comentário. Acho que a solução será irmos resolvendo os casos pontuais quando forem descobertos.

    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    @nayanne @marisa @obsneto @jonathas

    Pessoal, como o @diguliu disse, vamos resolevr os casos pontuais, caso aconteçam/continuem por favor reabrir essa issue com o link para o fórum.

    Obrigado.

    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    Status changed to closed

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Reassigned to @marisa

    Choose File ...   File name...
    Cancel
  • Bf66cdbbe0d7aee6a689d2f49f39e4ea?s=40&d=identicon
    Jonathas Santos Rocha @jonathas

    Erro não corrigido.

    Choose File ...   File name...
    Cancel
  • Bf66cdbbe0d7aee6a689d2f49f39e4ea?s=40&d=identicon
    Jonathas Santos Rocha @jonathas

    Status changed to reopened

    Choose File ...   File name...
    Cancel
  • 2fb770603e9880964ce0f8ac999d516f?s=40&d=identicon
    Rodrigo Souto @diguliu

    @jonathas você poderia por favor mandar um link ou um screenshot de onde esse problema ainda está acontecendo para que possamos corrigí-lo?

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    @lmatos dá uma olhada nisso, por favor.

    Choose File ...   File name...
    Cancel
  • 800e3af93350753db9dee1864cef10ff?s=40&d=identicon
    Thiago Ribeiro @thiago

    no aguardo da validação, @lmatos não vimos esse no dia da reunião, porém o @diguliu resolveu o problema, talvez quando o @jonathas validou não havia sido atualizado em dev ou prod.

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Reassigned to @lmatos

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Milestone changed to Prioridades para o Encerramento

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Todas que estiverem fechadas no Milestone de Encerramento serão homologadas pelo MP. Vou fechar para nosso controle interno de encerramento do projeto.

    @kanashiro , @marcosronaldo e @melissawen : para conhecimento.

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Status changed to closed

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa
    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Status changed to reopened

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    @diguliu ^ ?

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    @marisa e @lmatos : o @diguliu me avisou que já corrigiu o problema tanto em dev quanto em produção. Por favor, verificar e se estiver ok, fechar esta issue.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa (Edited )
    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa
    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Avisei ao @diguliu no IRC.

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm (Edited )

    @marisa , este é o comportamento esperado mesmo, no contexto desta issue. Segue o que foi dito por @diguliu:

    • 3:54 PM [diguliu] o que eles estavam reclamando era o fato de terem comentários que não estavam aparecendo mesmo
    • 3:55 PM [diguliu] não o fato de os comentários ficarem "esmagados" pelo nível de profundidades
    • 3:56 PM {diguliu} (não que isso não seja um problema tb, mas essa não é o ponto da issue)
    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Status changed to closed

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Oi @paulormm nos exemplos que demos acima existem comentários que não podem ser vizualidados conforme o problema inicial relatado. O fato de não mostrar é exatamente porque está esmagado, um problema está diretamente relacionado ao outro, sinceramente não entendi o que foi resolvido nessa issue já que para nos o problema inicial não foi resolvido.

    Choose File ...   File name...
    Cancel
  • 2fb770603e9880964ce0f8ac999d516f?s=40&d=identicon
    Rodrigo Souto @diguliu

    Olá @marisa, voltei a conseguir comentar nas issues.

    Veja que não estou dizendo que o comentário estar "esmagado" desse jeito não seja um problema, realmente essa estrutura de aninhamento de comentários do noosfero não comporta um nível muito profundo de respostas como era comum no portal antigo e isso inviabiliza a leitura do conteúdo. Mas essa issue, especificamente, não lida desse problema. Se você ler o histórico das mensagens você pode ver que essa issue se trata de comentários que não estavam sendo mostrados, "esmagados" ou não (veja esse trecho na descrição da issue "foram migrados 44 comentários, porém está sendo visualizado somente 11 comentários."). Realmente os comentários não estavam sendo mostrados por conta de um comentário html <!-- não escapado que fechava o conteúdo e deixava de listar todos os comentários a diante. Isso foi resolvido.

    Choose File ...   File name...
    Cancel
  • 4f4b696adc8e8769f43b5c837b3ea23b?s=40&d=identicon
    Marisa Souza dos Santos @marisa

    Oi Rodrigo. Entendido. Obrigada. Homologado

    Choose File ...   File name...
    Cancel
  • B4c7afc2cdffea95a8c2d0387acedc6b?s=40&d=identicon
    Paulo Meireles @paulormm

    Criei uma issue na comunidade do NoosferoGov para tentarmos uma colaboração para essa melhoria no Noosfero para o futuro:

    Choose File ...   File name...
    Cancel