codeigniter php

Question

Hej mam taki problem, piszę stronkę w codeigniter doczhodzę do części logowania i mimo że wszystko jest napisane jak trzeba to nie działa....a dokładniej to ten debilny password_verify...oto kod:

Robiłem chyba już wszystko i nie działa nic!!!!!

public function login()
    {

        logged_in() == false || redirect( '/' );

        if ( !empty( $_POST ) )
        {

            if ( $this->form_validation->run( 'site_account_login' ) == true )
            {

                $email = $this->input->post( 'email' , true );
                $haslo = $this->input->post( 'haslo' , true );

                $where = array( 'email' => $email );
                $user = $this->Site_model->get_single( 'users' , $where );

                if ( !empty( $user ) )
                {

                    if ( password_verify( $haslo , $user->haslo ) == true )
                    {

                        // Czy użytkownik jest aktywny

                        if ( $user->active == 1 )
                        {

                            // Zalogowanie użytkownika

                            $data_login = array(
                                'id' => $user->id,
                                'email' => $user->email,
                                'logged_in' => true,
                            );
                            $this->session->set_userdata( $data_login );

                            $this->session->set_flashdata( 'alert' , 'Zalogowałeś się poprawnie.' );
                            redirect( '/' );

                        }
                        else
                        {
                            $this->session->set_flashdata( 'alert' , 'Musisz aktywować swoje konto.' );
                            refresh();
                        }

                    }
                    else
                    {
                        // Użytkownik podał błędne hasło dla podanego adresu email

                        $this->session->set_flashdata( 'alert' , 'Błędne hasło.' );
                        refresh();
                    }

                }
                else
                {
                    // Nie istnieje taki użytkownik

                    $this->session->set_flashdata( 'alert' , 'Użytkownik z podanym adresem email nie istnieje.' );
                    refresh();
                }

            }
            else
            {
                $this->session->set_flashdata( 'alert' , validation_errors() );
                refresh();
            }

        }

        $this->load->view( 'site/account/login' );
    }

}

Comments

Kod umieść w bloczku i nie obrażaj biednego password_verify.

---

To jest najnowsza wersja CodeIgnitera? To wygląda gorzej niż Symfony 1.5. Jeszcze ten charakterystyczny piramida code style.

---

@OdsetekGlupoty, Sory mam przez to nerwy, jak mam umieścić kod w bloczku?? może to być dziwne te pytanie, ale już mi się mózg lasuje od tego.

---

@Ehlert, CI to framework, którego funkcje związane z HTML oraz sam silnik template nie mają wsparcia dla escapowania stringów więc XD 

Answer 1

Hehe ojoj, jak ty się w tym łapiesz. Takiej piramidy ifów dawno nie widziałem :P Potem ludzie mówią, że CI jest beznadziejny (a to mój najukochańszy framework ;)) :P Ogólnie przepisz sobie ten kod, bo w takiej plątaninie nie trudno się pogubić. Każde sprawdzenie wrzuć do osobnej funkcji. Np. sprawdzenie czy jest już użytkownik o takim mailu wrzuć do funkcji isEmailInDatabase (czy coś takiego). Wydziel sprawdzanie hasła, emailu (czy są zgodne z tym co jest w bazie danych). Jak sobei to podzielisz wszyscy co mozesz do osobnych funkcji to nagle zobaczysz, że potrzebujesz 1, góra 2 ifów do sprawdzenia ;)