Das Script ist überhaupt nich php 7 fähig schon alleine die datenbank klasse geht nur php 5.6Zitat von isaack
Rede nun mit mit den admin dann werde ich nun mal komplett das VMSone auf php min. php 7 schreiben.
Es würden dann mit absprache mit den Admin Therads von mir an gelegt und ca 1h am Tag investiert.
mfg jens
wenn man so deine Kommentare liest, und mal genau überlegt und auch das script genau anschaut, dann würde man mitbekommen, das 5.6 überhaupt keine sachlage mehr ist.
was du hier machst, ist einfach mal wieder oder sinn und verstand andere schlecht machen und dich selber als gott aufspielen.
anstatt immer wieder nur solche behauptungen in den raum zu stellen, leg doch mal beweise vor. zeige mir aus der datenbank klasse doch mal, warum es angeblich nicht auf php 7 laufen würde.
was mir aber am wichtigsten ist, ist zu wissen was du mit deinem getue und deinen ganzen äusserungen und bla bla erreichen willst hier.
Hallo,
Ich mag keinen unsicheren Code Punkt.
Und den hast du.
Schon alleine wie du meine Datenbank verwendest.
<php>
while( $r = ( !$object ) ? $results->fetch_object() : $results->fetch_object() )
{
$row->{$i} = $r;
$i++;
}
</php>
Werde ja noch wissen für was ich das geschrieben habe und du verändert hast.
Die Funktion dient dazu das man festlegen kann ob man ein Objekt oder ein Arry aus der abfrage haben will.
geht ja nun nicht mehr.
Brauch man wenn mann ein Array hat und noch andere hinzu fügen will.
Werde nächste Woche für VMSone eine PHP 7 Datenbankklasse veröffentlich und das VMSone auf einen neuen Stand bringen und mit mehr Sicherheit.
Und ich will nix bla bla machen sondern mag nicht das jemand meinen Code benutzt und dann noch falsch benutzt.
Desweiteren wenn du schon Klasse machst und benutzt dann mache es richtig. Globale Variablen haben nix mehr in Klassen zu suchen vorallem ab PHP 7 nicht mehr.
Desweiteren nutzt du ereg und das gibt es nicht mehr https://www.php.net/manual/de/function.ereg.php
preg_match wird so genutzt https://www.php.net/manual/de/function.preg-match.php
dirname(__DIR__) wird ich dir empfehlen nicht $_SERVER['DOCUMENT_ROOT']
Dein Script kann man nicht in einem sub Ordner oder im wamp xampp installieren und anschauen.
mfg jens
Geändert von jenssie (20.04.2019 um 14:52 Uhr)
stimmt, wenn man nur den teil sieht den du hier raus kopierst, dann kann es auch nicht gehen. schaut man sich allerdings die ganze function in der class an sieht es anders aus:
wie in der zeile 1 zu sehen, ist $object auf false gesetzt, wenn man allerdings die function aufruft und diese variable auf true setzt, bekommt man das objekt zurück. ansonsten ein array.PHP-Code:public function get_results( $query, $object = false )
{
self::$counter++;
//Overwrite the $row var to null
$row = null;
$results = $this->link->query( $query );
if( $this->link->error )
{
$this->log_db_errors( $this->link->error, $query );
return false;
}
else
{
$row = new stdClass();
$i = 1;
while( $r = ( !$object ) ? $results->fetch_object() : $results->fetch_object() )
{
$row->{$i} = $r;
$i++;
}
return $row;
}
}
aber das kannst du ja nicht wissen, da es ja auch nicht deine function ist wie du behauptest.
vielleicht sollte ich mal hier den ersteller der class anschreiben und mal deine daten geben, damit er sich bei dir melden kann.
Das hättest du natürlich auch schon lange machen können. sowas brauchst du ja nicht erst ankündigen.
Du denkst immer noch das es deiner ist. das finde ich echt witzig.
Warum sollte man es nicht mehr machen? Es hat jeder sein eigenen Programmierstyle. Das ist eben meiner.
Wo hast du den das gesehen?
Das ist doch mal eine ordentliche kritik, mit der man arbeiten kann.
Also wie gesagt ich werde noch wissen was ich 2014 geschrieben habe.
[code=php]
public function get_results( $query, $object = false )
{
self::$counter++;
//Overwrite the $row var to null
$row = null;
$results = $this->link->query( $query );
if( $this->link->error )
{
$this->log_db_errors( $this->link->error, $query );
return false;
}
else
{
$row = new stdClass();
$i = 1;
while( $r = ( !$object ) ? $results->fetch_object() : $results->fetch_array() )
{
$row->{$i} = $r;
$i++;
}
return $row;
}
}
function check_email($email) {
if(!filter_var($email_a, FILTER_VALIDATE_EMAIL)) {
return false;
}
$email_array = explode("@", $email);
$local_array = explode(".", $email_array[0]);
for ($i = 0; $i < sizeof($local_array); $i++) {
if(!ereg("^(([A-Za-z0-9!#$%&'*+/=?^_`{|}~-][A-Za-z0-9!#$%&'*+/=?^_`{|}~\.-]{0,63})|(\"[^(\\|\")]{0,62}\"))$", $local_array[$i])) {
return false;
}
}
if(!ereg("^\[?[0-9\.]+\]?$", $email_array[1])) {
$domain_array = explode(".", $email_array[1]);
if(sizeof($domain_array) < 2) {
return false;
}
for($i = 0; $i < sizeof($domain_array); $i++) {
if(!ereg("^(([A-Za-z0-9][A-Za-z0-9-]{0,61}[A-Za-z0-9])|([A-Za-z0-9]+))$", $domain_array[$i])) {
return false;
}
}
}
return true;
}
[/code]
Desweiteren ist der Code überhaupt nicht dein.
Berechtigungen
