Bogue de l'application du blog Codeigniter: le formulaire est chargé sans erreur de validation

Ce que je ferais dans ce scénario est de fusionner les deux méthodes en une seule méthode en se basant sur la validation pour savoir si j'enregistre l'entrée ou non.

votre code ressemblerait à ceci:

public function edit($id) {
    // Only logged in users can edit posts
    if (!$this->session->userdata('is_logged_in')) {
        redirect('login');
    }
    $Post = $this->Posts_model->get_post($id);
    // user does not own the post, redirect
    if ($this->session->userdata('user_id') !== $Post->author_id) {
        redirect('/' . $id);
    }

    // Form data validation rules
    $this->form_validation->set_rules('title', 'Title', 'required',  array('required' => 'The %s field can not be empty'));
    $this->form_validation->set_rules('desc', 'Short description', 'required',  array('required' => 'The %s field can not be empty'));
    $this->form_validation->set_rules('body', 'Body', 'required',  array('required' => 'The %s field can not be empty'));
    $this->form_validation->set_error_delimiters('<p class="error-message">', '</p>');

    // if validation fails, or the form isn't submitted
    if ($this->form_validation->run() === false ) {
        $data = $this->Static_model->get_static_data();
        $data['pages'] = $this->Pages_model->get_pages();
        $data['categories'] = $this->Categories_model->get_categories();
        $data['posts'] = $this->Posts_model->sidebar_posts($limit=5, $offset=0);
        $data['post'] = $Post;

        $data['tagline'] = 'Edit the post "' . $data['post']->title . '"';
        $this->load->view('partials/header', $data);
        $this->load->view('edit-post');
        $this->load->view('partials/footer');
    }else{
        // Update slug (from title)
        if (! empty($this->input->post('title'))) {
            $slug = url_title($this->input->post('title'), 'dash', TRUE);
            $slugcount = $this->Posts_model->slug_count($slug);
            if ($slugcount > 0) {
                $slug = $slug."-".$slugcount;
            }
        } else {
            $slug = $this->input->post('slug');
        }
        // Upload image
        $config['upload_path'] = './assets/img/posts';
        $config['allowed_types'] = 'jpg|png';
        $config['max_size'] = '2048';

        $this->load->library('upload', $config);

        if(!$this->upload->do_upload()){
            $errors = array('error' => $this->upload->display_errors());
            $post_image = $this->input->post('postimage');
        } else {
            $data = array('upload_data' => $this->upload->data());
            $post_image = $_FILES['userfile']['name'];
        }       
        $this->Posts_model->update_post($id, $post_image, $slug);
        $this->session->set_flashdata('post_updated', 'Your post has been updated');
        redirect('/' . $slug);
    }
}

et vous n'avez pas à implémenter la mise à jour séparément, il vous suffit de publier dans / edit / $ id au lieu de / update / $ id ... ceci est un exemple approximatif, votre vérification de slug ( ce que je n'ai pas abordé) n'est pas la bonne façon de le faire, si elle réussit la validation, le titre est déjà rempli car il est défini sur requis donc je suppose que vous vouliez dire if (! empty (slug)) code>, mais encore une fois dans votre else, vous définissez le slug directement à partir de l'entrée de l'utilisateur, donc je l'ajouterais à la validation et je m'assurerais qu'il est unique dans la base de données à l'exception du $ id en cours de modification.

Encore une fois, c'est le résultat d'un copier-coller de votre code d'origine.J'ai peut-être manqué quelque chose, alors faites-le bien lire pour vous assurer que rien ne manque et incluez les erreurs de validation dans les données que vous passez à la vue. p>

Vous avez 3 options:

1. Utilisez les données flash dans la méthode de mise à jour en cas d'échec (vous les utilisez déjà en cas de succès). Attribuez simplement les erreurs à une variable de données Flash et récupérez-la après avoir redirigé vers la modification.
2. Combinez les méthodes d'édition et de mise à jour (les plus courantes pour une utilisation non ajax).
3. Utilisez ajax et renvoyez des chaînes codées json pour les erreurs ou les messages de réussite.

Option 2:

Cette option résout également le problème d'authentification potentiel signalé dans les commentaires.

Veuillez lire les commentaires intégrés dans le code. p >

public function edit($id) {
        // Only logged in users can edit posts
        if (!$this->session->userdata('is_logged_in')) {
            redirect('login');
        }

        $data['post'] = $this->Posts_model->get_post($id);

        if ($this->session->userdata('user_id') == $data['post']->author_id) {
            show_error('Access denied'); // function exits
        }

        if ($_POST) {

            $this->form_validation->set_rules('title', 'Title', 'required', array('required' => 'The %s field can not be empty'));
            $this->form_validation->set_rules('desc', 'Short description', 'required', array('required' => 'The %s field can not be empty'));
            $this->form_validation->set_rules('body', 'Body', 'required', array('required' => 'The %s field can not be empty'));
            $this->form_validation->set_error_delimiters('<p class="error-message">', '</p>');

            //$id = $this->input->post('id'); not required anymore

            $config['upload_path'] = './assets/img/posts';
            $config['allowed_types'] = 'jpg|png';
            $config['max_size'] = '2048';

            $this->load->library('upload', $config);

            if ($this->form_validation->run() && $this->upload->do_upload()) {
                // always use the name from the upload lib
                // sometimes it changes it in case of duplicates (read docs for more)
                $post_image = $this->upload->data('file_name');

                // doesn't make sense with title validation rule, this will always be true to get
                // passed validation
                if (!empty($this->input->post('title'))) {
                    $slug = url_title($this->input->post('title'), 'dash', TRUE);
                    $slugcount = $this->Posts_model->slug_count($slug);
                    if ($slugcount > 0) {
                        $slug = $slug . "-" . $slugcount;
                    }
                } else {
                    $slug = $this->input->post('slug');
                }

                $this->Posts_model->update_post($id, $post_image, $slug);
                $this->session->set_flashdata('post_updated', 'Your post has been updated');
                redirect('/' . $slug);
            } else {
                $data['errors'] = validation_errors() . $this->upload->display_errors();
            }
        }

        $data = $this->Static_model->get_static_data();
        $data['pages'] = $this->Pages_model->get_pages();
        $data['categories'] = $this->Categories_model->get_categories();
        $data['posts'] = $this->Posts_model->sidebar_posts($limit = 5, $offset = 0);
        $data['tagline'] = 'Edit the post "' . $data['post']->title . '"';
        $this->load->view('partials/header', $data);
        $this->load->view('edit-post');
        $this->load->view('partials/footer');
    }

J'ai réussi à obtenir le résultat souhaité à l'aide de set_flashdata () comme suggéré par @Alex

dans le contrôleur J'ai: xxx > Dans la vue: xxx