Problem mit Datei upload

lima-city → Forum → Programmiersprachen → PHP, MySQL & .htaccess

Autor dieses Themas
kill-a-teddy

kill-a-teddy hat kostenlosen Webspace.
8:04, 10.1.2013
Hallo,

Ich habe ein Problem beim Dateien Upload:

Warning: move_uploaded_file() [function.move-uploaded-file]: Filename cannot be empty in C:\xampp\htdocs\entwicklung\Test\public\add.php on line 16

Warning: move_uploaded_file() [function.move-uploaded-file]: Unable to move 'C:\xampp\tmp\phpE48C.tmp' to '' in C:\xampp\htdocs\entwicklung\Test\public\add.php on line 16

Trotz Abfrage habe ich immer einen Datei Name empty, selbst wenn ich eine Datei auswähle:
```
if(isset($_FILES['datei']) && !empty($_FILES['datei']['name'])){
	$timestamp = time();
	$pfad = "uploads";
	$pfad  = $pfad . DIRECTORY_SEPARATOR . $timestamp . "_" . $_FILES['datei']['name'];

	if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad === true)) {
	}
	else{
    echo "ERROR";
	}
	}
```
Das Upload Feld heißt auch Datei:
```
<input type="file" name="datei" id="datei">
```
Kann mir jemand erklären, was ich falsch mache?

//EDIT:
Das Problem tritt übrigens nur beim Bearbeiten auf. Beim erstellen klappt alles wunderbar, der Code ist jedoch ein und derselbe...

Beitrag zuletzt geändert: 10.1.2013 8:09:57 von kill-a-teddy
Diskutiere mit und stelle Fragen: Jetzt kostenlos anmelden!

lima-city: Gratis werbefreier Webspace für deine eigene Homepage
~~h**s~~
11:09, 10.1.2013
steht das "=== true" nicht vielleicht in der falschen klammer ?
vielleicht so:
```
if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {
}
```
Autor dieses Themas
kill-a-teddy

kill-a-teddy hat kostenlosen Webspace.
11:20, 10.1.2013
hcms schrieb:
steht das "=== true" nicht vielleicht in der falschen klammer ?
vielleicht so:
```
if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {
}
```
Das True gehört zum if, nicht zum move_uploaded_file ^^
~~h**s~~

11:57, 10.1.2013

Das True gehört zum if, nicht zum move_uploaded_file ^^

soll das jetzt heissen "ja, klappt" oder "ne, so nicht" ?

damit das true als bestandteild der if-bedingung behandelt wird muss es ja wie ich oben schrieb gemacht werden - sonst übergibts du das true ja als bestandteil des 2ten parameters von move_uploaded_file
Autor dieses Themas
kill-a-teddy

kill-a-teddy hat kostenlosen Webspace.

11:59, 10.1.2013

hcms schrieb:

Das True gehört zum if, nicht zum move_uploaded_file ^^

soll das jetzt heissen "ja, klappt" oder "ne, so nicht" ?

damit das true als bestandteild der if-bedingung behandelt wird muss es ja wie ich oben schrieb gemacht werden - sonst übergibts du das true ja als bestandteil des 2ten parameters von move_uploaded_file

Ja, es gehört da hinter, aber das löst mein Problem nicht, was mich sehr verwundert...
czibere

czibere hat kostenlosen Webspace.

16:48, 10.1.2013
yo mey! bevor hier weiter nur müll gedroschen wird: kannst du bitte mitteilen, was vor und nach der zeile 16 im quellcode drin ist!!?? ja was! am besten poste hier den ganzen code von der datei. so kann man nicht helfen, wenn man im finsteren tappt!

Autor dieses Themas

7:15, 11.1.2013

czibere schrieb:
yo mey! bevor hier weiter nur müll gedroschen wird: kannst du bitte mitteilen, was vor und nach der zeile 16 im quellcode drin ist!!?? ja was! am besten poste hier den ganzen code von der datei. so kann man nicht helfen, wenn man im finsteren tappt!

Zeile 16 ist das if

if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {
}

Ganzer Code is a bisschen groß:

<?php
session_start();
require_once '../init.php';
header('Content-Type: text/html;charset=utf-8');

