Une campagne récente de ré-étiquetage massif a montré un effet des étiquettes populaires dans le bas de page :

https://pix.toile-libre.org/upload/original/1610822726.png

(l'évolution de la courbe correspond en fait à l'évolution temps de rendu de _layouts/_footer.html.haml )

Bref on va avoir besoin d'une mise en cache (pour éviter la sensibilité sur les étiquetages du dernier mois).

Les étiquettes elle-mêmes sont gérées par la table SQL tags et les étiquetages (une étiquette pour un utilisateur sur un contenu) via taggings. tag.taggings_count décompte le nombre d'étiquetages (quel que soit le contenu et l'utilisateur).

Normalement Ruby On Rails devrait gérer l'incrémentation et la décrémentation du compteur.

L'incrémentation marche bien. La décrémentation (dans le cas où l'étiquetage est supprimé par son utilisateur), voire le cas où le seul étiquetage existant est supprimé par son utilisateur ne fonctionne pas correctement.

Debug en cours… Une dépêche est créée en rédaction (en markdown donc). À un moment, le markdown est perdu. Il ne reste plus que la version HTML. On se retrouve avec l'interface utilisée pour les dépêches pré-2011 en HTML.

La dépêche en question : https://linuxfr.org/redaction/news/sortie-de-crowdsec-1-0-tutoriel-d-utilisation

Recherche d'éléments :

• pas d'erreurs, pas de HTTP 500

• frise chronologique : dépêche soumise en markdown (Révision nᵒ 1 - 13 décembre 2020 13:31:37), "Réorganisation" par la modération, renvoi de l'HTML (Révision (…)

• avoir accès facilement à tout ce qui est masqué pour un utilisateur donné
• pouvoir facilement démasquer sans un passage en base de données
• pouvoir voir facilement toutes les contributions à une dépêche d'un utilisateur donné et pouvoir les masquer

En modération, sur le markdown, avec Firefox au moins, il n’est pas possible de taper une correction via Grammelecte avec le caractère 'g' : ce caractère est intercepté, en raison du « retourner en haut de la page en appuyant sur g ou en bas de la page avec G » sur LinuxFr.org.

(ça n'a aucun rapport avec les 22 ans, mais bon d'ici qu'on le fasse, « nettoyage de printemps » sera hors saison)

Un peu de ménage en base de données ne ferait pas de mal, pour diverses raisons (ça prendrait moins de place, on stocke des choses inutiles, on stocke des données personnelles obsolètes pour rien, etc.).

Commençons par un nettoyage suite au Un an après la mise à jour majeure du site, grand nettoyage dans les comptes utilisateur (…)

Dans app/models/comment.rb

  def wikify_body
    nofollow = user.account.karma < 50
    self.body = wikify(wiki_body, nofollow: nofollow)
  end

du coup toute édition (y compris par un admin) d'un commentaire appartenant à un compte supprimé (entrée dans accounts supprimée mais entrée encore présente dans users) échoue :

Completed 500 Internal Server Error in 16ms (ActiveRecord: 1.7ms)
NoMethodError (undefined method `karma' for nil:NilClass):
app/models/comment.rb:42:in `wikify_body'
app/controllers/comments_controller.rb:64:in `update'

A priori s'il n'y a plus (…)

Corriger les infos visibles (par les admins au moins) pour avoir la date de dernière visite et pas juste de dernière connexion.

(détection via https://linuxfr.org/users/delgatux/liens/l-autorite-de-la-concurrence-bouge-enfin-sur-l-economie-numerique )

Le lien a été soumis ainsi :

"tags"=>"droit, concurrence, abus de position dominante, marché, facilités essentielles", "commit"=>"Poster le lien"

Les étiquetages créés : "droit", "concurrence", "abus", "marche" (et pas facilites).
Mais les étiquettes incrémentées comprennent notamment "position", "dominante", "facilités" et "essentielles" alors qu'aucun étiquetage n'existe pour ces tags (bref on a créé les étiquettes, mais elles ne correspondent à aucun étiquetage).

Du coup le script de vérification de la base SQL a râlé :

12718 position (…)

Ne pas afficher le « Aller plus loin » s’il n’y a pas de liens dans une dépêche.

(signalement initial : https://linuxfr.org/news/sauver-un-disque-dur-mecanique#comment-1799315)

Sur la page des statistiques sur les utilisateurs, la statistique des contenus produits par les visiteurs actifs dans les 3 et 12 derniers mois est erronée (elle ne peut être que décroissante car les visiteurs ont publié depuis Epoch plus que depuis 12 mois et plus que depuis 3 mois, et elle doit être différente par type de contenu).

Dans le champ Identifiant de "Se connecter", renvoyer une erreur explicite si l'utilisateur saisit une adresse de courriel (globalement un identifiant ne respectant pas les règles des identifiants).

Les interdictions temporaires de pouvoir écrire des commentaires ou sur la tribune ne sont pas assorties d'une perte de karma.

Il serait logique que ça soit le cas. (et que la doc soit mise à jour en conséquence sur
https://linuxfr.org/aide#aide-karma )

Dans app/models/account.rb

  def block(nb_days, user_id)
    $redis.set("block/#{self.id}", 1)
    $redis.expire("block/#{self.id}", nb_days * 86400)

Éviter que le Markdown ne produise de l'HTML invalide

• Hyperlien décrit, mais sans URL

[Description d'un lien mais pas de lien]()

avec href vide invalide

<a href="">Description d'un lien mais pas de lien</a>

• Tableau mais sans contenu

|Colonne 1|Colonne 2|
|---------|---------|

avec un <tbody> vide invalide

<table>
<thead>
<tr>
<th>Colonne 1</th>
<th>Colonne 2</th>
</

Identifier la source de l'entrée redis louche users//diff_karma

Indices :

• en prod, revient régulièrement si on la vire
• n'existe pas en dév
• contient 20 soit le karma par défaut actuellement

app/models/node.rb:    $redis.incrby("users/#{self.user_id}/diff_karma", value)
app/models/account.rb:    self.karma += $redis.get("users/#{self.user_id}/diff_karma").to_i
app/models/account.rb:    $redis.del("users/#{self.user_id}/diff_karma")
app/models/comment.rb:      $redis.incrby("users/#{self.user_id}/diff_karma", value)