Thema: BenutzerID in Datenbank speichern

Hallo zusammen.
Ich habe eine Komponente programmiert, mit der jeder User Charaktere für ein Rollenspiel hinzufügen kann. Nun möchte ich erreichen, dass die jeweilige BenutzerID ebenfalls in die Datenbank geschrieben wird, so, dass ich später die Charaktere im Zusammenhang des Users ausgeben kann. Das habe ich folgendermassen versucht:

Controller

PHP-Code:

<?php 
defined( '_JEXEC' ) or die( 'Restricted access' ); 
jimport('joomla.application.component.controller'); 
class GwcharsController extends JController { 
   function display() { 
      parent::display(); 
   }
   function __construct() { 
      parent::__construct(); 
      $this->registerTask( 'add', 'edit' ); 
   }
   function edit() { 
      JRequest::setVar( 'view', 'addgwchar' ); 
      JRequest::setVar( 'layout', 'form'  ); 
      JRequest::setVar( 'hidemainmenu', 1 ); 
      parent::display(); 
   }
   function save() { 
      $model = $this->getModel('addgwchar'); 
      if ($model->store()) { 
        $msg = JText::_( 'Character Saved' ); 
      } else { 
        $msg = JText::_( 'Error Saving' ) . ': ' . $model->getError();
      } 
      $this->setRedirect( 'index.php?option=com_gwchars&view=addgwchars', 
                          $msg); 
                          return false; 
   } 
   function remove() { 
      $model = $this->getModel('addgwchar'); 
      if($model->delete()) { 
         $msg = JText::_( 'Character Deleted' ); 
      } else { 
         $msg = JText::_( 'Error Deleting' ); 
      } 

      $this->setRedirect( 'index.php?option=com_gwchars&view=addgwchars', 
                          $msg ); 
                          }
    function cancel() { 
      $msg = JText::_( 'Canceled' ); 
      $this->setRedirect( 'index.php?option=com_gwchars&view=addgwchars', 
                          $msg ); 
   }
} 
?>

model

PHP-Code:

<?php 
defined('_JEXEC') or die('Restriced Access'); 
jimport('joomla.application.component.model'); 
class GwcharsModelAddgwchar extends JModel { 
   function __construct() { 
      parent::__construct(); 
      $array = JRequest::getVar('cid',  0, '', 'array'); 
      $this->setId($array[0]); 
   } 
   function setId($id) { 
      $this->_id = $id; 
      $this->_data = null; 
   }
   function getData() { 
      if (empty( $this->_data )) { 
         $query = ' SELECT * FROM #__gwchars '. 
                  '  WHERE id = '.$this->_id; 
         $this->_db->setQuery( $query ); 
         $this->_data = $this->_db->loadObject(); 
      } 
      if (!$this->_data) { 
        $this->_data = new stdClass(); 
        $this->_data->id = 0; 
        $this->_data->name = null; 
        $this->_data->gender = null;
        $this->_data->race = null;
        $this->_data->profession = null;
        $this->_data->published = null;
        /*UserID*/
        $this->_data->user = null; 
      } 
      return $this->_data; 
   }
   function store() { 
      $row =& $this->getTable(gwchar); 
      $data = JRequest::get( 'post' ); 
      if (!$row->bind($data)) { 
         $this->setError($this->getError()); 
         return false; 
      } 
      if (!$row->check()) { 
         $this->setError($this->getError()); 
         return false; 
      } 
      if (!$row->store()) { 
         $this->setError( $row->_db->getErrorMsg() ); 
         return false; 
      } 
      return true; 
   }
   function delete() { 
      $cids = JRequest::getVar( 'cid', array(0), 
                                'post', 'array' ); 
      $row =& $this->getTable(gwchar); 
      if (count( $cids )) { 
         foreach($cids as $cid) { 
            if (!$row->delete( $cid )) { 
               $this->setError( $row->getErrorMsg() ); 
               return false; 
            } 
         } 
      } 
      return true; 
   } 
} 
?>

Template

PHP-Code:

<!-- userID --> 
<input type="hidden" name="user" value="<?php echo $this->addgwchar->user; ?>" />
<input type="hidden" name="task" value="save" />
<input type="submit" name="Absenden" class="button validate" />

Leider funktioniert es nicht wie gewünscht. Die UserID wird nicht eingetragen. Kann mir da jemand auf die Sprünge helfen?
Danke schon im Voraus

habe das nun versucht:

Model

PHP-Code:

