95967 - Mandatory fields

Status: RESOLVED WORKSFORME

(Bugzilla :: Bugzilla-General, enhancement, P4)

Description

[Roni Abusch]

Can I make another fields as mandatory? (not only the summary field)

Comment 1

[Roni Abusch]

I've hacked the code in my installation to do it.

Comment 2

[Dave Miller [:justdave]]

that doesn't mean we don't want to fix it in the distribution code, though.

Although since your patches probably didn't provide an option for it, and just
did it directly, they may be worth submitting anyway, just so we can see what
all had to be done.  We may have to change it to make it optional, but it'd
still be useful I think.

Comment 3

[Roni Abusch]

I've just added simple code to two files:
To CGI.pl I've added in make_popup a new option that say "choose one" that 
returns nothing:
sub make_popup {
    my ($name,$src,$default,$listtype,$onchange) = (@_);
    my $popup = "<SELECT NAME=$name";
    if ($listtype > 0) {
        $popup .= " SIZE=5";
        if ($listtype == 2) {
            $popup .= " MULTIPLE";
        }
    }
    if (defined $onchange && $onchange ne "") {
        $popup .= " onchange=$onchange";
    }
    $popup .= "><OPTION VALUE='' SELECTED>Choose one";
    $popup .= make_options($src, "",
                                 ($listtype == 2 && $default ne ""));
    $popup .= "</SELECT>";
    return $popup;
}

And to enter_bug.cgi I've added alerts to find if the fields have been changed:
<INPUT TYPE=\"submit\" VALUE=\"    Commit    \" ONCLICK=\"
if (this.form.version.value =='') { alert('Please enter product version for 
this bug.'); return false;}
if (this.form.bug_severity.value =='') { alert('Please enter severity for this 
bug.'); return false;}
if (this.form.op_sys.value =='') { alert('Please enter operating system for 
this bug.'); return false;}
if (this.form.rep_platform.value =='') { alert('Please enter platform for this 
bug.'); return false;}
if (this.form.component.value =='') { alert('Please enter component for this 
bug.'); return false;}
if (this.form.priority.value =='') { alert('Please enter priority for this 
bug.'); return false;}
if (this.form.short_desc.value =='') { alert('Please enter a summary sentence 
for this bug.'); return false; }
if (this.form.comment.value =='') { alert('Please enter a description for this 
bug.'); return false; }\">

Comment 4

[Matthew Tuck [:CodeMachine]]

Moving to new Bugzilla product ...

Comment 5

[Dave Miller [:justdave]]

.

Comment 6

[Dave Miller [:justdave]]

Reassigning all of my "future" targetted bugs to indicate that I'm not presently
working on them, and someone else could feel free to work on them. (sorry for
the spam if you got this twice, it didn't take right the first time)

Comment 7

[Jeffrey Bacon]

I think that this would be very good in combination with the custom fields
described in bug 91037.

Comment 8

[Frédéric Buclin]

Custom fields can be marked as mandatory. Hardcoded fields always have values, so mandatory doesn't mean anything.