$artikel = new artikel();
if(isset($_GET['hash'])){
	$hash = $_GET['hash'];
}

if(isset($_REQUEST['Bearbeiten'])){
	$timestamp = time();
	$datum = date("Y-m-d",$timestamp);
	$uhrzeit = date("H:i:s",$timestamp);
	$time = $datum." ".$uhrzeit;
	if(isset($_FILES['datei']) && !empty($_FILES['datei']['name'])){
	$pfad = "uploads";
	$pfad  = $pfad . DIRECTORY_SEPARATOR . time() . "_" . $_FILES['datei']['name'];
	if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {}
	else{
    echo "ERROR";
	}
	}
	$artikel->Artikel_Name						= $_REQUEST['name'];
	$artikel->Artikel_Beschreibung				= $_REQUEST['beschreibung'];
	$artikel->Artikel_Geaendert_von				= $_SESSION['id'];
	$artikel->Artikel_Aenderungsdatum			= $time;
	if(isset($pfad)){$artikel->Artikel_Bild 	= $pfad;}
	$artikel->ArtikelÜberarbeiten($_REQUEST['hash']);
	header("Location: index.php");
	exit;
	}

if(isset($_REQUEST['Abschicken'])){
	$timestamp = time();
	$datum = date("Y-m-d",$timestamp);
	$uhrzeit = date("H:i:s",$timestamp);
	$time = $datum." ".$uhrzeit;
	if(isset($_FILES['datei']) && !empty($_FILES['datei']['name'])){
	$pfad = "uploads";
	$pfad  = $pfad . DIRECTORY_SEPARATOR . time() . "_" . $_FILES['datei']['name'];
	if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {}
	else{
    echo "ERROR";
	}
	}
	$artikel->Artikel_Name						= $_REQUEST['name'];
	$artikel->Artikel_Beschreibung				= $_REQUEST['beschreibung'];
	$artikel->Artikel_Erstellt_von				= $_SESSION['id'];
	$artikel->Artikel_Geaendert_von				= $_SESSION['id'];
	$artikel->Artikel_Aenderungsdatum			= $time;
	$artikel->Artikel_Angebotsdatum				= $time;
	if(isset($pfad)){$artikel->Artikel_Bild 	= $pfad;}
	$artikel->ArtikelSpeichern();
	header("Location: index.php");
	exit;
	}

if(isset($hash)){
	$data = $artikel->getArtikelbyHash($hash);
	foreach($data as $value){
	$original_time = $value->Artikel_Angebotsdatum;
	}
}

?>
<!DOCTYPE HTML>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html;charset=UTF-8">
<title>Neuen Artikel Einfügen</title>
<link rel="stylesheet" type="text/css" media="all" href="css/screen.css" />

<!-- Template stylesheet -->
<script type='text/javascript' src='js/jquery-1.7.1.js'></script>
<script type='text/javascript' src='js/jquery.ui.js'></script>
<script type='text/javascript' src='js/jquery.fancybox-1.3.0.js'></script>
<script type='text/javascript' src='js/jquery.validate.js'></script>
<script type='text/javascript' src='js/cufon.js'></script>
<script type='text/javascript' src='js/Aaargh_400.font.js'></script>
<script type='text/javascript' src='js/browser.js'></script>
<script type='text/javascript' src='js/video.js'></script>
<script type='text/javascript' src='js/jquery.backstretch.js'></script>
<script type='text/javascript' src='js/custom.js'></script>
<!--[if IE 7]>
<link rel="stylesheet" href="css/ie7.css" type="text/css" media="all"/>
<![endif]-->
<style type="text/css">

.post_header h2, h1, h2, h3, h4, h5
{
	color: #ffc600;
}

body, #page_content_wrapper .sidebar .content .sidebar_widget li a
{
	color: #ffffff;
}

.nav, .subnav { font-size:32px; }