<?php 
defined('_JEXEC') or die('Restriced Access'); 
jimport('joomla.application.component.model'); 
class GwcharsModelAddgwchar extends JModel { 
   function __construct() { 
      parent::__construct(); 
      $array = JRequest::getVar('cid',  0, '', 'array'); 
      $this->setId($array[0]);
      /*userID*/
      $getuser = & JFactory::getUser();
      $this->setUser( $getuser );
   } 
   function setId($id) { 
      $this->_id = $id; 
      $this->_data = null; 
   }
   
   /*userID*/
   function setUser($user) {
       $this->_user = $user;
       $this->_data = null; 
   }
   function getData() { 
      if (empty( $this->_data )) { 
         $query = ' SELECT * FROM #__gwchars '. 
                  '  WHERE id = '.$this->_id; 
         $this->_db->setQuery( $query ); 
         $this->_data = $this->_db->loadObject(); 
      } 
      if (!$this->_data) { 
        $this->_data = new stdClass(); 
        $this->_data->id = 0; 
        $this->_data->name = null; 
        $this->_data->gender = null;
        $this->_data->race = null;
        $this->_data->profession = null;
        $this->_data->published = null;
        /*UserID*/
        $this->_data->user = null; 
      } 
      return $this->_data; 
   }
   function store() { 
      $row =& $this->getTable('gwchar'); 
      $data = JRequest::get( 'post' ); 
      if (!$row->bind($data)) { 
         $this->setError($this->getError()); 
         return false; 
      } 
      if (!$row->check()) { 
         $this->setError($this->getError()); 
         return false; 
      } 
      if (!$row->store()) { 
         $this->setError( $row->_db->getErrorMsg() ); 
         return false; 
      } 
      return true; 
   }
   function delete() { 
      $cids = JRequest::getVar( 'cid', array(0), 
                                'post', 'array' ); 
      $row =& $this->getTable(gwchar); 
      if (count( $cids )) { 
         foreach($cids as $cid) { 
            if (!$row->delete( $cid )) { 
               $this->setError( $row->getErrorMsg() ); 
               return false; 
            } 
         } 
      } 
      return true; 
   } 
} 
?>

die userID wird aber nach wie vor nicht in die Datenbank geschrieben :S

1. Möglichkeit: den Debugger anschmeissen und dort die Werte überprüfen..

2. Möglichkeit: füge überall wo Du Werte änderst ein print_r() oder var_dump() hinzu und bevor dann redirected wird (falls), ein jexit()

Dann solltest Du sehen wo es hakt

/edit: Also ich soll die UserID gar nicht per Formular übertragen, sondern direkt im controller? Würde insofern sinn machen als sonst einfach jeder mit Firebug im hidden input ne andere ID reinpflanzen könnte. Habs darum also im Model versucht:

PHP-Code:

<?php 
defined('_JEXEC') or die('Restriced Access'); 
jimport('joomla.application.component.model'); 
class GwcharsModelAddgwchar extends JModel { 
   function __construct() { 
      parent::__construct(); 
      $array = JRequest::getVar('cid',  0, '', 'array'); 
      $this->setId($array[0]);
   } 
   
   function setId($id) { 
      $this->_id = $id; 
      $this->_data = null; 
   }

   function getData() { 
   /*set userID*/
   $user         = & JFactory::getUser();
      if (empty( $this->_data )) { 
         $query = ' SELECT * FROM #__gwchars '. 
                  '  WHERE id = '.$this->_id; 
         $this->_db->setQuery( $query ); 
         $this->_data = $this->_db->loadObject(); 
      } 
      if (!$this->_data) { 
        $this->_data = new stdClass(); 
        $this->_data->id = 0; 
        $this->_data->name = null; 
        $this->_data->gender = null;
        $this->_data->race = null;
        $this->_data->profession = null;
        $this->_data->published = null;
        /*write userID*/
        $this->_data->user = $user->get('id'); 
      } 
      return $this->_data; 
   }
   function store() { 
      $row =& $this->getTable('gwchar'); 
      $data = JRequest::get( 'post' ); 
      if (!$row->bind($data)) { 
         $this->setError($this->getError()); 
         return false; 
      } 
      if (!$row->check()) { 
         $this->setError($this->getError()); 
         return false; 
      } 
      if (!$row->store()) { 
         $this->setError( $row->_db->getErrorMsg() ); 
         return false; 
      } 
      return true; 
   }
   function delete() { 
      $cids = JRequest::getVar( 'cid', array(0), 
                                'post', 'array' ); 
      $row =& $this->getTable(gwchar); 
      if (count( $cids )) { 
         foreach($cids as $cid) { 
            if (!$row->delete( $cid )) { 
               $this->setError( $row->getErrorMsg() ); 
               return false; 
            } 
         } 
      } 
      return true; 
   } 
} 
?>

