Les grands messages dans les journaux du personnel perturbent le format CSV d'Excel

Contribuer Bug
1

Le personnel peut exporter les journaux d’activité du personnel depuis /admin/logs/staff_action_logs pour consulter l’historique complet, car seule une sous-ensemble de ces journaux est affichée dans l’interface d’administration. Certains journaux peuvent contenir des virgules dans les détails, ce qui perturbe la mise en forme du fichier CSV.

8 « J'aime »
2

Can commas be escaped in this output @vinothkannans?

4 « J'aime »
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 « J'aime »
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 « J'aime »
9

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

4 « J'aime »
10

Je voudrais clôturer celle-ci. Faisons la chose simple ici : limitons les cellules à 30 k, puis clôturons. Les modifications liées au delta sont extrêmement complexes.

4 « J'aime »
15

Réouverture de ce ticket. Nous devons mettre en place une solution, il est presque impossible de réviser les journaux exportés avec des messages géants qui brisent la mise en forme.

1 « J'aime »
22

Peut-être que la solution la plus simple ici est :

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

Je pense que cela est autorisé en CSV et devrait fonctionner.