Url`s automatisch in links umwandeln

mike36

Hallo, hab mit folgendem script das Problem, das meine url`s aus der datenbankabfrage nivht in anklickbare links umgewandelt werden.

PHP

<?php


function html_links($str) {
 $str = preg_replace('"(((f|ht){1}tp://)[a-z0-9-.] [.]{1}[a-z]{1,6}[-a-z0-9@:%_$.! *, .~#;?&//=]*)"i', '<a href="$1">$1</a>', $str);
 $str = preg_replace('"([snr>]{1}|^)((www.)[a-z0-9-]{1,255}[.]{1}[a-z]{1,6}[-a-z0 -9@:%_$.!*, .~#;?&//=] )"i', '$1<a href="http://$2">$2</a>', $str);
 $str = preg_replace('"(([snr>]){1}|mailto:|^)(([-a-z0-9@:%_$.!*,; .~#;?&//=] )@([a-z0-9-.] [.]{1}[a-z]{1,6}))"i', '$2<a href="mailto:$3">$3</a>', $str);
 $str = preg_replace('"(((f|ht){1}tp://)([0-9]{1,3}([./sn]|$){1}){4}[-a-z0-9@:%_$.!*, .~#;?&//=]*)"i', '<a href="$1">$1</a>', $str);
 return $str;
 }


    $host = "localhost";     
    $user = "user";            
    $pass = "password";             
    $dbase = "database";            


     


    $connection = mysql_connect("$host" , "$user" , "$pass") 


                  OR die ("Keine Verbindung zu der Datenbank moeglich.");


    $db = mysql_select_db($dbase , $connection) 


                  OR die ("Auswahl der Datenbank nicht moeglich."); 

    $x = $_GET['nid'];


    $query = "SELECT * FROM files,node_revisions WHERE files.nid = '$x' AND node_revisions.nid = '$x' AND files.filename = '_original' ";

    $abfrage = mysql_query($query) or die (mysql_error());

    while ($row = mysql_fetch_array($abfrage)) {

        $title = $row['title'];
        $bild = $row['filepath'];
        $body = $row['body'];

        echo "<tr><td><strong>".$title."</strong></td></tr><br><br>";
        echo "<tr><td><img src=\"".$bild."\" title=\"\" alt=\"\"></td><tr>\n";
        echo "<tr><td><pre>".html_links($body)."</pre></td></tr><br>";

        echo "<a href=\"index.php\">zurück</a>";

    }
    mysql_close();
?>

Alles anzeigen

Arne Drews

Hallo,

Grundsätzlich kann man dazu nur etwas sinnvolles sagen, wenn Du etwas mehr Infos gibst.
Bspw. kennen wir Deine Funktion html_link() nicht.

Zudem:

mysql_* Funktionen sind deprecated, nutze eine der Alternativen MySQLi oder PDO.
Nutze kein SELECT *, sondern gib die Spalten an, die Du brauchst: SELECT `title`, `filepath`, `body`
Wo soll der klickbare Link sein, bzw. aus welchem Feld soll der generiert werden?

Gruß Arne

mike36

Meine Funktion html_link() steht gleich am anfang von dem Code den ich gepostet habe.
Meine SELECT-Abfrage habe ich jetzt folgendermasen umgeändert:

PHP

$query = "SELECT files.nid, files.filepath, files.filename, node_revisions.nid, node_revisions.title, node_revisions.body FROM files,node_revisions WHERE files.nid = '$x' AND node_revisions.nid = '$x' AND files.filename = '_original' ";

Der Text der aus der Datenbank gelesen werden soll stammt aus der Tabelle "node_revision.body" und wird hier ausgelesen und soll dann in "klickbare" Links umgewandelt werden:

PHP

while ($row = mysql_fetch_array($abfrage)) {

        $title = $row['title'];
        $bild = $row['filepath'];
        $body = $row['body'];

        echo "<tr><td><strong>".$title."</strong></td></tr><br><br>";
        echo "<tr><td><img src=\"".$bild."\" title=\"\" alt=\"\"></td><tr>\n";
        echo "<tr><td><pre>".html_links($body)."</pre></td></tr><br>";

        echo "<a href=\"index.php\">zurück</a>";

    }

Alles anzeigen

Die Ausgabe der abgefragten Dateien funktioniert, nur werden mir eben die Links aus dem Text welchen ich abfrage nicht in "klickbare" links umgewandelt.

Arne Drews

Hallo,

Zitat von mike36

Meine Funktion html_link() steht gleich am anfang von dem Code den ich gepostet habe.

Stimmt. Hatte ich übersehen.

Zitat von mike36

Die Ausgabe der abgefragten Dateien funktioniert, nur werden mir eben die Links aus dem Text welchen ich abfrage nicht in "klickbare" links umgewandelt.

Ach so, Du meinst Links, die sich quasi im node_revisions.body befinden?

Ja, Deine Pattern matchen nicht.
Ich habe mal als Test ein ganz einfaches Pattern verwendet:

PHP

$str = preg_replace( '#((f|ht)tp://[\w.-_]+\.[a-z]{2,5})#i', '<a href="$1">$1</a>', $str );

Das ist zwar kein sehr genaues Pattern, aber das matcht zumindest.

Du musst also Deine Pattern überarbeiten, der Rest des Scriptes scheint zu stimmen.

Gruß Arne

mike36

Also, du hattest Recht. Das Problem waren die Pattern. Hab jetzt die komplette Funktion total neu geschrieben und jetzt gehts.

PHP

function html_links($str, $attributes=array()) {
  $attrs = '';
  foreach ($attributes as $attribute => $value) {
    $attrs .= " {$attribute}=\"{$value}\"";
  }
$str = ' ' . $str;
$str = preg_replace(
  '`([^'=\'>])(((http|https|ftp)://|www.)[^\s<]+[^\s<\.)])`i',
  '$1<a href="$2"'.$attrs.'>$2</a>',
  $str
);
$str = substr($str, 1);
$str = preg_replace('`href=\"www`','href="http://www',$str);
return $str;
}

Alles anzeigen

Hab aber absolut keine Ahnung, was da bei meinem ersten Versuch nicht gepasst hat, aber jetzt gehts ja.
Warum einfach wenn mans auch kompliziert nachen kann.
Vieen Dank für deine Hilfe.

Arne Drews

Hallo,

Warum setzt Du das erst in Backticks und entfernst die danach wieder

EDIT: Grad gesehen, Du nutzt die als Delimiter, war ich geistig nicht drauf vorbereitet...
Mit diesem Pattern greifst Du auch sftp und subdomains ab:

Code

#(\s+)?((s?f|ht)tps?://([\w\d-]+\.)?[\d\w-]+\.[a-z]{2,5}(.*)?)(\s+)?#i

Geht sicherlich besser, aber für Dein Zweck sollte das passen.

Gruß Arne

mike36

Danke,
an sftp`s und subdomains hatte ich garnicht gedacht.

Vielleicht sollte man einfach so spät in der Nacht nicht noch rumprogrammiern/experimentieren:-)

Tags