Annonce

  •  » Plugins
  •  » Insensible à la casse...

#316 2011-03-04 14:31:05

Whiler
Membre
Clichy
2004-12-24
189

Re: Insensible à la casse...

Eric a écrit:

Perso, une de mes galeries aligne au max 73 users. C'est trop peu...

Mais c'est 4 fois plus que moi ;o)))

Sinon, en parlant de bench... Luc, t'as comparé les perfs entre mb_string et l'expression régulière d'hier ? (j'l'ai pas fait, juste de la curiosité)

Hors ligne

#317 2011-03-04 18:29:23

Eric
Former Piwigo Team
VALENCE (FR)
2005-03-25
4579

Re: Insensible à la casse...

En marge des dev de fond sur LCAS, je viens de remarquer un bug de compatibilité avec le plugin [extension by flop25] Adult_content.
Ce plugin a la particularité de créer des users fictifs pour gérer les restrictions d'accès à des galeries au contenu "réservé à un public averti". Le truc avec ces users c'est leur forme : +18 et +16. Et çà, dans LCAS, ben çà passe pas bien. :-/

J'ouvre le [Bugtracker] ticket 2211 et me l'attribue pour régler çà en excluant les comptes créés par ce plugin. ;-)



D'un autre côté, cela soulève un problème plus sérieux sur LCAS proprement dit : Rien n'empêche un vrai user de s'inscrire avec un login du type 123+-. Et là, c'est le drame... :-s

Hors ligne

#318 2011-03-04 18:35:27

Whiler
Membre
Clichy
2004-12-24
189

Re: Insensible à la casse...

Faut juste ajouter un contrôle de l'indice dans la table ASCII ?
(comme d'hab.. j'ai pas le code sous les yeux....)

Hors ligne

#319 2011-03-04 18:46:15

Eric
Former Piwigo Team
VALENCE (FR)
2005-03-25
4579

Re: Insensible à la casse...

Whiler a écrit:

Faut juste ajouter un contrôle de l'indice dans la table ASCII ?

Pas que, non... Je n'ai pas encore fait de tests approfondis (ce soir peut-être, c'est dans ma todo list) mais il me semble que LCAS_change_case() n'aime pas les chiffres.

Hors ligne

#320 2011-03-04 19:11:10

Whiler
Membre
Clichy
2004-12-24
189

Re: Insensible à la casse...

Désolé.. j'ai pas été assez précis dans la rédaction de am pensée ;o))

J'voulais dire:
Si (65 <= Ord(CHAR) <= 90) | (97 <= Ord(CHAR) <= 122) Alors
  LCAS_change_case...
Sinon
  CHAR = CHAR...


Mais, en y repensant.. et comme on est en unicode.. ça tient pas la route.....

Et cela me ferait donc plutot dire :

Si LCAS_change_case(CHAR) == '' Alors
  CHAR = CHAR
Sinon
  CHAR = LCAS_change_case...

J'espère avoir été un peu plus clair ;o))

Hors ligne

#321 2011-03-04 21:00:58

Eric
Former Piwigo Team
VALENCE (FR)
2005-03-25
4579

Re: Insensible à la casse...

Whiler a écrit:

Désolé.. j'ai pas été assez précis dans la rédaction de am pensée ;o))

J'voulais dire:
Si (65 <= Ord(CHAR) <= 90) | (97 <= Ord(CHAR) <= 122) Alors
  LCAS_change_case...
Sinon
  CHAR = CHAR...


Mais, en y repensant.. et comme on est en unicode.. ça tient pas la route.....

Et cela me ferait donc plutot dire :

Si LCAS_change_case(CHAR) == '' Alors
  CHAR = CHAR
Sinon
  CHAR = LCAS_change_case...

J'espère avoir été un peu plus clair ;o))

LOL ! Oui, c'est plus clair et je crois que c'est ce qu'il faut faire pour éviter les problèmes avec des caractères numérique et spéciaux.

Hors ligne

#322 2011-03-05 00:02:01

LucMorizur
Membre
Vienne (Isère, 38)
2009-03-01
1969

Re: Insensible à la casse...

Pfou !!... moi, j'ai du mal à suivre !...

Whiler a écrit:

Eric a écrit:

Perso, une de mes galeries aligne au max 73 users. C'est trop peu...

Mais c'est 4 fois plus que moi ;o)))

Pas mieux

Whiler a écrit:

Sinon, en parlant de bench... Luc, t'as comparé les perfs entre mb_string et l'expression régulière d'hier ? (j'l'ai pas fait, juste de la curiosité)

Non je ne l'ai pas fait, mais l'ancienne procédure bouclait sur chacun des caractères du username proposé (donc entre 3 et 15 passages par analyse) ; la nouvelle procédure tenant en une seule instruction, j'imagine une amélioration sensible.

Il faudrait effectivement mesurer, ça ainsi que vos autres propositions avec beaucoup d'utilisateurs.

Hors ligne

#323 2011-03-05 00:12:42

Gotcha
Ex Equipe Piwigo
Pierrelatte (26)
2007-03-14
13331

Re: Insensible à la casse...

LucMorizur a écrit:

Pfou !!... moi, j'ai du mal à suivre !...

Et moi donc -_-;

Ok je sors...


Ayez comme premier réflexe de consulter le wiki.
Ensuite, veuillez effectuer une recherche sur le forum avant de poser votre question.

LE FAIRE EST LE REVELATEUR DE L'ETRE

Hors ligne

#324 2011-03-05 11:31:29

LucMorizur
Membre
Vienne (Isère, 38)
2009-03-01
1969

Re: Insensible à la casse...

Eric a écrit:

