Project

General

Profile

Evolution #4143

[Ajout constante] : DELAI_SUPPRESSION_AUTEUR_NON_VALIDE

Added by Guillaume Fahrner over 1 year ago. Updated over 1 year ago.

Status:
En cours
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
05/28/2018
Due date:
% Done:

0%

Resolution:

Description

on peut ajouter une constante dans le core ? pour pouvoir régler au bout de combien de temps on delete les auteur avec le statut=nouveau

Ca se passerait ici : https://core.spip.net/projects/spip/repository/entry/spip/ecrire/genie/optimiser.php#L201

History

#1 Updated by b b over 1 year ago

  • Target version set to 3.3

Bonne idée, reste à faire jouer la nomenklatura pour trouver un nom à cette constante ;)

#2 Updated by RastaPopoulos ♥ over 1 year ago

Pour le nom moi déjà j'ai toujours pas compris pourquoi et quand est-ce qu'on mettait un souligné au tout début ou pas.

Sinon est-ce qu'il y a déjà une ou des constantes qui travaillent autour du même sujet (pour se caler dessus) ?

Sinon un nom qui dit ce que ça fait, le plus court possible. Là le nom est un peu long… mais si on peut pas faire plus court pour comprendre ce que ça fait, ça me parait pas super grave, on l'utilise qu'une fois une constante.

#3 Updated by b b over 1 year ago

  • Status changed from Nouveau to En cours

#4 Updated by jluc - over 1 year ago

Vu quelques constantes déjà définies un peu en rapport :
_STATUT_AUTEUR_CREATION
_TAGS_NOM_AUTEUR
AUTEURS_MIN_REDAC (sans underscore)
_DELAI_CACHE_resultats (en partie en minuscules !)
_DELAI_RECUPERER_URL_CACHE
il vaut mieux un underscore au début :
_DELAI_SUPPRESSION_AUTEUR_NON_VALIDE
mais il y a aussi
_PETITIONS_DELAI_SIGNATURES_REJETEES
et ça pourrait donc aussi être
_AUTEURS_DELAI_REJET

#5 Updated by RastaPopoulos ♥ over 1 year ago

Ça me parait une bonne pratique de commencer par un identifiant commun à un groupement fonctionnel, comme pour un espace de nom.

Donc ouais moi ça me parait pas mal de commencer par _AUTEURS

#6 Updated by b b over 1 year ago

_AUTEURS_DELAI_SUPPRESSION_NON_VALIDE ?

#7 Updated by RastaPopoulos ♥ over 1 year ago

En moins verbeux _AUTEURS_DELAI_REJET ça me paraissait compréhensible mais sinon ça peut être encore plus direct par rapport à l'explication de départ : _AUTEURS_DELAI_NOUVEAU (= le délai durant lequel on garde les statuts "nouveau"), tout simplement non ?

#8 Updated by b b over 1 year ago

+1 pour un truc plus court, moins verbeux, et une préférence pour _AUTEURS_DELAI_NOUVEAU.

Also available in: Atom PDF