feat(pages): patrons page

[wJoenn]

Summary of changes and context

closes #304

Can probably be refactored a bit but I'm proud of that one query to get everything 👀

    @users = User
             .select("users.uid AS uid, users.username AS username, COUNT(referees.id) AS referrals")
             .select("CASE WHEN COUNT(referees.id) > 0 THEN CEIL(100 * LN(4)) ELSE 0 END AS aura")
             .joins("LEFT JOIN users referees ON users.id = referees.referrer_id")
             .group("users.id")
             .order(aura: :desc)

A User::aura method has been added to the model in case we wanna access the aura somewhere else at some point

  def self.aura(referrals)
    (100 * Math.log(1 + referrals, Math::E)).ceil
  end

I made it a class method because the #order(aura: :desc) in the afore mentioned Active::Record query would call the User#aura instead of the SELECT aura otherwise for some reason 🤔
I've trying doing #order("aura DESC") but it was doing the same.

The introduction text for the /patrons page is missing atm, waiting for @pil0u

Sanity checks

• Linters pass
• Tests pass
• Related GitHub issues are linked in the description

[pil0u]

Je review vendredi 👀

[Aquaj]

Manque des specs. (Ça aurait chopé l'erreur du LN(4) par exemple).

[Aquaj]

La logique de calcul de l'aura n'a pas sa place dans le controller. Plutôt à mettre dans une classe dédiée, ou au pire dans le modèle.

[wJoenn]

@Aquaj Je suis en train de reflechir a la remarque que tu fais et je comprend mais j'ai pas d'idée sur comment appliquer ca d'une meilleure facon sans perdre en performance.

La ce que la query me permet de faire c'est récupéré, en une seule query SQL, tous les Users, avec le nombre de referees qu'ils ont, et leur aura.

SELECT
  users.uid AS uid,
  users.username AS username,
  COUNT(referees.id) AS referrals,
  CASE WHEN COUNT(referees.id) > 0 THEN CEIL(100 * LN(COUNT(referees.id) + 1)) ELSE 0 END AS aura
FROM "users"
LEFT JOIN users referees ON users.id = referees.referrer_id
GROUP BY "users"."id"
ORDER BY "aura" DESC

J'ai l'impression que isolé le calcul de l'aura va nous forcer a répéter plusieurs queries au leiu d'en avoir qu'une seule comme ici.
Avec plus de 1000 users queried a chaques fois on a un besoin de faire au plus performant sur cette requete je penses, et donc de la maintenant en un bloque comme ici

[wJoenn]

@Aquaj C'est pour ca qu'on fait +1 dans la formule non ?

(100 * Math.log(1 + referrals, Math::E)).ceil

Comme ca si tu as 1 referee alors tu as 100 * ln(1 + 1)

[Aquaj]

Tout à fait. Elle manque juste à l'implem SQL en place. En allant revoir l'issue j'ai vu que le +1 était bien là donc j'ai supprimé mon message (j'avais espoir que personne l'ait vu 😄) pour aller édit ma review à la place.