En marge des dev de fond sur LCAS, je viens de remarquer un bug de compatibilité avec le plugin [extension by flop25] Adult_content.
Ce plugin a la particularité de créer des users fictifs pour gérer les restrictions d'accès à des galeries au contenu "réservé à un public averti". Le truc avec ces users c'est leur forme : +18 et +16. Et çà, dans LCAS, ben çà passe pas bien. :-/

J'ouvre le [Bugtracker] ticket 2211 et me l'attribue pour régler çà en excluant les comptes créés par ce plugin. ;-)



D'un autre côté, cela soulève un problème plus sérieux sur LCAS proprement dit : Rien n'empêche un vrai user de s'inscrire avec un login du type 123+-. Et là, c'est le drame... :-s

Je regarderai ça ce soir.

Hors ligne

#325 2011-03-06 12:11:04

LucMorizur
Membre
Vienne (Isère, 38)
2009-03-01
1969

Re: Insensible à la casse...

Eric a écrit:

Rien n'empêche un vrai user de s'inscrire avec un login du type 123+-. Et là, c'est le drame... :-s

Je ne reproduis pas le problème...

Quel drame se passe lorsqu'un utilisateur s'enregistre avec le username "123+-" ? Je viens d'essayer sur ma galerie , tout s'est bien passé, même avec [extension by flop25] Adult_content d'activé. (Pas d'administration, d'ailleurs, sur ce plugin ?). Et LCAS était avait pour option l'insensibilité à la casse et aux accents.

Hors ligne

#326 2011-03-06 13:07:07

Eric
Former Piwigo Team
VALENCE (FR)
2005-03-25
4579

Re: Insensible à la casse...

Le problème que j'ai rencontré initialement n'était pas lié à l'utilisation conjointe de LCAS et [extension by flop25] Adult_content mais à l'activation de [extension by flop25] Adult_content lorsque LCAS est actif. A ce moment, j'ai ce message:

PHP Notice:  Undefined index: login in E:\\www\\PWG_2.2-RC3-MySql5.5.8\\plugins\\LCAS\\main.inc.php on line 123, referer: http://localhost/PWG_2.2-RC3-MySql5.5.8/admin.php?page=plugins_list
(...)

C'est un cas très particulier, j'en conviens.

Ceci dit, j'avais aussi des problèmes pour inscrire un user "123+" avec LCAS... Mais je n'arrive plus à reproduire et je n'ai pas eu le réflexe de copier le message d'erreur :-(

Je vais essayer de revoir çà à tête reposée.

Hors ligne

#327 2011-03-07 20:56:56

LucMorizur
Membre
Vienne (Isère, 38)
2009-03-01
1969

Re: Insensible à la casse...

Comment c'est long, 25 secondes...

Eric a écrit:

(...)
L'idéal serait de mesurer les temps de traitement avec le code actuel sur une table de plusieurs centaines de users (+ de 100 au minimum) et d'effectuer la même mesure avec d'autres méthodes (table de cache, requêtes full MySql,...).

Perso, une de mes galeries aligne au max 73 users. C'est trop peu...

Je suis vachement content, j'ai créé un petit script qui me rajoute des utilisateurs dans ma galerie 2.2.0RC3 en local. Le username a une longueur choisie au hasard en 3 et 15 caractères, et les caractères sont choisis au hasard entre chr(32) et chr(126) (je vous raconte pas les noms de mes inscrits). J'utilise quand même register_user() , histoire de faire les choses proprement. J'ai limité le script à 25 secondes de fonctionnement afin de ne pas dépasser les 30 secondes max de WampServer ; je pensais avoir à le faire tourner 2-3 fois pour atteindre le millier d'utilisateur...

Maintenant que j'ai plus de 4000 inscrits sur ma galerie (sur laquelle il n'y a aucune photo, en plus, les cons !!)... il faut que je me fasse un script qui en élimine la moitié :-/ ... ce sera quand même plus réaliste pour les tests...

Hors ligne

#328 2011-03-07 21:03:53

Whiler
Membre
Clichy
2004-12-24
189

Re: Insensible à la casse...

J'avais pas besoin de script... avant de mettre le plugin pour le CAPTCHA, j'avais bien une dizaine de users qui s'inscrivaient par jour avec des noms comme les tiens ;o))) et des sites web des plus originaux ! (si vous cherchez un crédit, du sexe, ... j'ai plein de liens ;o))

Hors ligne

#329 2011-03-07 21:07:09

Whiler
Membre
Clichy
2004-12-24
189

Re: Insensible à la casse...

Du coup, j'viens de revérifier... j'en ai moins, mais il y en a toujours qui passent... bon, ben j'vais refermer les inscriptions...

Hors ligne

#330 2011-03-07 23:14:06

LucMorizur
Membre
Vienne (Isère, 38)
2009-03-01
1969

Re: Insensible à la casse...

Mmm... je pense que j'ai pas trop besoin de ce genre de liens non plus ;-) ...


Bon, après réinstallation d'un galerie 2.2.0RC3, création de 2000 utilisateurs, pas de souci remarqué sur le fonctionnement de LCAS : refus d'un enregistrement avec un username déjà présent que ce soit en début, milieu, ou fin de table ; bon affichage des username en conflit dans la page d'admin... et fonctionnement normal à l'identification des utilisateurs.

Par contre, ces tests étaient en local ; le vrai test ce sera chez Free. Mais depuis la maison : à l'hôtel, ça le fait vraiment pas... (Je veux dire, rien que déjà pour transférer une galerie en 2.2.0RC3 par FTP.)

Hors ligne

  •  » Plugins
  •  » Insensible à la casse...

Pied de page des forums

Propulsé par FluxBB

github twitter newsletter Faire un don Piwigo.org © 2002-2024 · Contact