z0rglub a écrit:

A mon avis, la desactivation de l'acces a pwg_high va interesser tres peu de monde, inutile de faire quelque chose de complique pour un besoin aussi simple. N'oubliez pas que le code doit etre maintenu... Rub, ce post est une recommandation que je te conseille de suivre, mais je te laisse decider.

Justement ce midi, je me disais que j'allais faire compliquer pour par grand chose comparer aux besoins initials. Je devais partir sur le cache, ect... et surtout, le fait de mettre en place un nouveau champ dans #_groups pouvait porter à confusion (propriété pour les catégories ou les users du groupe), je pense que cela aurait embrouillé les utilisateurs.

Bref, j'avais du rajouter des requêtes, des calculs de cache, ect...

Donc, pour avancer dans la notification par mail et/ou pour ne pas compliquer pwg pour rien, je vais me contenter de rajouter le champs #enabled_high uniquement dans la table #_user_info

Je suis pour une revue des permissions mais effectivement dans le cas d'un plus grand chantier afin de bien prendre en compte les différentes demandes.

Mettons #enabled_high pour les groupes dans la version 1.7.

vimages a écrit:

zOrglub n'a pas tort.. :o)

Yep, et puis on s'emballe vite...

http://bugs.phpwebgallery.net/view.php?id=127 a été mis à jour

Dernière modification par rub (2006-03-13 18:50:33)

Dans [Subversion] r1085, tout est fini (ou presque):
  o les nouveaux status
  o pwg_high
  o adviser

vimages et VDigital, je ne sais pas si vous testez la BSF mais si c'est le cas, j'attends vos feedback! (notamment pour le adviser)

z0rglub a écrit:

Il faudrait ajouter de la documentation dans l'aide de l'écran de admin/user_list. J'aimerais bien que tu ajoutes $conf['allow_adviser'] à false par défaut, qui permettrait de cacher cette fonctionnalité avancée.

Pour l'écran d'aide, j'ai simplement mis un petit truc, je vais donc mettre une phrse plus détaillée.

Par de soucis pour le $conf['allow_adviser'], à false, la saisie du mode adviser ne sera pas possible et les tests sur le mode adviser actif seront à faux...

rub a écrit:

z0rglub a écrit:

Il faudrait ajouter de la documentation dans l'aide de l'écran de admin/user_list. J'aimerais bien que tu ajoutes $conf['allow_adviser'] à false par défaut, qui permettrait de cacher cette fonctionnalité avancée.

Pour l'écran d'aide, j'ai simplement mis un petit truc, je vais donc mettre une phrse plus détaillée.

Par de soucis pour le $conf['allow_adviser'], à false, la saisie du mode adviser ne sera pas possible et les tests sur le mode adviser actif seront à faux...

Après reflexion et un petit dodo, le paramétre $conf['allow_adviser'] servira uniquement pour afficher ou masquer les radio-bouttons de la proprieté adviser dans la modification multiple des utilisateurs.
Par contre, la fonctionnalité sera toujours active.

VDigital a écrit:

Je veux bien faire des tests, bien que jusqu'au 29 mars, je n'ai pas beaucoup de temps.

z0rglub une génération de BSF. STP.

Pas de soucis pour les tests!
Pour la génération , il faudrait que z0rglub attende la livraison du $conf['allow_adviser'] (afin la fin du WE normalement)

J'ai lu aussi dans ce sujet que les status webmaster et guest ne devaient pas être affectables par l'interface. C'est le cas en 1.6RC1.

J'ai vu la note du bug 301. L' invité n'était pas affectable via l'interface en 1.5. Je ne sais plus pourquoi il le devrait en 1.6. Quel est l'intérêt de créer plusieurs guests, puisque celui-ci correspond à un utilisateur non-connecté (d'où le paradoxe!).

Pour le webmaster, on peut à la rigueur vouloir nommer quelqu'un d'autre webmestre. Je pensais que c'était ce statut qui faisait la soupape superuser et donc forcément limité.

Des infos?

z0rglub a écrit:

rub a écrit:

vimages et VDigital, je ne sais pas si vous testez la BSF mais si c'est le cas, j'attends vos feedback! (notamment pour le adviser)

Il faudrait ajouter de la documentation dans l'aide de l'écran de admin/user_list. J'aimerais bien que tu ajoutes $conf['allow_adviser'] à false par défaut, qui permettrait de cacher cette fonctionnalité avancée.

Enfin...
Je vais le tester vraiment maintenant avec vimages... (toi aussi, je pense).
Il faut peut être qu'il nous mette l'un et l'autre "conseiller" et ...

$conf['allow_adviser'] = true;

8-)

Les options suivantes semblent toutes actives [Conseiller] même si je ne les ai pas toutes testées ([nettoyer]).

Donc:
[synchroniser] [supprimer] [tester] [générer le listing] [nettoyer] du Site manager
sont actifs même en Administrateur [Conseiller].

=> Bug 0000456 (je n'irai pas jusqu'à parler de trou de sécurité mais presque [nettoyer])

Et même...
Moi simple [Conseiller], je connais l'adresse du site externe.
Reste à le donner comme mien... dans ma galerie.
(C'était déjà vrai avant mais là cela devient flagrant).

[Conseiller] Sur la Notation, il peut supprimer des notes données...

Bug 0000457 .