TomH27

Hallo zusammen.
Ich hab da ein kniffliges Problem und quäl mich nun schon zwei Tage.

Folgender Sachverhalt.
Formulardaten werden per PHP an eine mysql-Datenbank übergeben und ein angehängtes Bild wird upgeloadet..
Die alte mysql Version war 3.x. Nun sind wir gezwungenermassen auf 4.0.x umgestiegen.

Die Formulardaten werden immer noch übertragen aber das Bild wird nicht mehr in das Verzeichnis geladen. In der Datenbank wird hinterlegt dass ein Bild mit der Bezeichnung####.jpg vorhanden ist aber im vorgesehenen Verzeichnis befindet es sich nicht. Keine Fehlermeldung, nichts.
In der datenbank konnte ich keinen fehler finden. Zugriffsrechte passen.

Hat jemand eine Idee?

Hier die datei mit der das Bild upgeloadet wird:

#########################
<?
/*
functions:

constructor upload($name) - name als string nicht variable
function set_rights($width,$height,$maxsize=0,$mustfit=fals e)
function make_upload($path)
function script_alert()

--------------------------------------------------------------------*/

// class upload
class upload
{
var $name;
var $source;
var $source_size;
var $filename;
var $fileext;
var $width;
var $height;
var $size;
var $alert = "";
var $mustfit = false;

// constructor upload
# PARAMS: $name - name als string nicht variable

function upload() //$name
{
$this->name = ereg_replace( "[\r\t\n\v ]", "", $_FILES["upl_document"]["name"] ); //trimming bullshit
$this->source = $_FILES["upl_document"]["tmp_name"];
$this->source_size = $_FILES["upl_document"]["size"];
$this->source_sizekb = round( $_FILES["upl_document"]["size"]/1024 );
$this->fileext = substr( $this->name, strrpos( $this->name, "." ), strlen( $this->name ) );

// Workaround for Movie-Files .mov
if ( $this->fileext == ".mo" ) $this->fileext = ".mov";

// Generate new Hash, used as Filename (Old Filename is saved in special Column)
$this->hash = substr( uniqid( rand() ), 0, 10 );

// Filename including real Extension and with running Zere (Original Uploaded File)
$this->filenameWOzero = $this->hash.$this->fileext;
$this->filename = "0".$this->filenameWOzero;

list($width,$height) = GetImageSize($this->source);
$this->width2 = $width;
$this->height2 = $height;
}

// function set_rights
# PARAMS: $width,$height,$maxsize=0,$mustfit=false

function set_rights($width,$height,$maxsize=0,$mustfit=fals e)
{
$this->width = $width;
$this->height = $height;
$this->size = $maxsize;
$this->mustfit = $mustfit;
}

// function make_upload
# PARAMS: $path

function make_upload($path) {
// Check der Dateigröße
if (($this->source_size>$this->size) && $this->size>0) {
$kb = number_format(strval($this->size/1024));
$this->alert = "Das Dateivolumen überschreitet ".$kb." KB. Bitte verkleinern Sie Ihr Foto.";
}
// Check der Bildgröße
if ( ($this->width>0 && $this->height>0) && $this->source && in_array($this->fileext,array(".gif",".png",".jpg",".jpeg")) ) {
list($width,$height) = GetImageSize($this->source);
$kb = number_format(strval($this->size/1024));
if ($width>$this->width || $height>$this->height) {
$this->alert = "Die Dateigrösse überschreitet ".$this->width." mal ".$this->height." Pixel. Bitte verkleineren Sie Ihr Foto. ";
}
}
// Check Datei selbst
if (!$this->source_size) {$this->alert = "";}

// Wenn alles OK
if (!$this->alert && $this->source_size) {
if (copy($this->source,$path."/".$this->filename)) {return $this->filename;}
} else {return false;}
}

// function script_alert
function script_alert() {
if ($this->alert) {echo "<script language=\"Javascript\">alert('".$this->alert."')</script>";}
}

}
?>
##########################

Mister Ad

TomH27

Hat keiner eine Idee?

Doch:

