Discussion :

[Invité]

On dirait un dialogue de sourds ... où chacun prêche pour sa paroisse ...

J'avais écrit "Hello boys" parce que vous êtes plusieurs !
Donc Shikiryu, inutile de prendre mes commentaires "personnellement".

J'ai aussi écrit "J'ai essayé aussi t'intégrer href="https: ..."
Ca parle donc de MES essais de code, ce n'est pas une critique sur le tien ...

La problématique étant de "supprimer tous les liens externes",
-> http et https sont a prendre en compte,
-> il faut tester avec tous les cas "viables" possibles.

C'est une question intéressante, et la solution en intéressera sûrement plus d'un.

Pas de quoi se fâcher, donc.

[stealth35]

utilise strip_tags et c'est réglé pas besoin de bidouille avec les regex

[Shikiryu]

Donc Shikiryu, inutile de prendre mes commentaires "personnellement".

J'ai aussi écrit "J'ai essayé aussi t'intégrer href="https: ..."
Ca parle donc de MES essais de code, ce n'est pas une critique sur le tien ...

La problématique étant de "supprimer tous les liens externes",
-> http et https sont a prendre en compte,
-> il faut tester avec tous les cas "viables" possibles.

Excuse moi si j'ai pu paraitre non-aimable ou autre, je ne le prend pas personnellement, au contraire, tout est constructif.
L'intervention de lucas était bonne, l'utilisation de parse_url est plutôt meilleure qu'un strpos que je proposais.
Mais formulé comme tu l'as fait, on dirait plutôt que tu testes "juste" "nos" propositions. Il aurait été judicieux de montrer le code testé plutôt que seulement leurs résultats...

Une histoire de malentendu

utilise strip_tags et c'est réglé pas besoin de bidouille avec les regex

Alors, là, je ne vois pas du tout en quoi ça résoudrait le problème. L'OP ne veut pas supprimer les balises html (sinon pourquoi utiliser CKEditor...) mais seulement supprimer les url des liens s'ils ne sont pas d'un domaine donné car c'est une demande du cahier des charges.

[stealth35]

Alors, là, je ne vois pas du tout en quoi ça résoudrait le problème. L'OP ne veut pas supprimer les balises html (sinon pourquoi utiliser CKEditor...) mais seulement supprimer les url des liens s'ils ne sont pas d'un domaine donné car c'est une demande du cahier des charges.

tu peux choisir les balises que tu veux supprimer avec strip_tags, suffis de virer juste les <a>, sinon après si ont veux faire plaisir a son gentils chef et perdre du temps il faut utiliser DOMDocument et virer les liens grace a parse_url

EDIT : exemple

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
$text = '<a href="http://www.blabla.com/test/test.php">essai url</a>';
$doc = new DOMDocument();
 
$fragment = $doc->createDocumentFragment();
$fragment->appendXML($text);
$doc->appendChild($fragment);
 
$links = $doc->getElementsByTagName('a');
 
foreach($links as $link)
{
    $host = parse_url($link->getAttribute('href'), PHP_URL_HOST);
 
    if($host !== $_SERVER['HTTP_HOST'])
    {
        $link->setAttribute('href', '#');
    }
}
 
echo $doc->saveHTML();

[lucas74]

mais ??!
ca marche ça non :
(regardez la fonction de test et l'output !!)

chezmoiçamarche

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
<?php
function search($s)
{
    static $authorized_hosts = array('google.com');
    $url = parse_url($s[1]);
    if(array_key_exists('host', $url)
       && !in_array($url['host'], $authorized_hosts))
    {
        return 'href="#"';
    }
    return $s[0];
}
 
function antispam($input)
{
       // on va chercher tous les href sans se préoccuper du domaine
	$pattern = '#href="((http:\/\/)?(www\.)?(.*?)")#'; 
	return preg_replace_callback($pattern,"search", $input);
}
 
 
echo antispam('<a href="http://google.com/">lolwut</a> <a href="http://127.0.0.1/">xd</a>');

Output :
<a href="http://google.com/">lolwut</a> <a href="#">xd</a>

[stealth35]

mais ??!
ca marche ça non :
(regardez la fonction de test et l'output !!)

oui ça marche mais c'est de la bidouille, pour parser du HTML c'est DOMDocument

[lucas74]

oui ça marche mais c'est de la bidouille, pour parser du HTML c'est DOMDocument

on est d'accord, l'idée cest d'utiliser parse_url et pas une regex

[Arget]

Whahou ! Je pars en Week End, et voilà une grande quantité de réponses, je n'en demandais pas tant ! xD
Alors j'ai pas mal suivit le débat, et je vais bien essayer tout cela, mais seulement à partir de demain, je ne recommence pas avant demain !

En tout cas je vous remercie fortement, et m'excuse par la même occasion pour le temps que j'ai mit à vous répondre !

[Arget]

Bon alors j'ai testé celui de Lucas74, et ça marche parfaitement !
Merci beaucoup en tout cas !