Closed Bug 143486 Opened 22 years ago Closed 22 years ago

enter_bug.cgi: Using ?format=simple doesn't work

Categories

(Bugzilla :: Creating/Changing Bugs, defect)

2.15
defect
Not set
major

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: burnus, Assigned: burnus)

Details

(Keywords: regression)

Attachments

(1 file)

It is now possible to use more than one format for enter_bug.cgi (bug 138903).
But here it doesn't work.
I traced it down to
  my $format = ValidateOutputFormat($::FORM{'format'}, "create");
is called.
This sets $scriptdir to "create". It works with "bug/create"

Patch to come.
Target Milestone: --- → Bugzilla 2.16
Keywords: patch, review
Comment on attachment 83053 [details] [diff] [review]
Patch enter_bug.cgi to make it work with format

This looks like a side-effect of moving the template directory.  good catch.

But we don't need "create" in the list at all anymore.	They are all in the
bug/create directory.  let's remove "create" from the list.
Attachment #83053 - Flags: review-
Keywords: regression
thought I reassigned there, guess I missed...
Assignee: myk → burnus
> It is now possible to use more than one format for enter_bug.cgi

Well, actually it's not, because the "simple" (Bugzilla Helper replacement)
template sitting on my hard drive hasn't been checked in yet :-) I hope to have
it up for review this weekend.

Gerv
> But we don't need "create" in the list at all anymore. They are all in the
> bug/create directory.  let's remove "create" from the list.

I'm not sure how I have to understead it.

-my $format = ValidateOutputFormat($::FORM{'format'}, "create");
+my $format = ValidateOutputFormat($::FORM{'format'}, "create", "bug/create");

the second parameter is the $script
the third  parameter is the $subdir

and in global.pl:
  $subdir = $script if !defined($subdir);

I don't see another solution as done in attachment 83053 [details] [diff] [review].

> > It is now possible to use more than one format for enter_bug.cgi
> Well, actually it's not.
I ment my own create-simple.html.tmpl ;-)
OK, maybe I misunderstood the parameters on that code...   Gerv?
Comment on attachment 83053 [details] [diff] [review]
Patch enter_bug.cgi to make it work with format

This is the right fix. At least, it makes my create-simple.html.tmpl work :-)

Gerv
Attachment #83053 - Flags: review-
Attachment #83053 - Flags: review+
Fixed on trunk and branch.

Checking in enter_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/enter_bug.cgi,v  <--  enter_bug.cgi
new revision: 1.67; previous revision: 1.66
done

Checking in enter_bug.cgi;
/cvsroot/mozilla/webtools/bugzilla/enter_bug.cgi,v  <--  enter_bug.cgi
new revision: 1.66.2.1; previous revision: 1.66
done

Gerv
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: