I post lunghi nei log dello staff rompono il formato CSV di Excel

Contribuisci Bug
1

Lo staff può esportare i registri delle azioni dallo staff da /admin/logs/staff_action_logs per visualizzare la cronologia completa, poiché nell’interfaccia di amministrazione viene mostrata solo una parte dei log. Alcuni log possono contenere virgole nei dettagli, che interrompono la formattazione del file CSV.

8 Mi Piace
2

Can commas be escaped in this output @vinothkannans?

4 Mi Piace
4

No, the commas are not breaking the CSV file. Maximum number of characters allowed in a single cell of MS Excel sheet is 32,767. If a column exceeded the characters limit then it’s breaking the CSV format. In our case details column having more characters in some post_edit staff actions. If I open the same CSV file in Google Sheets then it renders all the columns correctly.

If needed I’ll limit the maximum number of characters for details column to ~30,000 in the export unconditionally.

8 Mi Piace
5

30k chars still seems like a lot. Do we need that many characters in a cell? If so, why?

6

It’s due to giant posts. For example, many of our internal runbooks tend to be quite long. Likely only the post_edit staff action as Vinoth noted would have this many characters.

7

Also we storing both old and new post raws for post_edit staff actions in discourse/app/services/staff_action_logger.rb at 4383afb769d97bc9724d5448c0583b14c39782d0 · discourse/discourse · GitHub.

8

We should aggressively truncate this, I don’t see the point of creating such bulk in the database over something so trivial.

5 Mi Piace
9

Okay, I’ll try to improve this by storing only the edit differences in the raw text.

4 Mi Piace
10

Vorrei chiudere questa. Facciamo la cosa più semplice: limitiamo le celle a 30k, poi chiudiamo. Le modifiche relative al delta sono estremamente complesse.

4 Mi Piace
15

Riapro questa discussione. Dobbiamo implementare una qualche soluzione, è quasi impossibile rivedere i log esportati con post enormi che rompono la formattazione.

1 Mi Piace
22

Forse la soluzione più semplice qui è:

> '\"' +  'post \"body\" ' .gsub('\"', '\"\"') + '\"'
=> "\"post \"\"body\"\" \""

Penso che questo sia consentito in CSV e dovrebbe funzionare.