Richtig debuggen
1. Man bemerkt, dass ein Skript nicht das tut, was es soll.
2. Man versucht, die Stelle die daran Schuld sein kann, schonmal einzugrenzen. Falls dies nicht geht, wird zunächst das komplette Skript als fehlerhaft angesehen.
3. An markanten Stellen im Skript lässt man sich wichtige Variableninhalte ausgeben und ggf. auch in bedingten Anweisungen eine kurze Ausgabe machen, um zu überprüfen, welche Bedingung ausgeführt wurde (oder auch nicht).
4. Schritt 3 wird so lange wiederholt, bis Unstimmigkeiten im Skript auffallen
5. Damit hat man das Problem (Unstimmigkeit) gefunden und kann versuchen diese zu beheben. Hierzu dienen dann die PHP-Dokumentation und andere Quellen als Ratgeber.
6. Lässt sich das konkrete Problem trotzdem nicht beheben, kann man in Foren um Rat fragen.
7. Das Programm läuft und man kann die Debug-Ausgaben wieder entfernen.

Die Ausgaben per echo kann man sich ersparen, in dem man einen Debugger benutzt, was eigentlich selbstverständlich sein sollte.


Ein Hinweis: So einen unformatiert geposteten Quellcode sieht sich kaum jemand an. Und wie du die Klasse benutzt, verheimlichst du uns.

Gruß
phpfan

TomH27

Wie sollte ich das Script denn formatieren?

So benutze ich die Klasse

##########
$upload = new upload();
$upload -> set_rights( 10000, 10000, 40960000 );
$upload -> make_upload( 'pix/' );

// If Upload successfull prepare Save-Array
if ( $upload->make_upload( 'pix/' ) )
{
// Conditional Variables
$pic = $upload->hash.".jpg";

/* Defining Source */
$source = 'pix/member4'.$upload->filename;
/* Defining Source of BMP */
$source2 = 'pix/member4'.$upload->hash.'.bmp';
/* Workaround Upld->BMP */
exec( '/usr/bin/convert -flatten +profile "*" '.$source.' '.$source2 );
/* CMS Thumb */
exec( '/usr/bin/convert -geometry 85 +profile "*" -contrast -quality 85 '.$source2.' pix/thumb/'.$pic );
/* New Pic */
exec( '/usr/bin/convert -geometry 200 +profile "*" -contrast -quality 85 '.$source2.' pix/'.$pic );
// Unlink interim BMP
@unlink( $source2 );
// Unlink Original
@unlink( $source );
}
}

Corvin

Code einrücken:
http://tut.php-q.net/einruecken.html

Und gewöhn dir bitte an, die [php]-Tags zu benutzen.

Der_Gerhard

Du wiederholst Dich
SCNR

__________________
**********************************
Nein, ich bin nicht die Signatur.
Ich putze hier nur.
**********************************

Corvin

Klick mal auf "Zitat", dann siehst du, dass das nicht an mir liegt, sondern ein Bug der Forensoftware sein muss.

Der_Gerhard

Ja klar. ^^
Das würde ich jetzt auch behaupten.

__________________
**********************************
Nein, ich bin nicht die Signatur.
Ich putze hier nur.
**********************************

Corvin

Hast du vielleicht zu viel getrunken?
Na.. egal, das gehört hier nicht hin. :X

Der_Gerhard

Also mein Zitat hat eben noch einwandfrei funktioniert. Darum hab ich das auch gleich mal in dem Posting demonstriert.
Mit Edit hatte ich heute auch noch keine Probleme hier im Forum.

Und die beiden Postings hab ich ganz ohne Konsum irgendwelche bewusstseinsverändernder Mittel geschrieben. Auch ohne Vollsuff kann ich darauf verzichten, toternst durch's Leben zu laufen.

Es war ja nicht sofort ersichtlich, dass ausgerechnet dieses eine Posting da die Forensoftware sprengt.
Denn das Posting ist in der Tat etwas seltsam.

__________________
**********************************
Nein, ich bin nicht die Signatur.
Ich putze hier nur.
**********************************