Codeigniter: не могу заставить мою правку работать - PullRequest
Новая
       3

Codeigniter: не могу заставить мою правку работать

0 голосов
/ 29 сентября 2011

Итак, в основном я пытаюсь объединить то, что я нашел в этих двух уроках, чтобы создать работающий CRUD: http://fwebde.com/php/simple-crud-with-codeigniter/

http://ie.mirror.twsweb -int.com / CodeIgniter / user_guide / учебник / create_news_items.html

По сути, я могу заставить страницу редактирования отобразить соответствующие детали элемента в форме, но всякий раз, когда я нажимаю кнопку отправки, я получаю ошибку 404, и таблица не обновляется.

Мой edit.php, связанный с моим новостным контроллером: 

<?php echo validation_errors(); ?>
<h2>Edit a news item</h2>

<?php echo form_open('news/edit') ?>

<p>
<label for="title">Title</label> 
<?php echo form_input('title',$news_item['title']); ?>
</p>

<p>
<label for="text">Text</label>
<?php echo form_textarea('text',$news_item['text']); ?>
</p>    

<?php echo form_hidden($news_item['id']); ?>

<p>
<?php echo form_submit('submit', 'Save Changes'); ?>
</p>

  <?php echo form_close(); ?>

</br>
</br>
<a href='<?php echo site_url('news');?>'>Back</a>

Мой новостной контроллер по методу редактирования: 

public function edit($slug)
{
    $data['news_item'] = $this->news_model->get_news($slug);

    if (empty($data['news_item']))
    {
        show_404();
    }

    $data['title'] = 'Edit: '.$data['news_item']['title'];

    $this->load->helper('form');
    $this->load->library('form_validation');

    $this->form_validation->set_rules('title', 'Title', 'required');
    $this->form_validation->set_rules('text', 'text', 'required');

    if($this->form_validation->run() === FALSE)
    {
        $this->load->view('templates/header', $data);   
        $this->load->view('news/edit', $data);
        $this->load->view('templates/footer');
    }
    else
    {
        $this->news_model->update_news( $this->input->post('id'),
                                        $this->input->post('title'),
                                        $this->input->post('text'));
        $this->load->view('news/success');
    }
}

Подходящие методы в моей модели: 

public function get_news($slug = FALSE)
{
    if ($slug === FALSE)
    {
        $query = $this->db->get('news');
        return $query->result_array();
    }

    $query = $this->db->get_where('news', array('slug' => $slug));
    return $query->row_array();
}

public function set_news()
{
    $this->load->helper('url');

    $slug = url_title($this->input->post('title'), 'dash', TRUE);

    $data = array(
        'title' => $this->input->post('title'),
        'slug' => $slug,
        'text' => $this->input->post('text')
    );

    return $this->db->insert('news', $data);
}

public function update_news($id, $title, $content) {
    $data = array(
        'title' => $title,
        'content' => $content
    );

    $this->db->where('id', $id);
    $this->db->update('news', $data);
}

Мои маршруты: 

$route['news/edit/(:any)'] = 'news/edit/$1';
$route['news/create'] = 'news/create';
$route['news/(:any)'] = 'news/view/$1';
$route['news'] = 'news';

Ответы [ 2 ]

2 голосов
/ 04 октября 2011

В основном было несколько ошибок:

1) предложение Ифаура

2) передача первичного ключа и slug в качестве параметров, которые будут использоваться позже

3) забыл, что я изменил столбец таблицы (забыл, что я переименовал столбец содержимого как текст)

Модель в окончательном виде: 

public function get_news($slug = FALSE)
{
    if ($slug === FALSE)
    {
        $query = $this->db->get('news');
        return $query->result_array();
    }

    $query = $this->db->get_where('news', array('slug' => $slug));
    return $query->row_array();
}

public function set_news()
{
    $this->load->helper('url');

    $slug = url_title($this->input->post('title'), 'dash', TRUE);

    $data = array(
        'title' => $this->input->post('title'),
        'slug' => $slug,
        'text' => $this->input->post('text')
    );

    return $this->db->insert('news', $data);
}

public function update_news($id, $title, $content) {
    $data = array(
        'title' => $title,
        'text' => $content
    );

    $this->db->where('id', $id);
    $this->db->update('news', $data);
}

Изменить в окончательном виде: 

<h2>Edit a news item</h2>

<a href="<?php echo site_url('news/delete/'.$news_item['id']); ?>">Delete this article</a> 

<?php echo validation_errors(); ?>

<?php echo form_open('news/edit/'.$news_item['slug']) ?>

<p>
<label for="title">Title</label> 
<?php echo form_input('title',$news_item['title']); ?>
</p>

<p>
<label for="text">Text</label>
<?php echo form_textarea('text',$news_item['text']); ?>
</p>    

<?php echo form_hidden('slug',$news_item['slug']); ?>
<?php echo form_hidden('id',$news_item['id']); ?>

<p>
<?php echo form_submit('submit', 'Save Changes'); ?>
</p>

<?php echo form_close(); ?>

</br>
</br>
<a href='<?php echo site_url('news');?>'>Back</a>

Метод редактирования контроллера новостей в окончательной форме: 

public function edit($slug)
{
    $data['news_item'] = $this->news_model->get_news($slug);

    if (empty($data['news_item']))
    {
        show_404();
    }

    $data['title'] = 'Edit: '.$data['news_item']['title'];

    $this->load->helper('form');
    $this->load->library('form_validation');

    $this->form_validation->set_rules('title', 'Title', 'required');
    $this->form_validation->set_rules('text', 'text', 'required');

    if($this->form_validation->run() === FALSE)
    {
        $this->load->view('templates/header', $data);   
        $this->load->view('news/edit', $data);
        $this->load->view('templates/footer');
    }
    else
    {
        $this->news_model->update_news( $this->input->post('id'),
                                        $this->input->post('title'),
                                        $this->input->post('text'));


        $data['news_item'] = $this->news_model->get_news($slug);
        $this->load->view('templates/header', $data);                                   
        $this->load->view('news/success');
        $this->load->view('news/edit', $data);
        $this->load->view('templates/header', $data);   
    }
}
2 голосов
/ 29 сентября 2011

Вы отправляете форму для метода edit, который ожидает идентификатор элемента $slug, вы можете изменить свое представление на следующее: 

<?php echo form_open('news/edit/' . $news_item['id']) ?>
...