Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Limitation du nombre de flux par utilisateur #680

Closed
ldidry opened this issue Oct 26, 2014 · 12 comments
Closed

Limitation du nombre de flux par utilisateur #680

ldidry opened this issue Oct 26, 2014 · 12 comments

Comments

@ldidry
Copy link
Contributor

ldidry commented Oct 26, 2014

Il serait utile de pouvoir définir un nombre max de flux par utilisateur, surtout s'il y a beaucoup d'utilisateurs 😄

= 0 => open bar

!= 0 => limité à x flux par utilisateur

@Alkarex
Copy link
Member

Alkarex commented Oct 26, 2014

Voir #681 (comment)

@Alkarex Alkarex self-assigned this Oct 27, 2014
Alkarex added a commit that referenced this issue Oct 27, 2014
New 'limits' sub-array in config.php with 'max_feeds' and
'max_categories'. If the values are < 0, then it is the default value
(16384).
#680
@Alkarex
Copy link
Member

Alkarex commented Oct 27, 2014

Voilà @ldidry : Il y a maintenant une limite par défaut de 16384 sur le nombre de flux et sur le nombre de catégories, configurable pour l'instant à la main dans ./data/config.php

'limits' => array (
  'max_feeds' => 16384,
  'max_categories' => 16384,
),

Une valeur < 0 est transformée dans la valeur par défaut (16384), donc vous pouvez mettre -1 mais pas 0 qui vaut vraiment zéro.

En attendant une mise à jour, vous pouvez ajouter les nouvelles valeurs à la main, ou bien forcer l'écriture d'un nouveau fichier config.php en faisant un changement sur la page http://freshrss.example.org/i/?c=auth

@marienfressinaud Je n'ai pas fait la traduction des deux nouvelles chaînes, car je ne suis pas à jour sur la manière de faire l'i18n...

@Alkarex Alkarex removed their assignment Oct 27, 2014
@Alkarex
Copy link
Member

Alkarex commented Oct 27, 2014

Reste à faire :

  • i18n
  • éventuellement une possibilité de changer les valeurs depuis l'interface Web de l'administrateur

@aledeg
Copy link
Member

aledeg commented Oct 27, 2014

est ce qu'il y a une raison particulière pour utiliser 2^14 comme limite?

@Alkarex
Copy link
Member

Alkarex commented Oct 27, 2014

Nous avons actuellement une limite SMALLINT dans MySQL (32767) mais nous sommes aussi en auto-incrémentation, ce qui fait que nous avons des trous lorsque l'utilisateur supprime des flux/catégories. Pour le faire comme il faut, il faudrait une logique un peu plus élaborée. Mais cela laisse de la marge tout de même, je trouve :-)
Sinon pour changer, c'est https://github.com/marienfressinaud/FreshRSS/blob/c6f35ef51c2c536efb4f8e73603ae46b8512cc24/lib/Minz/Configuration.php#L63

@aledeg
Copy link
Member

aledeg commented Oct 27, 2014

je suis preneur pour voir la personne qui épuise ce compteur pour les catégories ^^

Alkarex added a commit that referenced this issue Oct 28, 2014
@marienfressinaud
Copy link
Member

Je m'occupe de la traduction. J'y réfléchissais ces derniers jours, en fait mon nouveau système n'est pas super pratique, ça l'était plus avant (mais moins visuel). Il faut que j'y réfléchisse encore mais il faudrait que l'on revienne à un système de traduction directement dans le dépôt.

@marienfressinaud
Copy link
Member

Je suis en train de corriger la fonction d'import pour tenir compte des limites ;)

marienfressinaud added a commit that referenced this issue Oct 29, 2014
marienfressinaud added a commit that referenced this issue Oct 29, 2014
@marienfressinaud
Copy link
Member

Clairement pas aussi simple que ce que je pensais, mais ça semble fonctionner pas trop mal. Les limites s'appliquent donc désormais aussi à l'importation des fichiers OPML ou Json.

@marienfressinaud marienfressinaud added UI 🎨 User Interfaces and removed Update action 💾 labels Dec 9, 2014
@marienfressinaud marienfressinaud added this to the 2.0.0 milestone Dec 9, 2014
@marienfressinaud
Copy link
Member

Je laisse ce ticket ouvert pour ajouter une interface pour gérer les limites. Ceci dit on pourrait aussi imaginer faire passer ça par une extension (bientôt).

@Alkarex
Copy link
Member

Alkarex commented May 16, 2015

#833

@Alkarex Alkarex added this to the 0.10-dev milestone Jul 5, 2015
@Alkarex Alkarex removed this from the 1.1.2 milestone Jul 5, 2015
@Alkarex
Copy link
Member

Alkarex commented Jul 5, 2015

The work for the UI continues in #892 and for the install process in #890

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants