Editing the Saved Addresses shows red fields

VERIFIED FIXED in Firefox 63

Status

()

defect
P1
normal
VERIFIED FIXED
7 months ago
6 months ago

People

(Reporter: david.olah, Assigned: MattN)

Tracking

({regression})

Trunk
mozilla64
Unspecified
All
Points:
---

Firefox Tracking Flags

(firefox-esr60 unaffected, firefox62 unaffected, firefox63 verified, firefox64 verified)

Details

(Whiteboard: [webpayments])

Attachments

(2 attachments)

(Reporter)

Description

7 months ago
Posted image 2018-10-03_1624.png
[Affected versions]: 
Nightly 64.0a1 (2018-10-02), Firefox 63.0b11

[Affected platforms]:
Platforms: Windows 10, Mac OS X, Ubuntu 16.04.

[Preconditions]:

Modify the following prefs via about:config:

- extensions.formautofill.available: on
- extensions.formautofill.creditCards.available: true

[Steps to reproduce]:
1. Go to about:preferences#privacy and search for Forms & Passwords
2. Check the Autofill addresses and click the Saved Addresses button
3. Add a new address with a name
4. Select the same address and hit the Edit button

[Expected result]:
- The uncompleted fields are not highlighted with red border.

[Actual result]:
- The uncompleted fields are highlighted with red border (if you change the name, the rest of the fields remain red but the form can be saved.)
The change at https://hg.mozilla.org/mozilla-central/diff/4b668c00c226/browser/extensions/formautofill/content/manageDialog.js was just wrong. The 4th argument is for a closing callback.
Blocks: 1427961
Keywords: regression
Priority: -- → P2
Whiteboard: [webpayments]
Flags: qe-verify+
QA Contact: hani.yacoub
Assignee: nobody → MattN+bmo
Status: NEW → ASSIGNED
Priority: P2 → P1

Comment 3

6 months ago
Pushed by mozilla@noorenberghe.ca:
https://hg.mozilla.org/integration/autoland/rev/d71434f40755
Properly pass noValidate to EditAddress and set @novalidate. r=jaws

Comment 4

6 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d71434f40755
Status: ASSIGNED → RESOLVED
Last Resolved: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Comment on attachment 9017400 [details]
Bug 1496069 - Properly pass noValidate to EditAddress and set @novalidate. r?jaws

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1427961

User impact if declined: Users won't be able to save corrections to saved addresses if they are missing some "required" fields

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: No

Needs manual test from QE?: Yes

If yes, steps to reproduce: Ensure that editing a record with some missing fields doesn't show the red outlines and that the save button is enabled.

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Fairly straightforward change to pass an additional dialog argument and fix setting the attribute

String changes made/needed: None
Attachment #9017400 - Flags: approval-mozilla-beta?
Comment on attachment 9017400 [details]
Bug 1496069 - Properly pass noValidate to EditAddress and set @novalidate. r?jaws

Resetting the uplift request flag for approval‑mozilla‑release as we are now in the release candidates phase
Matt, are the flag changes needed to reproduce the issue in comment #0 a supported scenario on our release channel or would one hit this bug only if he changed values in about:config?

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: None

User impact if declined: 

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: Yes

If yes, steps to reproduce: 

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): 

String changes made/needed:
Attachment #9017400 - Flags: approval-mozilla-beta? → approval-mozilla-release?

Comment 7

6 months ago
Verified as fixed on Firefox Nightly 64.0a1 on Windows 10 x 64, Windows 7 x32, Mac OS X 10.13 and on Ubuntu 16.04 x64.
Status: RESOLVED → VERIFIED
(In reply to Pascal Chevrel:pascalc from comment #6)
> Matt, are the flag changes needed to reproduce the issue in comment #0 a
> supported scenario on our release channel or would one hit this bug only if
> he changed values in about:config?

Neither of those prefs need to be changed to hit this bug in en-US builds for users in the US where Address Autofill is enabled by default.
Comment on attachment 9017400 [details]
Bug 1496069 - Properly pass noValidate to EditAddress and set @novalidate. r?jaws

Taking for RC2 as this is a visible UI regression for a feature affecting our top locale and it was verified as fixed on Nightly.
Attachment #9017400 - Flags: approval-mozilla-release? → approval-mozilla-release+
eslint and autofill tests passed locally.
Flags: needinfo?(MattN+bmo)

Comment 14

6 months ago
Build ID: 20181018182531
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:63.0) Gecko/20100101 Firefox/63.0

Verified as fixed on Firefox 63.0 RC build 2 on Windows 10 x 64, Windows 7 x32, Mac OS X 10.13 and Ubuntu 16.04 x64.
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.