[ZF] Sprawdzenie czy użytkownik głosował Opcje

[razielnr1]

Sprawdzam czy użytkownik już głosował, jeśli nie - to może zagłosować ; jeśli tak - nie może oddać głosu i pojawia się komunikat, że już oddał głos.
Co prawda nie może oddać już głosu jeśli głosował, lecz nie pojawia się komunikat i nie wiem co jest źle w warunku.

Akcja:

[PHP] pobierz, plaintext 
public function voteAction() 
    {
 
        $auth = Zend_Auth::getInstance();
        $identity = $auth->getIdentity();
 
 
        if ($identity) {
 
            $votes = new Application_Model_DbTable_Votes();
 
            $answer_id = $this->getRequest()->getParam('answer');
            $identity->users_id;
 
 
            $vote = $votes->fetchRow($votes->select()
                    ->where('answer = ?', $answer_id));
 
 
            $answers = new Application_Model_DbTable_Answers();
 
            $mark = $this->getRequest()->getParam('mark');
 
 
            if (!$vote) {
 
                $answer = $answers->find($answer_id);
 
                if ($answer->count()) {
 
                    $answer = $answer->current();
 
                    if ($mark == 'plus') { 
                        ++$answer->mark; 
                        $votes->createRow(array(
                        'answer' => $answer_id,
                        'author' => $identity->users_id
                            ))->save();
                    }
                    else {
                        --$answer->mark; 
                        $votes->createRow(array(
                        'answer' => $answer_id,
                        'author' => $identity->users_id
                            ))->save(); 
                    }
                $answer->save();
                }   
            } 
 
            $this->view->vote = $vote;   
 
        }
 
        $session = new Zend_Session_Namespace;
 
        $this->_helper->redirector->gotoUrl($session->redirect);
 
    }
[PHP] pobierz, plaintext 

Widok:

[PHP] pobierz, plaintext 
<?php if ($this->identity): ?>  
            <?php if ($this->identity->users_id != $answer['author']): ?>
            <?php if ($this->identity->users_id != $this->vote['author'] && $this->vote['answer'] == NULL): ?>
            <a href="<?php echo $this->url(array('action' => 'vote', 'controller' => 'index', 'answer' => $answer['answers_id'], 'mark' => 'plus')); ?>">
                <i style="margin-right: 5px" class="icon-thumbs-up"></i>
                <span style="color: green;">Dobra odpowiedź</span>
            </a>
            &nbsp&nbsp&nbsp&nbsp
            <a href="<?php echo $this->url(array('action' => 'vote', 'controller' => 'index', 'answer' => $answer['answers_id'], 'mark' => 'minus')); ?>">
                <i style="margin-right: 5px" class="icon-thumbs-down"></i>
                <span style="color: red;">Zła odpowiedź</span>
            </a>
            <?php elseif ($this->identity->users_id == $this->vote['author'] && $this->vote['answer'] != NULL): ?>
            Oddałeś
            <?php endif; ?>
            <?php endif; ?>
            &nbsp&nbsp&nbsp&nbsp
            <span style="font-weight: bold;">
                Średnia ocen <?php echo '(' . $answer['mark'] . ')' . '&nbsp&nbsp&nbsp&nbsp'; ?>
            </span>  
            <?php endif; ?> 
 
[PHP] pobierz, plaintext 

[kamilus]

To nie ma prawa zadziałać.

[PHP] pobierz, plaintext 
<?php if ($this->identity->users_id != $this->vote['author'] && $this->vote['answer'] == NULL): ?>
[PHP] pobierz, plaintext 

Jeżeli dopiero głosowałeś to obiekt $this->vote w widoku jest pusty.

fetchRow zwraca obiekt, a nie tablicę.
Ogólnie nie jestem pewny linii 2 w widoku. Brakuje tutaj trochę logicznego myślenia.
Sprawdź czy został już oddany głos:

[PHP] pobierz, plaintext 
$vote = $votes->fetchRow($votes->select()
                    ->where('answer = ?', $answer_id));
[PHP] pobierz, plaintext 

zamień na:

[PHP] pobierz, plaintext 
$vote = $votes->fetchRow($votes->select()
                    ->where('answer = ?', $answer_id)->where('author=?',$identity->users_id);
[PHP] pobierz, plaintext 

Wówczas pierwszym warunkiem w widoku powinno być sprawdzenie, czy $this->vote jest obiektem (jeżeli tak to oddano głos).

dalej:

[PHP] pobierz, plaintext 
$votes->createRow(array( ... 
[PHP] pobierz, plaintext 

powinno być

[PHP] pobierz, plaintext 
$vote = $votes->createRow(array( ... 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$this->view->vote = $vote; 
[PHP] pobierz, plaintext 

Przenieś do warunku dla !$vote (wewnątrz ifa - przed zamykający } ).

I teraz w widoku możesz dać:

[PHP] pobierz, plaintext 
<? if($this->vote) {
 // .. odano głos
} else if ($this->vote->answer == NULL) {
 // ... nie oddałeś i tutaj spis opcji 
} 
[PHP] pobierz, plaintext 

[razielnr1]

Super fachowa pomoc. Dziękuję!