Update klappt nicht

Peter2014 · 2. September 2014

Hallo,

ich verstehe Ajax nicht wirklich. Mein Script (habe ich nicht selber geschrieben) sieht so aus

JavaScript

$(document).ready(function(){

		      function slideout() {
        setTimeout(function() {
          $("#debugMess").slideUp("slow", function () {
      });
    }, 2000);}
 
    $("#debugMess").hide();
 
    $(function() {
        $("#customersList").sortable({ placeholder: "customersListHighlight", opacity: 0.5, cursor: 'move', update: function() {
          var order = $(this).sortable("serialize") + '&action=updateCustomerPos';
          $.post("UpdatePos.php", order, function(theResponse) {
            $("#debugMess").html(theResponse);
            $("#debugMess").slideDown('slow');
            slideout();
          });                                
        }                 
      });
      $( "#customersList" ).disableSelection();
    });
 });

Alles anzeigen

Meine Daten lese ich so aus

PHP

$showDebugMessage = true;







				$stmt = $mysqli->prepare("SELECT id, name FROM Test ORDER BY pos ASC");
				$stmt->execute();
				$stmt->bind_result($id, $name);            
				$stmt->store_result();	

		  echo "<div id="wrapper">\n";
  if($showDebugMessage) echo "  <div id="debugMess"></div>\n";

		  echo "<div id="test">\n";
 		  echo "<ul id="customersList">\n";

		while($stmt->fetch()) {

			echo "    <li id="recArray_".$id."">\n";
			echo "      <div class="customerName">".$name."</div>\n";
			echo "    </li>\n";
  }
  echo "  </ul>\n";
  echo "</div>";

Alles anzeigen

die Update Datei hat folgenden Code

PHP

require("inc.zugangsdaten.php");




$action = $_POST['action'];  
$updRecArray = $_POST['recArray'];
 
if($action == "updateCustomerPos")
{ 
  $counter = 1;

  foreach ($updRecArray as $recordIDValue) 
  {
	  if ($stmt = $mysqli->prepare("Update 
										Test SET 
										pos=?

									WHERE id = ?"))
	{   
			$pos		= $counter;
			$id			= $_recordIDValue;

			$stmt->bind_param("ss",	
										$pos,
										$id
								  );

			$stmt->execute();
			}
			else {
				echo $mysqli -> error;
			}

			$counter++; 

  }
  echo 'Position gespeichert';
  echo '<pre>';
print_r($updRecArray);
echo '</pre>';
}

Alles anzeigen

Wenn ich nun ein Eintrag verschiebe klappt es auch, mein Firebug gibt mir folgende Meldung aus

Code

Position gespeichertArray
(
    [0] => 1
    [1] => 2
    [2] => 3
    [3] => 4
)

das heißt für mich, dass das Update erfolgreich war. Allerdings lade ich die Seite neu oder schau in die Datenbank hat sich leider nichts getan. Fehlermeldungen erscheinen auch nicht. Das ganze Script habe ich von hier http://fluuux.de/2013/03/posit…per-drag-and-drop-andern/ seht ihr vielleicht den Fehler?

wolf · 2. September 2014

Ja ich seh viele Fehler, vorallem in der Einrückung, die das lesen und verstehen extrem behindern.
TEIL1

JavaScript

$(document).ready(function(){
    function slideout() {
        setTimeout(
            function() {
                $("#debugMess").slideUp("slow", function(){});
            }, 2000
        );
    }




    $("#debugMess").hide();




    $(function() {
        $("#customersList").sortable({
            placeholder : "customersListHighlight",
            opacity     : 0.5,
            cursor      : 'move',
            update      : function() {
                    var order = $(this).sortable("serialize") + '&action=updateCustomerPos';
                    $.post("UpdatePos.php", order, function(theResponse) {
                        $("#debugMess").html(theResponse);
                        $("#debugMess").slideDown('slow');
                        slideout();
                    });                                
                }                 
        });
        $( "#customersList" ).disableSelection();
    });
});

Alles anzeigen

das script sicher!
Das hier STOPPT

PHP

echo "<div id="wrapper">\n";

Fehler siehst du selber nicht oder? <- ich nehme das an weil du den fehler immer gemacht hast.
Wenn du innerhalb eines Strings Anführungszeichen(einfach oder doppelt) verwenden willst
a) darf der string nicht mit diesem angefangen/beendet werden
b) dieses durch ein \(backslash) escaped werden.!

PHP