.nav li.current-menu-item > a, .nav li > a:hover, .nav li > a:active { color:#ffc800; }

h1 { font-size:40px; }

h2 { font-size:32px; }

h3 { font-size:26px; }

h4 { font-size:24px; }

h5 { font-size:22px; }

h6 { font-size:18px; }

a { color:#ffffff; }

a:hover, a:active { color:#ffffff; }

input[type=submit], input[type=button], a.button { 
	background: #000000;
	background: -webkit-gradient(linear, left top, left bottom, from(#333333), to(#000000));
	background: -moz-linear-gradient(top,  #333333,  #000000);
	filter:  progid:DXImageTransform.Microsoft.gradient(startColorstr='#333333', endColorstr='#000000');
}
input[type=submit]:active, input[type=button]:active, a.button:active
{
	background: #000000;
	background: -webkit-gradient(linear, left top, left bottom, from(#000000), to(#333333));
	background: -moz-linear-gradient(top,  #000000,  #333333);
	filter:  progid:DXImageTransform.Microsoft.gradient(startColorstr='#333333', endColorstr='#000000');
}

input[type=submit], input[type=button], a.button { 
	color: #ffffff;
}
input[type=submit]:hover, input[type=button]:hover, a.button:hover
{
	color: #ffffff;
}

input[type=submit], input[type=button], a.button { 
	border: 1px solid #000000;
}

.post_header h2, h1, h2, h3, h4, h5
{
	color: #ffc600;
}

</style>

</head>

<body class="single single-post postid-1055 single-format-standard">
<input type="hidden" id="pp_menu_display" name="pp_menu_display" value=""/>
<div id="menu_mini_state">
<div id="mini_logo">
<a href=""><img src="#" alt="" />LOGO</a>
</div>
</div>
<!-- Begin template wrapper -->
<div id="wrapper">
<div id="menu_mini_state_btn"></div>
<div id="menu_wrapper">
<!-- Begin logo -->						
<a id="custom_logo" class="logo_wrapper" href="#"><img src="#" alt=""/>LOGO</a>			
<!-- End logo -->
<!-- Begin main nav -->
<div class="menu-mena%c2%bc-container">
<ul id="main_menu" class="nav">
<li class="menu-item menu-item-type-post_type"><a href="index.php">Startseite</a></li>
<?php if(isset($_SESSION['hash'])){?><li class="menu-item menu-item-type-post_type"><a href="add.php">Neuer Artikel</a></li><?php }?>
<?php if(isset($_SESSION['hash'])){?><li class="menu-item menu-item-type-post_type"><a href="profile.php?hash=<?php echo $_SESSION['hash'];?>">Profil</a></li><?php }?>
<li class="menu-item menu-item-type-post_type"><a href="search.php">Durchsuchen</a></li>
</ul></div>		    
		    <!-- End main nav -->
		    
		    <br class="clear"/>
		    
		    <div id="copyright">
				Copyright &copy; 2012 Martin McCormack</div>
		</div>
		<!-- Begin content -->
		<div id="page_content_wrapper">
		
			<div class="inner">
			
				<div class="sidebar_content">
				
				
				<script type="text/javascript"> 
			jQuery.backstretch( "images/background.jpg", {speed: 'slow'} );
			</script>
			
<!-- Begin each blog post -->
<div id="<?php echo $value->Artikel_Hash;?>" class="post_wrapper">
<?php if(isset($_SESSION['hash']) && isset($_SESSION['rights']) && $_SESSION['rights'] == "2"){?>
<form enctype="multipart/form-data" action="add.php" method="post">
	<label>Name:</label><br>
	<input type="text" id="name" name="name" min="11" max="15" placeholder="Ich bin ein Titel" <?php if(isset($value->Artikel_Name)){echo 'value="'.htmlspecialchars($value->Artikel_Name).'"';}?> required>
	<br><br>
	<label>Beschreibung:</label><br>
	<textarea id="beschreibung" name="beschreibung"><?php if(isset($value->Artikel_Beschreibung)){echo htmlspecialchars($value->Artikel_Beschreibung);}?></textarea>
	<br><br>
	<label for="datei">Bild anhängen</label><br>
	<input type="file" name="datei" id="datei">
	<input type="hidden" name="hash" id="hash" <?php if(isset($value->Artikel_Hash)){echo 'value="'.htmlspecialchars($value->Artikel_Hash).'"';}?>></input>
	<br><br>
	<?php 
	if(isset($hash)){ 
		echo '<input type="submit" name="Bearbeiten" value="Bearbeiten"/>';
	}
	else{
		echo '<input type="submit" name="Abschicken" value="Abschicken"/>';
	}?>
	</form>
	<?php }
	else{ echo "Sorry, du darfst diese Seite leider nicht betrachten.";}
	?>
</div>
<!-- End each blog post -->
<hr></hr>
</div>
<div class="sidebar_wrapper">
<div class="sidebar">
<div class="content">
<ul class="sidebar_widget">
<li id="customizable-search" class="widget widget_customizable_search">
<h2 class="widgettitle">SUCHEN</h2>
<form id="searchform" action="search.php">
			<div>
				<input type="text" value="" name="s" size="20" id="s" /><input type="submit" value="Suchen"  />
			</div>
</form>
</li>
</ul>		
</div>				
</div>
</div>
<br class="clear"/>
</div>
<br class="clear"/>
</div>
<!-- End content -->
<br class="clear"/>
</div>
<br class="clear"/>	
</body>
</html>

Beitrag zuletzt geändert: 11.1.2013 7:26:30 von kill-a-teddy

czibere

czibere hat kostenlosen Webspace.

0:44, 12.1.2013

kill-a-teddy schrieb:
... Ganzer Code is a bisschen groß:
<?php
session_start();
require_once '../init.php';
header('Content-Type: text/html;charset=utf-8');

$artikel = new artikel();
...

mann mann! wo hast du den code her???? es ist ein desaster :(
nachdem ich ihn gebürend auseinander dividiert habe, sind gleich mehrer probleme auf einmal sichtbar geworden!

aber zuerst ganz leise hier mal meine vermutung:

$artikel->ArtikelÜberarbeiten($_REQUEST['hash']);

das kann wohl nicht gehen (functionsname mit umlaut). ich würde so umbenennen

$artikel->ArtikelBearbeiten($_REQUEST['hash']);

was du natürlich auch in

require_once '../init.php';

tun solltest (was ich zwar nur vermuten kann, die datei kenne ich ja nicht).

sollte die änderung das problem nicht lösen, gebe das

if(count($_FILES)) {
  echo '<h4>_FILES</h4><pre>'.print_r($_FILES,true).'</pre>';
}

nach session_start(); ein und guck dir die ausgabe an.
ist das alles vergebens, werde ich noch ein zwei dinge von dir brauchen, so dass ich echt testen kann. (wenn du willst ;)

ganz nebenbei!

//EDIT:
Das Problem tritt übrigens nur beim Bearbeiten auf. Beim erstellen klappt alles wunderbar, der Code ist jedoch ein und derselbe...

nööö!?
1. Abschicken:

if(isset($_REQUEST['Abschicken'])) {

  $timestamp = time();
  $datum     = date("Y-m-d",$timestamp);
  $uhrzeit   = date("H:i:s",$timestamp);
  $time      = $datum." ".$uhrzeit;

  if(isset($_FILES['datei']) && !empty($_FILES['datei']['name'])) {

    $pfad = "uploads";
    $pfad = $pfad . DIRECTORY_SEPARATOR . time() . "_" . $_FILES['datei']['name'];

    if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {
      // willst du's niemandem mitteilen, dass es gut gegangen ist????;
    } else {
      echo "ERROR"; // das ist nicht sichtbar!!!! [siehe header(... unten]
    }
  }

  $artikel->Artikel_Name            = $_REQUEST['name'];
  $artikel->Artikel_Beschreibung    = $_REQUEST['beschreibung'];
  $artikel->Artikel_Geaendert_von   = $_SESSION['id'];
  $artikel->Artikel_Aenderungsdatum = $time;
  $artikel->Artikel_Erstellt_von    = $_SESSION['id'];  // nur wenn 'abschicken'
  $artikel->Artikel_Angebotsdatum   = $time;            // nur wenn 'abschicken'

  if(isset($pfad)) {
    $artikel->Artikel_Bild = $pfad;
  }

  $artikel->ArtikelSpeichern();
  header("Location: index.php");  // richtig ist: "Location: http://<host>/<path2>/<file>.php"
  exit();
}

2. Bearbeiten:

if(isset($_REQUEST['Bearbeiten'])) {

  $timestamp = time();
  $datum     = date("Y-m-d",$timestamp);
  $uhrzeit   = date("H:i:s",$timestamp);
  $time      = $datum." ".$uhrzeit;

  if(isset($_FILES['datei']) && !empty($_FILES['datei']['name'])) {

    $pfad = "uploads";
    $pfad = $pfad . DIRECTORY_SEPARATOR . time() . "_" . $_FILES['datei']['name'];

    if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {
      // willst du's niemandem mitteilen, dass es gut gegangen ist????;
    } else {
      echo "ERROR"; // das ist nicht sichtbar!!!! [siehe header(... unten]
    }
  }

  $artikel->Artikel_Name            = $_REQUEST['name'];
  $artikel->Artikel_Beschreibung    = $_REQUEST['beschreibung'];
  $artikel->Artikel_Geaendert_von   = $_SESSION['id'];
  $artikel->Artikel_Aenderungsdatum = $time;



  if(isset($pfad)) {
    $artikel->Artikel_Bild = $pfad;
  }

  $artikel->ArtikelÜberarbeiten($_REQUEST['hash']); // kommst in diese funktion überhaupt rein???? [umlaut im namen!!??]
  header("Location: index.php");  // richtig ist: "Location: http://<host>/<path2>/<file>.php"
  exit();
}

ich würde allerding die 2 if's ganz umgestalten (statt die zeilen 11 - 57 in deinem code):

if(!empty($_REQUEST['Abschicken'])) {
  $art_function  = 'ArtikelSpeichern';
  $art_func_hash = null;
}
if(!empty($_REQUEST['Bearbeiten'])) {
  $art_function  = 'ArtikelBearbeiten';  // statt 'ArtikelÜberarbeiten'!!!!
  $art_func_hash = $_REQUEST['hash'];
}
if(!empty($_REQUEST['Abschicken']) OR !empty($_REQUEST['Bearbeiten']) {

  $timestamp = time();
  $datum     = date("Y-m-d",$timestamp);
  $uhrzeit   = date("H:i:s",$timestamp);
  $time      = $datum." ".$uhrzeit;

  if(isset($_FILES['datei']) && !empty($_FILES['datei']['name'])) {

    $pfad = "uploads";
    $pfad = $pfad . DIRECTORY_SEPARATOR . time() . "_" . $_FILES['datei']['name'];

    if(move_uploaded_file($_FILES['datei']['tmp_name'], $pfad) === true) {
      // willst du's niemandem mitteilen, dass es gut gegangen ist????;
    } else {
      echo "ERROR"; // das ist nicht sichtbar!!!! [siehe header(... unten]
    }
  }

  $artikel->Artikel_Name            = $_REQUEST['name'];
  $artikel->Artikel_Beschreibung    = $_REQUEST['beschreibung'];
  $artikel->Artikel_Geaendert_von   = $_SESSION['id'];
  $artikel->Artikel_Aenderungsdatum = $time;
  if($art_function === 'ArtikelSpeichern') {
    $artikel->Artikel_Erstellt_von  = $_SESSION['id'];
    $artikel->Artikel_Angebotsdatum = $time;
  }

  if(isset($pfad)) {
    $artikel->Artikel_Bild = $pfad;
  }

  $artikel->$art_function($art_func_hash);
  header("Location: index.php");  // richtig ist: "Location: http://<host>/<path2>/<file>.php"
  exit();
}

probier's aus. solllte trotzt fehlender testmöglichkeit laufen.

Diskutiere mit und stelle Fragen: Jetzt kostenlos anmelden!

lima-city: Gratis werbefreier Webspace für deine eigene Homepage

Dir gefällt dieses Thema?

Über lima-city

lima-city bietet dir kostenlosen und werbefreien Speicherplatz für Deine Homepage. Sofort anmelden und direkt loslegen mit Webspace, PHP, Datenbanken, günstigen Domains und einer tollen Community!

kostenloser Webspace werbefrei: lima-city