[spip-dev] Liens ...

Quelques modifs dans inc_texte.php3 pour reconnaître

[->spip@rezo.net]

comme étant un lien

<a href="mailto:spip@rezo.net">spip@rezo.net</a>

En ligne 632, ajouter :

if (!$lien_texte && ereg("[^@]+@[^@]+\.[^@.]+", $lien_url)) {
  $lien_texte = $lien_url;
  $lien_url = ‘mailto:’.$lien_url;
}

L'expression régulière est un peu basique mais devrait suffire, il me semble.

Sinon, pourquoi y-a-t'il :

if ((!$lien_texte) and (!$lien_interne))
  $lien_texte = "<tt>$lien_url</tt>";

Et non tout simplement :

if ((!$lien_texte) and (!$lien_interne))
  $lien_texte = $lien_url;

-Nicolas

Tu peux faire un peu mieux !

@ Nicolas Hoizey (nhoizey@phpheaven.net) :

if (!$lien_texte && ereg("[^@]+@[^@]+\.[^@.]+", $lien_url)) {
  $lien_texte = $lien_url;
  $lien_url = ‘mailto:’.$lien_url;
}

L'expression régulière est un peu basique mais devrait suffire, il me semble.

Il y a des URLs http://;;;;;;@;;;;;;;, et jamais de : ni de // dans les
mails... On doit trouver un peu partout sur le web des ereg qui correspondent
aux mails, cherchons un peu ?

Il faudrait aussi détecter [->www.google.com] comme un http://

Sinon, pourquoi y-a-t'il :

if ((!$lien_texte) and (!$lien_interne))
  $lien_texte = "<tt>$lien_url</tt>";

Et non tout simplement :

if ((!$lien_texte) and (!$lien_interne))
  $lien_texte = $lien_url;

euh, c'est plus joli comme ça... tu as raison, il faudrait au minimum <tt
class='spip'>

-- Fil

Tu peux faire un peu mieux !

Ah bin oui, j'avais bien dit que c'est basique, je voulais déjà voir si ça vous
intéressait ... :slight_smile:

Il y a des URLs http://;;;;;;@;;;;;;;, et jamais de : ni
de // dans les mails...

En effet.

On doit trouver un peu partout sur le web des ereg qui
correspondent aux mails, cherchons un peu ?

On ne va quand même pas valider l'exactitude des adresses, cela prendrait trop
de ressources vu les expressions régulières qu'il faut (cf les archives des
newsgroups).

Donc on doit pouvoir s'en sortir avec :

[.a-zA-Z0-9_+-]+@[^@]+\.[^.@]+

C'est mieux ?

Il faudrait aussi détecter [->www.google.com] comme un http://

En gros, si ce n'est ni un lien interne, ni un mailto, on considère que c'est
un lien externe ???

$lien_texte = "<tt>$lien_url</tt>";
vs
$lien_texte = $lien_url;

euh, c'est plus joli comme ça... tu as raison, il faudrait
au minimum <tt class='spip'>

Bin pourquoi un 'tt' surtout ??? Pourquoi vouloir mettre les liens externes
sans libellé en police de taille fixe ? C'est moche et pas fait pour.

-Nicolas

@ Nicolas Hoizey (nhoizey@phpheaven.net) :

[.a-zA-Z0-9_+-]+@[^@]+\.[^.@]+
C'est mieux ?

ouiche

> Il faudrait aussi détecter [->www.google.com] comme un http://

En gros, si ce n'est ni un lien interne, ni un mailto, on considère que c'est
un lien externe ???

euh, non, car [->/toto] c'est pas http:///toto… non, c'est juste le www :wink:

>> $lien_texte = "<tt>$lien_url</tt>";
>> vs
>> $lien_texte = $lien_url;
>
> euh, c'est plus joli comme ça... tu as raison, il faudrait
> au minimum <tt class='spip'>

Bin pourquoi un 'tt' surtout ??? Pourquoi vouloir mettre les liens externes
sans libellé en police de taille fixe ? C'est moche et pas fait pour.

alors <a class='spip_url'> en lieu et place de <a class='spip_out'> ?

-- Fil

[.a-zA-Z0-9_+-]+@[^@]+\.[^.@]+
C'est mieux ?

ouiche

Coooool ... :wink:

En gros, si ce n'est ni un lien interne, ni un mailto, on considère
que c'est un lien externe ???

euh, non, car [->/toto] c'est pas http:///toto… non, c'est
juste le www :wink:

Tous les sites n'ont pas un 'www' en sous-domaine, donc pourquoi ne pas forcer
les utilisateurs à écrire correctement leurs URLs avec 'http://' ?

alors <a class='spip_url'> en lieu et place de <a class='spip_out'> ?

Je vote pour, au détail près que c'est <a class="spip_url"> qu'il faut mettre,
avec des doubles quotes ... :slight_smile:

-Nicolas

Heu, dites.... c'est pas dans la 1.2, si ?

Fil wrote:

Heu, dites.... c'est pas dans la 1.2, si ?

Perso, j'aimerais bien que ça y soit, mais je n'ai pas les clefs pour
entrer, donc à vous de voir ... :wink:

-Nicolas

@ Nicolas Hoizey (nhoizey@phpheaven.net) :

> Heu, dites.... c'est pas dans la 1.2, si ?

Perso, j'aimerais bien que ça y soit, mais je n'ai pas les clefs pour
entrer, donc à vous de voir ... :wink:

Vu qu'on est en version "pr"éalable, on n'ajoute pas de nouvelles
fonctionnalités, sinon on ne la sort jamais la 1.2 ! Donc : corrections de
bugs, doc, tests, et c'est tout. On peut en revanche préparer ses modifs
pour la suite...

-- Fil

Vu qu'on est en version "pr"éalable, on n'ajoute pas de nouvelles
fonctionnalités, sinon on ne la sort jamais la 1.2 ! Donc :
corrections de bugs, doc, tests, et c'est tout. On peut en revanche
préparer ses modifs pour la suite...

Selon moi, virer les '<tt>' des liens externes sans label, c'est
justement de la correction de bug.

Pour les mailto automatiques, c'est tendancieux ... :slight_smile:

Pour le coup de l'Output Buffering, c'est clair que ce sera pour la
prochaine fois, si vous voulez bien y réfléchir avec moi ...

-Nicolas

@ Nicolas Hoizey (nhoizey@phpheaven.net) :

Selon moi, virer les '<tt>' des liens externes sans label, c'est
justement de la correction de bug.

d'acc. Corrigé.

Pour le coup de l'Output Buffering, c'est clair que ce sera pour la
prochaine fois, si vous voulez bien y réfléchir avec moi ...

la situation actuelle pourrait être considérée comme un bug, mais le risque
est trop élevé

-- Fil

Pour le coup de l'Output Buffering

la situation actuelle pourrait être considérée comme un bug, mais le
risque est trop élevé

Je suis d'accord.

-Nicolas