//falsch
echo "<div id="wrapper">\n";
//ok !ACHTUNG! in »'«(einfachen Anführungszeichen) stehende variablen, funktionen u.ä werden nicht ausgeführt (hier »\n«)
echo '<div id="wrapper">\n';
    //lösung - einfache und doppelte verknüpfen
    echo '<div id="wrapper">'."\n";
//ebenfalls ok
echo "<div id="wrapper">\n";




//andersrum das geliche
//falsch
echo '<div id='wrapper'>\n';
//richtig
echo "<div id='wrapper'>\n";
//ebenfalls ok Same Problem wie oben mit den einafchen anf. Zeichen
echo '<div id=\'wrapper\'>\n';

Alles anzeigen

PHP

//wie ich es bevozuge:
echo '<div id="'.$someVar.'">';

Wenn du auf eine html ähnliche einrückung achtest, kann dies die lesbarkeit erhöhen aber auch veringern..

PHP

$showDebugMessage = true;
//DB abfrage
$stmt = $mysqli->prepare("SELECT `id`, `name` FROM `Test` ORDER BY `pos` ASC");
$stmt->execute();
$stmt->bind_result($id, $name);            
$stmt->store_result();




echo '<div id="wrapper">'."\n";
if($showDebugMessage){
    echo '<div id="debugMess"></div>'."\n";
} //Bitte IMMER IMMER klammern nutzen!!!!
    echo '<div id="test">'."\n";
        echo '<ul id="customersList">'."\n";
        while($stmt->fetch()) {
            echo '<li id="recArray_'.$id.'">'."\n";//das hier war die böse Zeile ^^
                echo '<div class="customerName">"'.$name.'"</div>'."\n";
            echo "</li>\n";
        }
        echo "</ul>\n";
    echo "</div>";
//hier wird ein div nicht geschlossen?!

Alles anzeigen

PHP

require("inc.zugangsdaten.php");
//immer trimmen
$action = trim($_POST['action']);  
$updRecArray = trim($_POST['recArray']);




if($action == "updateCustomerPos"){ 
    $counter = 1;
    foreach ($updRecArray as $recordIDValue){
        if ($stmt = $mysqli->prepare("
            Update Test 
            SET pos=?
            WHERE id = ?")
        ){#if msqli->prepare
            $pos = $counter;
            $id = $_recordIDValue;
            $stmt->bind_param("ss", $pos, $id );
            $stmt->execute();
        }#end if
        else {
            echo $mysqli -> error;
        }
        $counter++; 
    }#end foreach
    echo 'Position gespeichert';
    //debug
    echo '<pre>';
    print_r($updRecArray);
    echo '</pre>';
}#end if action==updCustPost

Alles anzeigen

Wenn du bisher keine Fehlermeldung bekommen hast -> am anfang des scriptes

PHP

error_reporting(E_ALL);

Schreiben. -> Wenn dann noch immer nix kommt

PHP

ini_set('display_errors',1);

VErstehst du denn den Code wenigstens komplett?
Und von AJAX sehe ich keine Spur :0

Peter2014 · 2. September 2014

Danke für deine Hilfe. Nun sehe ich auch eine Fehlermeldung, diese sieht so aus

Code

Notice: Undefined index: recArray in UpdatePos.php on line 6
Warning: Invalid argument supplied for foreach() in UpdatePos.php on line 10

In der Zeile 6 steht dieses

PHP

$updRecArray = trim($_POST['recArray']);

In der Zeile 10 steht dieses

PHP

foreach ($updRecArray as $recordIDValue){

wolf · 2. September 2014

Z6 Besagt, dass kein post wert mit dem namen recArray übergeben wurde..
Z10 Besagt das in foreach kein array reingegeben wurde, was warsch. auf den Fehler aus Z6 zurückzuführen ist

Peter2014 · 2. September 2014

Es liegt wohl an dieser Zeile, dein Code macht dieses

Und das ist eigentlich falsch. Richtig müsste es doch lauten? <li id="recArray_2">? Oder liege ich jetzt falsch?

EDIT:

Die Zeile muss so lauten

PHP

echo '<li id="recArray_'.$id.'">'."\n";

Jetzt bekomme ich noch ein Fehler

Code

Notice: Undefined variable: _recordIDValue

OK nun läuft das Script, danke für deine Hilfe. Der letzte Fehler lag hier

PHP

$id = $_recordIDValue;

hier war ein _ zu viel drin

wolf · 3. September 2014

Oops, ja das hatte ich übersehen.

Ich hoffe du hast was draus gelernt

Update klappt nicht

Jetzt mitmachen!

Teilen