Dylan Swift wrote:
Hello All,
Thanks to all for providing us with Cobbler - I have only been using
it a few days and I have managed to save myself several hours work by
using it to perform bios updates to IBM blades. So in order to do my
bit, here is a patch that I believe will fix a ticket I have raised on
trac:
When editing images the page always reports 'image type' iso,
regardless of the type specified in the image definition in
/var/lib/cobbler/config/images.d/<image_definition_file>
I'm not familiar with Python or Cheetah, so I may be undoing something
that serves a genuine purpose, but it appears that two additional
lines have been added to the template file
/usr/share/cobbler/webui_templates/image_edit.tmpl in an if statement
where the image_type is checked (around line #337). It looks like it
has been added for a reason, but I can't quite work out why! It may be
obvious to someone, but as I say I'm new to cobbler and
python/cheetah.
So, I propose the following patch to fix this behaviour
--- /usr/share/cobbler/webui_templates/image_edit.tmpl.orig
2009-01-26 21:01:01.000000000 +0000
+++ /usr/share/cobbler/webui_templates/image_edit.tmpl 2009-01-26
21:01:27.000000000 +0000
@@ -329,22 +329,20 @@
<td>
<label for="imagetype">Image Type</label>
</td>
<td>
<select name="imagetype" id="imagetype">
#for $it in [ "direct", "iso", "memdisk",
"virt-clone" ]:
<option value="$it"
#if $image and $image.image_type == $it
selected="1"
- #else if $it == "iso"
- selected="1"
#end if
>$it</option>
#end for
</select>
<p class="context-tip">What type of OS?</p>
</td>
</tr>
Regards
Dylan
_______________________________________________
cobbler mailing list
cobbler(a)lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/cobbler
This looks like (busted) code to make sure one type is always selected
if none is selected. (Sometimes the web app doesn't get as much love as
the CLI, so it takes a while for new things to be perfect with it).
Anyway, your patch is fine, as it forces someone to choose a type rather
than accept the default.
Applied to both master and devel branches, thanks!
--Michael