Codeigniter: A PHP Error was encountered

Question

I'm new to codeigniter.I'm able to add the data but unable to retrieve from data base to edit. Error shown below:

Severity: Notice Message: Undefined variable: mailshot Filename: mailshot/edit.php Line Number: 17

Help me with fixing error. Here is the code for Controller and the form followed:

public function edit() {

    /* Load model */
    $this -> load -> model('mailshot_model');
    /* End here */

        $data = array();

    if ($this -> input -> post()) {

        $data = array('title' => strip_tags(addslashes(trim($this -> input -> post('title')))),
        'url' => strip_tags(addslashes(trim($this -> input -> post('mailshoturl')))),
        'type' => strip_tags(addslashes(trim($this -> input -> post('type')))),
        'status' => strip_tags(addslashes(trim($this -> input -> post('status')))),
        'modified_by' => $this -> userid);

        $lastid = $this -> mailshot_model -> update_record($data);

        $links = array('msid' => $lastid,
        'ms_title' => strip_tags(addslashes(trim($this -> input -> post('title')))),
        'status' => strip_tags(addslashes(trim($this -> input -> post('status')))),
        'modified_by' => $this -> userid);


        /* Load Model */    
        $this -> load -> model('mailshotlinks_model');
    /* End here */
        $linkid = $this -> mailshotlinks_model -> update_record($links);
        if (isset($lastid)) {
            redirect(base_url() . 'mailshot', 'location');

        }
        //$lastid = $this -> mailshot_model -> add_record($data);
         else {
            $data = array("error" => 'Unable to create record.Please try agin ');
        }
    }

    $this -> layout -> view('mailshot/edit', $data);

}

<form class="mws-form" action="#" method="post" id="addevent_form"  enctype="multipart/form-data" >
                <div class="mws-form-inline">
                    <div class="mws-form-row">
                        <label class="mws-form-label">Mailshot Title:<span style="color: red">*</span></label>
                        <div class="mws-form-item">
                        <input type="text" name="title" id="title" value="<?php echo $mailshot['title']; ?>" class="medium">
                    </div>
                    </div>
                </div>
                <div class="mws-form-inline">
                    <div class="mws-form-row">
                        <label class="mws-form-label">URL:<span style="color: red">*</span></label>
                        <div class="mws-form-item">
                        <input type="text" name="mailshoturl" id="mailshoturl" value="<?php echo $mailshot['url']; ?>" class="medium">
                    </div>
                    </div>
                </div>



                <div class="mws-form-inline">
                    <div class="mws-form-row">
                        <label class="mws-form-label">Type:<span style="color: red">*</span></label>
                        <div class="mws-form-item">
                        <select name="type" id="type">
                            <option value="" selected="selected">Choose Mailshot Type</option>
                            <option value="paid" <?php if($mailshot['type'] == 'paid'){ echo "selected=selected"; } ?>>Paid</option>
                            <option value="free" <?php if($mailshot['type'] == 'free'){ echo "selected=selected"; } ?>>Free</option>


                        </select>
                        </div>
                    </div>
                </div>


                <div class="mws-form-inline">
        <div class="mws-form-row">
        <label class="mws-form-label">Status:<span style="color: red">*</span></label>
             <div class="mws-form-item">
              <select name="status" id="status">
                <option value="">--None--</option>
               <option value="Active" <?php if($mailshot['status']=='Active'){ echo "selected=selected"; }?>>Active</option>
              <option value="Inactive" <?php if($mailshot['status']=='Inactive'){ echo "selected=selected"; }?>>In Active</option>
              <option value="Disabled" <?php if($mailshot['status']=='Disabled'){ echo "selected=selected"; }?>>Disabled</option>
              </select>
             </div>
             </div>
      </div>

                <div class="mws-button-row">
                    <input type="submit" value="Submit" class="btn btn-success">
                </div>
            </form>

edit question and remove unnecessary code http://stackoverflow.com/help/mcve — Karan Thakkar, Sep 25 '14 at 10:39

score 0 · Answer 1 · answered Sep 25 '14 at 10:50

Don't forget to use get() method after all your querys.

 function listAll() {

            $query = $this -> db -> query("select a.id as id ,
             a.title as title,
             a.url as url,
             a.type as type,
             a.type as status,
             a.description as description,
             a.status as status,
             a.created_date as created_date,
             a.modified_date as modified_date,
             u.username AS created_by,
             c.username AS modified_by from " . $this -> tablename . " as a 
             LEFT JOIN tbl_admin u ON a.created_by = u.id
             LEFT JOIN tbl_admin c ON a.modified_by = c.id order by a.id desc")->get();
            return $query -> result_array();
        }

        function listone($id) {

            $query = $this -> db -> query("SELECT
             a.id as id ,
             a.title as title,
             a.url as url,
             a.type as type,
             a.type as status,
             a.status as status,
             a.modified_date as modified_date,
             c.username AS modified_by
             FROM " . $this -> tablename . " AS a
             LEFT JOIN tbl_admin c ON a.modified_by = c.id
             WHERE a.id=" . $id)->get();

            return $query -> result_array();
        }

        function get_record($id) {
            $query = $this -> db -> query("SELECT
             a.id as id ,
             a.title as title,
             a.url as url,
             a.type as type,
             a.type as status,
             a.status as status,
             a.created_date as created_date,
             a.modified_date as modified_date,
             u.username AS created_by,
             c.username AS modified_by
             FROM " . $this -> tablename . " AS a
             LEFT JOIN tbl_admin c ON a.modified_by = c.id
             WHERE  a.id=" . $id)->get();

            return $query -> row_array();
        }

posting random code snippets without explanation? you've just earned my downvote. — Karoly Horvath, Sep 25 '14 at 10:52
:(( I'm posting answer, maybe someone in hurry. And after, I explain it. :P — Aleksandr Golubovskij, Sep 25 '14 at 10:55
Hi Aleksandr, Thank you for post. The get() method works. Even I need to change the result_array() to row_array() as per my requirement in listone() function. Sorted out. Thank You for support. — Deepak Yadav, Oct 09 '14 at 09:19

Codeigniter: A PHP Error was encountered

}

1 Answers1