aiuto con update

[kaneda986]

Salve ragazzi, il titolo fa abbastanza pena, ma spero che sia comprensivo:

devo fare tramite un tasto l'UPDATE di un campo di una tabella.

ora non funziona.. non so il perché, chissà dove avrò sbagliato.


      function &add($mconsi) {
        $sql = "UPDATE  #__users SET  `consiglio` =  '1' WHERE  `jos_users`.`id` = $mconsi";
        $this->_db->setQuery($sql);
        $users = $this->_db->loadObjectList();
        return $users;
       
        } 

non fate caso che cosa vado a modificare , con l'INSERT e CANCEL funziona. ma secondo me non è corretto il procedimento per l'UPDATE, mi potete dare una mano?

[mmleoni]

a me sembra tutto sbagliato... 

sicuramente è perché mancano dei pezzi, ma (nell'ordine):
.&add: perchè &??
.$mconsi: sicuro che sia un int?
.this_db: è inizializzato??
.$this->_db->loadObjectList: un update riporta al massimo il numero delle righe oggetto della modifica, non una lista di oggetti!
.return $users: che rappresenta $users? al massimo può essere uno scalare (vedi sopra)

ciao,
marco

[mattialori]

prova così:

Codice: [Seleziona]

function add($mconsi) {
        $sql = "UPDATE  #__users SET  `consiglio` =  '1' WHERE  `jos_users`.`id` = $mconsi";
        $this->_db->setQuery($sql);
        $users = $this->_db->query();
        return;
       
        } 


[mmleoni]

Codice: [Seleziona]

function add($mconsi) {
        $sql = "UPDATE  #__users SET  `consiglio` =  '1' WHERE  `jos_users`.`id` = $mconsi";
        $this->_db->setQuery($sql);
        $users = $this->_db->query();
        return;
       
        } 


scusate ma:
.a che serve assegnare il risultato di query() ad una variabile che non viene usta?
.a che serve un return [void] prima della chiusura del metodo/function?

@kaneda986
che ti aspettavi ci fosse in $users?

ciao,
marco

[mattialori]

si scusa user davanti a query() è stata una sfuggita, il return vuoto lo ho messo senza un valido motivo, se gl interessa ritornare qualcosa