Leider funktioniert das nicht. Weiss jemand warum?

Ich würde die Userid nicht bei getData zuweisen, sondern in deinem Controller oder View, bevor du die store-Methode aufrufst.

Zitat von progandy

Ich würde die Userid nicht bei getData zuweisen, sondern in deinem Controller oder View, bevor du die store-Methode aufrufst.

PHP-Code:

   function store() { 
      $row =& $this->getTable('gwchar'); 
      $data = JRequest::get( 'post' );
      /*Get userID*/
      $data['user_id'] = JFactory::getUser()->get('id');
      if (!$row->bind($data)) { 
         $this->setError($this->getError());
         return false; 
      } 
      if (!$row->check()) { 
         $this->setError($this->getError()); 
         return false; 
      } 
      if (!$row->store()) { 
         $this->setError( $row->_db->getErrorMsg() ); 
         return false; 
      } 
      return true; 
   } 


Meinst du so? Bin mir grad nicht sicher ob das sauber ist, ausserdem funktioniert es nicht. Die UserID wird aus ungerklärlichen Gründen nicht in die Datenbank geschrieben und print_r($data); gibt 1 aus XD (Also es werden alle Daten vom Formular in die DB geschrieben, aber eben nicht die UserID)
Kann mir da jemand helfen?

dein id feld der tabelle heisst wirklich "user_id" und nicht zufaellig nur "id" oder gar anders ? ist das feld "user_id" in deiner von jtable abgeleiteten klasse definiert ? abgesehen davon, spar dir den manuellen aufruf von bind, check und store und nutze jtable::save, was von sich aus bind, check und store sowie checkin aufruft und ein override der methoden in deiner jtable klasse ermoeglicht. sofern du selbst keine check methode implementiert hast, macht check in deinem fall nichts ausser "return true" zurueckgeben. aktuell werden alle post ($_POST) daten nach durchlauf des "strict filter" an jtable und deren properties, und letztendlich an die datenbankfelder gebunden. es gilt hier zu schauen ob das so auch gewollt ist.

Zitat von Matrikular

dein id feld der tabelle heisst wirklich "user_id" und nicht zufaellig nur "id" oder gar anders ? ist das feld "user_id" in deiner von jtable abgeleiteten klasse definiert ? abgesehen davon, spar dir den manuellen aufruf von bind, check und store und nutze jtable::save, was von sich aus bind, check und store sowie checkin aufruft und ein override der methoden in deiner jtable klasse ermoeglicht. sofern du selbst keine check methode implementiert hast, macht check in deinem fall nichts ausser "return true" zurueckgeben. aktuell werden alle post ($_POST) daten nach durchlauf des "strict filter" an jtable und deren properties, und letztendlich an die datenbankfelder gebunden. es gilt hier zu schauen ob das so auch gewollt ist.

Peinlich peinlich, ganz richtig. Ein simpler Schreibfehler. Tut mir leid Leute euch deswegen derartige Umstände bereitet zu haben.
was jtable::save anbelangt: Ich habe erst grad angefangen mit joomla komponente programmierung und habe mir auf Galileo computing den Guide durchgelesen sowie mir diverse Komponente wie z.B. Eventlist (die mir sauber und professionell vorkamen) angeschaut und da hab ich bis jetz NIE jtable::save gefunden. Dies soll aber nicht heissen, dass jtable::save nicht die bessere Lösung ist ;-)
Ich habe mir das gerade auf Joomwiki angeschaut, stehe aber noch auf dem Schlauch. Wie würde das konkret kurz umrissen in meinen Beispieldaten aussehen?
Den letzten Abschnitt habe ich gar nicht verstanden:

aktuell werden alle post ($_POST) daten nach durchlauf des "strict filter" an jtable und deren properties, und letztendlich an die datenbankfelder gebunden. es gilt hier zu schauen ob das so auch gewollt ist.

Zwischen den Zeilen lese ich, dass ich irgendetwas an dem Code nicht verstehe ;-) Was soll genau wie gemeint sein? Falls du keine Zeit für eine Antwort hast oder ich mich eingehender mit der Materie beschäftigen sollte um die angepsrochenen Dinge zu verstehen einfach sagen, habe nämlich bei dem Post grad das Gefühl, dass ich zu wenig Grundwisen habe ;-)
Gruss