1956751 - status reset to '---' for 'status-firefox138' on bug creation page if 'Status' field changed

Status: RESOLVED FIXED

(bugzilla.mozilla.org :: Bug Creation/Editing, defect, P3)

Description

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

On the bug creation page, the value of the 'status-firefox138' field is reset to '---' if the 'Status' field is changed.

Comment 1

[David Lawrence [:dkl]]

This occurs due to code change that happened when we redeveloped the enter bug page to be more like the show bug page and share a lot of the same code.

The offending lines are: https://github.com/mozilla-bteam/bmo/blob/master/extensions/TrackingFlags/template/en/default/bug/script.html.tmpl#L49-L77

I need to check first if the tracking flag has been changed before resetting back to '---' if you change the status after.

The workaround for now is to choose the status first, then select the status tracking flag.

Will take a look

Comment 2

[Kohei Yoshino [:kohei]]

Regressed from my previous work — I’ll volunteer to take care of it.

Comment 3

[David Lawrence [:dkl]]

(In reply to Kohei Yoshino [:kohei] from comment #2)

Regressed from my previous work — I’ll volunteer to take care of it.

Thanks Kohei!

Comment 4

[Kohei Yoshino [:kohei]]

Looking at this now. It’s not a regression as I can also see the behaviour on the legacy Enter Bug page. I just moved the relevant code to a new location.

These are the bugs and commits that implemented the code:

• Bug 972040 = 3e7f7dd
• Bug 1170179 = ee871d6

I think we can just remove the else clause.

Comment 5

[BMO Github Automation]

Attachment: [mozilla-bteam/bmo] Bug 1956751 - status reset to '---' for 'status-firefox138' on bug creation page if 'Status' field changed (#2503)

Comment 6

[BMO Github Automation]

Authored by https://github.com/kyoshino
https://github.com/mozilla-bteam/bmo/commit/3f990fc4c3c64080cf9882198c4c4163e402f927
[master] Bug 1956751 - status reset to '---' for 'status-firefox138' on bug creation page if 'Status' field changed