Wrong column names in views
Bug #1170680 reported by
Victor Nagy
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
LenaSYS |
Fix Committed
|
Undecided
|
Oscar Jonsson |
Bug Description
The column names in pretty much all the views are using the old column names. They should be fixed to correspond the correct names. Examples are the views in views/admin like views/admin/
Related branches
lp://staging/~c11oscjo/lenasys/corrected_names
- Gustav Hartvigsson: Approve
-
Diff: 54 lines (+13/-13)2 files modifiedcodeigniter/application/models/codeviewer/codeviewer_model.php (+12/-12)
codeigniter/application/views/admin/manage_course.php (+1/-1)
Changed in lenasys: | |
status: | New → Confirmed |
Changed in lenasys: | |
assignee: | nobody → Oscar Jonsson (c11oscjo) |
status: | Confirmed → In Progress |
status: | In Progress → Fix Committed |
Changed in lenasys: | |
status: | New → Confirmed |
Changed in lenasys: | |
status: | Confirmed → Fix Committed |
To post a comment you must log in.
This is still an issue.
For example in views/admin/ manage_ course. php:
<tbody> "window. location. href='< ?php echo base_url( );?>admin/ manageexample/ <?php echo $cid ?>/<?php echo $item->Name ?>'"><i class="icon-cog icon-white"></i> Manage< /button> </td>
<?php
foreach ($examples as $item):
?>
<tr>
<td><?php echo $item->name ?></td>
<td><button class="btn btn-primary" onClick=
</tr>
<?php endforeach;?>
</tbody>
Should be
<tbody> "window. location. href='< ?php echo base_url( );?>admin/ manageexample/ <?php echo $cid ?>/<?php echo $item->exampleName ?>'"><i class="icon-cog icon-white"></i> Manage< /button> </td>
<?php
foreach ($examples as $item):
?>
<tr>
<td><?php echo $item->exampleName ?></td>
<td><button class="btn btn-primary" onClick=
</tr>
<?php endforeach;?>
</tbody>
Notice how $item->name has changed to $item->exampleName. The property "exampleName" is the exact name of the column in the proper database table. All views should be updated to use the proper column names instead.