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.

Sign up for GitHub

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

Jump to bottom

PHP : Booléens plutôt que chaînes de texte #260

Closed
Alkarex opened this issue Nov 10, 2013 · 2 comments
Closed

PHP : Booléens plutôt que chaînes de texte #260

Alkarex opened this issue Nov 10, 2013 · 2 comments
Assignees
@Alkarex
Milestone
0.7.0

Comments

@Alkarex
Copy link
Member

Alkarex commented Nov 10, 2013

Considérer remplacer les nombreux x == 'yes' ou y = 'no' et similaires par de vrais booléens pour alléger à la fois l'écriture du code et améliorer les performances.
@Alkarex Alkarex mentioned this issue Dec 22, 2013
Open
Alkarex added a commit that referenced this issue Dec 28, 2013
@Alkarex
Bouge anon_access dans config.php
9ac1496 
L'accès anonyme n'est pas au niveau utilisateur mais au niveau global.
Bouge FreshRSS_Configuration::conf->anonAccess() qui était stocké dans
*_user.php vers Minz_Configuration::allowAnonymous() stocké dans
config.php
Contribue à #126
+ autres optimisations
Contribue à #260
Alkarex added a commit that referenced this issue Dec 28, 2013
@Alkarex
Suite nettoyage des types et chaînes
856d2c1 
Contribue à #260
Alkarex added a commit that referenced this issue Dec 28, 2013
@Alkarex
Refactorisation FreshRSS_Configuration
96bdbaf 
Implémente #260
(évite les comparaisons de chaînes au profit des vrais booléens et
entiers)
Grosse simplification et réduction du code relatif à la configuration.
Supprime ConfigurationDAO.
Permet de simplifier considérablement configureController.
Évite de multiples copies des mêmes données en mémoire.
Évite de garder plusieurs versions de la configuration en mémoire
(auparavant : dans un tableau au niveau de ModelArray + au niveau de
FreshRSS_Configuration + en Session + des copies temporaires comme
ConfigurationDAO).
Ne stocke plus 'conf' en Session (n'était presque pas utilisé).
Évite de recharger plusieurs fois Translate inutilement.
Contribue à #303
@Alkarex
Copy link
Member Author

Alkarex commented Dec 28, 2013

Implémenté

@Alkarex Alkarex closed this as completed Dec 28, 2013
@ghost ghost assigned Alkarex Dec 28, 2013
@Alkarex
Copy link
Member Author

Alkarex commented Mar 22, 2014

#303

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Alkarex Alkarex

Labels
None yet
Projects
None yet
Milestone
0.7.0
Development

No branches or pull requests
1 participant
@Alkarex