PHP Funktion Umschreiben

Michael_

Hi,

Kann mir jemand beim Abändern dieser Funktion hier kurz helfen?

Code

function filecodereplace ($string1, $string2, $filename)
{
  $fp          = fopen($filename, "r");
  $size        = filesize($filename);
  $contents    = fread($fp, $size);
  fclose($fp);
  $file_replace = preg_replace("/$string1/", $string2, $contents);
  $fp          = fopen($filename, "w");
  fputs($fp, $file_replace);
  fclose($fp);
}

Alles anzeigen

Diese Funktion ermöglicht es mir in einer Datei eine Codestelle zu suchen und zu ersetzen.
Wenn diese mehrfach vorkommt wird sie auch mehrfach ersetzt.

Ich habe nun hier unzählige Datein in denen überall das selbe ersetzt werden soll.
Das wäre ein großer Aufwand diese funktion derart oft aufzurufen.

Ich hätte die gern so abgeändert das die funktion automatisch in ALLEN Datein die in dem Ordner wo die Datei mit der Funktion drin ist + ALLE Datein in Unterverzeichnissen
sucht und ersetzt.
Einzige Bedingung nur PHP Datein sollen bearbeitet werden.

Kann mir da jemand helfen?

Mittels dieses Scripts soll eine Webseite (Skript) automatisch auf PHP 7.1 umgeschrieben werden btw.
Da das CMS mehrfach eingesetzt wird eine extrem Zeitsparende Variante.

mrshadowtux

Muss es denn in PHP sein? Für die Aufgabe reicht doch auch ein Shell-Befehl auf dem betreffenden Linux-Server:

Code

find . -iname "*.php" -exec sed -i "s/suchbegriff/zielbegriff/gi" {} \;

Wenn du auf dem Server nur PHP zur Verfügung hast dafür, warten wir mal die anderen Antworten ab. Bin gerade zu müde, um mir dahingehend was zu überlegen.

s4ndwichMakeR

Das war auch mein erster Gedanke, aber ich vermute mal, der Kollege hat keinen Shell-Zugang.

Und ja … bin auch zu müde gerade.

Michael_

Muss mit PHP gemacht werden da nur Webspace.
Was noch gut wäre ist wenn ich in dem Script den Pfad zum Root ordner angeben könnte Datein aus selbigem und Datein aus Unterordnern selbigen sollen dann alle bearbeitet werden (alle PHP Datein, keine z.b. .js datein)

Ja muss auch nicht gleich sein würde morgen z.b. reichen

Michael_

Code

function multiplefilereplace ($string1, $string2)
{
foreach (glob("../../*.php") as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
}

Das hier funktioniert schonmal in mehreren Datein Allerdings nur im angegebenen verzeichnis.
Wie kann ich es erreichen das auch datein in den unterverzeichnissen bearbeitet werden?

**winfreak**

http://php.net/manual/de/clas…oryiterator.php

3sec Google.
Bitte niemals glob benutzen.

Das erste Beispiel ist, was du willst. $Regex dann im foreach nutzen und fertig.

Michael_

Ich hab die Funktion aktuell so abgeändert aber das ist nicht so das wahre

Code

function multiplefilereplace ($string1, $string2)
{
foreach (glob("../../*.php") as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
foreach (glob("../../archive/*.php") as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
foreach (glob("../../acp/*.php") as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
foreach (glob("../../acp/lib/*.php") as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
}

Alles anzeigen

Den sollte mal ein weiteres Verzeichnis eine bearbeitung benötigen dann wird es nicht berücksichtigt.

Code

function test ($string1, $string2) 
{
$Directory = new RecursiveDirectoryIterator('../../');
$Iterator = new RecursiveIteratorIterator($Directory);
$Regex = new RegexIterator($Iterator, '/^.+\.php$/i', RecursiveRegexIterator::GET_MATCH);
foreach ($Regex as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
}

Alles anzeigen

Ich hab diverses wie auch das probiert aber ich kriegs nicht zum laufen

**winfreak**

Nach einer Funktionsdefinition wird übrigens alles eingerückt. Man kann sich zwar dagegen entscheiden, ist aber einfach zu gängig.

"Ich hab diverses wie auch das probiert aber ich kriegs nicht zum laufen"

An was liegt es denn? Stimmt bei letzterem Beispiel denn die Pfadangabe? Sehen die Dateinamen richtig aus, wenn du mal $filename in der Schleife mit einem echo $filename, '<br>'; ausgeben lässt?

Michael_

Bei Glob habe ich ../../*.php als Pfad angegeben und das verzeichnis stimmt.

Array
Warning: file_get_contents() expects parameter 1 to be a valid path, array given in /www/htdocs/w0125294/script/updater/system/functions.php on line 24

echo $filename;
gibt garnix aus.

Code

function test ($string1, $string2)
{
$Directory = new RecursiveDirectoryIterator('../../');
$Iterator = new RecursiveIteratorIterator($Directory);
$Regex = new RegexIterator($Iterator, '/^.+\.php$/i', RecursiveRegexIterator::GET_MATCH);
foreach ($Regex as $filename)
{
echo $filename;
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
}

Alles anzeigen

Michael_

Ich habs mal so probiert

Code

function fullreplace ($string1, $string2)
{
foreach (glob("../../".'*') as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
}

Fehlermeldungen

Code

Warning: file_put_contents(../../acp): failed to open stream: Is a directory in /www/htdocs/w0125294/skript/update/system/functions.php on line 34


Warning: file_put_contents(../../archive): failed to open stream: Is a directory in /www/htdocs/w0125294/skript/update/system/functions.php on line 34


Warning: file_put_contents(../../attachments): failed to open stream: Is a directory in /www/htdocs/w0125294/skript/update/system/functions.php on line 34


Warning: file_put_contents(../../cache): failed to open stream: Is a directory in /www/htdocs/w0125294/skript/update/system/functions.php on line 34


Warning: file_put_contents(../../images): failed to open stream: Is a directory in /www/htdocs/w0125294/skript/update/system/functions.php on line 34


Warning: file_put_contents(../../js): failed to open stream: Is a directory in /www/htdocs/w0125294/skript/update/system/functions.php on line 34


Warning: file_put_contents(../../update): failed to open stream: Is a directory in /www/htdocs/w0125294/skript/update/system/functions.php on line 34

Alles anzeigen

Die angegeben verzeichnisse würden ja schonmal stimmen und alle haben CHMOD 777 die Datein auch,
woran liegt das?

Außerdem wäre es schön wenn ich das verzeichnis "update" ausschließen könnte aus der Liste.

**DosAmp**

Wenn du schon unbedingt glob nutzen willst, dann gibt glob('../../*') logischerweise den gesamten Inhalt des überübergeordneten Verzeichnis aus, also auch alle Vetter-Verzeichnisse. Du willst das vermutlich auf PHP-Dateien (*.php) einschränken.

Michael_

Ja und wie mache ich das?
Das alle PHP Datein in den gelisteten Verzeichnissen angezeigt/bearbeitet werden?

Ich versuch das nun seit stunden schon -.-

../../*
Zeigt mir zwar ne Fehlermeldung an und alle Ordner aber
../../*.php bearbeitet nur die Datein in dem root verzeichnis ../../

Es muss auch nicht glob sein hauptsache eine funktion die das tut was ich will -.-

mrshadowtux

Zwei Punkte:
1. Nutze statt glob die von winfreak vorgeschlagene Variante
2. Wenn dir jemand helfen soll, rück ordentlich ein, sodass Lesbarkeit gegeben ist

Michael_

Wenn mir jemand eine Funktion gibt die funktioniert mit der winfreak variante dann nehm ich die gerne.
Ich komm einfach nicht weiter jetzt helft doch bitte mal.

ENDLICH!

Code

function get_files ($dir,&$endungen)
{
  $result = array();
  foreach(glob($dir.'/*') as $file)
  {
    $path_parts = pathinfo($file);
    if(   is_file($file) &&
          isset($path_parts['extension']) &&
          in_array($path_parts['extension'],$endungen)
       ) $result[] = $file;
    if(is_dir($file))
          $result = array_merge($result,get_files($file,$endungen));
  }
  return $result;
}


function fullreplace ($string1, $string2)
{
$dir = '../../';
$endungen = array('php','sql');
foreach(get_files($dir,$endungen) as $filename)
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}
}

Alles anzeigen

Das bearbeitet alle meine Datein!
Allerdings würde ich nun gerne noch das verzeichnis "update" ausschließen.

BITTE wie krieg ich das hin?
Sonst überschreibt der auch in den updater Datein die Codestellen was ich nicht möchte.

MASTER FUNKTION

Code

function get_files ($dir,&$endungen)
{
  $result = array();
  foreach(glob($dir.'/*') as $file)
  {
    $path_parts = pathinfo($file);
    if(   is_file($file) &&
          isset($path_parts['extension']) &&
          in_array($path_parts['extension'],$endungen)
       ) $result[] = $file;
    if(is_dir($file))
          $result = array_merge($result,get_files($file,$endungen));
  }
  return $result;
}


function fullreplace ($string1, $string2)
{
$dir = '../..';
$endungen = array('php','sql');
foreach(get_files($dir,$endungen) as $filename)
{
if("../../mega_xxxx_updater" != substr($filename,0,23))
{
    $file = file_get_contents($filename);
    file_put_contents($filename, preg_replace("/$string1/", $string2, $file));
}        
}
}

Alles anzeigen

Funktioniert einwandfrei.
Fast ein ganzer Tag arbeit.

**winfreak**

PHP

<?php


function replace_occurences_of_string_in_php_files_at_path($path, $search_string, $replace_string)
{
    $directory = new Recursivedirectoryiterator($path);
    $iterator = new Recursiveiteratoriterator($directory);
    $php_files = new RegexIterator($iterator, '/^.+\.php$/i', RecursiveRegexIterator::GET_MATCH);


    foreach ($php_files as $file) {
        $filename = (isset($file[0]) === true && is_string($file[0])) ? (string) $file[0] : null;


        if (is_null($filename)) {
            echo 'ERROR: File structure changed, could not extract filename!';
            return false;
        }


        if (is_readable($filename) === false) {
            echo "ERROR: Could not read {$filename}!";
            return false;
        }


        if (is_writable($filename) === false) {
            echo "ERROR: Could not write to {$filename}!";
            return false;
        }


        $file_contents = file_get_contents($filename);
        $file_contents_replaced = str_replace($search_string, $replace_string, $file_contents);


        file_put_contents($filename, $file_contents_replaced);
    }


    return true;
}

Alles anzeigen

Das hier wäre eine, einigermaßen saubere, Lösung mit dem Iterator. Da ich nicht davon ausgehe, dass du mit Exceptions umgehen kannst, ganz klassisch mit return.

Ein bisschen mehr general purpose wird es so:

PHP

<?php


function recursive_replace($path, $file_extension = 'php', $search_string, $replace_string)
{
    $directory = new Recursivedirectoryiterator($path);
    $iterator = new Recursiveiteratoriterator($directory);
    $php_files = new RegexIterator($iterator, "/^.+\.{$file_extension}$/i", RecursiveRegexIterator::GET_MATCH);


    foreach ($php_files as $file) {
        $filename = (isset($file[0]) === true && is_string($file[0])) ? (string) $file[0] : null;


        if (is_null($filename)) {
            echo 'ERROR: File structure changed, could not extract filename!';
            return false;
        }


        if (is_readable($filename) === false) {
            echo "ERROR: Could not read {$filename}!";
            return false;
        }


        if (is_writable($filename) === false) {
            echo "ERROR: Could not write to {$filename}!";
            return false;
        }


        $file_contents = file_get_contents($filename);
        $file_contents_replaced = str_replace($search_string, $replace_string, $file_contents);


        file_put_contents($filename, $file_contents_replaced);
    }


    return true;
}

Alles anzeigen

Michael_

Danke!
str_replace hat das irgendeinen nachteil gegenüber preg_replace weil mit preg_replace komm ich grad garnicht klar wenn ich versuche z.b. mysql_query( mit mysqli_query($db, zu ersetzen.
Mit str_replace nun funktioniert das bestens daher bin ich ganz froh das ich die funktion nun auch habe.

Der Doktor

Nutz sichere Replace Funktionen

mrshadowtux

Der Vorteil von preg_replace ist, dass du reguläre Ausdrücke verwenden und somit auch Sachen suchen kannst, mit denen str_replace nix anfangen kann, str_replace kann nur Strings als ganzes ersetzen. preg_replace hingegen kann nach bestimmten Mustern im Text suchen und diese entsprechend anpassen. Ist also viel mächtiger.

**winfreak**

Zitat von Der Doktor

Nutz sichere Replace Funktionen

Was ist an str_replace() unsicher, bitte? Und was wäre eine sichere Alternative ;)?

Michael_

Ich habe nun ein Problem hier mit str_replace.

Wenn ich z.b.

Code

mysql_query($variable

im quelltext stehen habe und mit

Code

mysqli_query($variable

ersetzen will geht das ganz einfach mittels:

Code

singlefilereplace_modern("mysql_query(\$variable", "mysqli_query(\$variable", "../../acp/lib/class_parse.php");

Wenn ich nun aber als einfaches beispiel

Code

$this->replace[] = "\$this->formaturl('\\1')";

einfach nur entfernen möchte mittels

Code

singlefilereplace_modern("\$this->replace[] = \"\$this->formaturl('\\1')\";", "", "../../acp/lib/class_parse.php");

dann passiert nichts, auch wenn ich

Code

singlefilereplace_modern("\$this->replace[] = \"\$this->formaturl('\\1')\";", "JUNGE", "../../acp/lib/class_parse.php");

nehme passiert garnichts.
Woran genau liegt das? ich denke es hat mit den \ zu tun und somit Syntax aber ich weiß nicht wie ich das Problem beheben kann und was ich überhaupt falsch mache.

vllt. liegt es auch an meiner Funktion die ich hierfür benutze?

Code

function singlefilereplace_modern ($string1, $string2, $string3)
{
foreach (glob("$string3") as $filename)
{
    $file = file_get_contents($filename);
file_put_contents($filename, str_replace($string1, $string2, $file));    
}
}

Brauche ich dafür preg_replace? Falls ja wie gehe ich damit richtig um?
Ergo wie passe ich den Code der gesucht und ersetzt werden soll so an das er passt?

PHP Funktion Umschreiben

Jetzt mitmachen!

Benutzer online in diesem Thema