817901 - Wi-Fi hotspot (Open) cannot be turned on with empty password

Status: RESOLVED FIXED

(Firefox OS Graveyard :: General, defect, P1)

Description

[Greg Grisco]

1. With Wi-Fi hotspot turned off, choose the Open security type and delete existing password.
2. Enable Wi-Fi hotspot

Actual Result:  Wi-Fi hotspot cannot be enabled, the slider moves back to the Off position.

Comment 1

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Did you have a data connection established at the time?  If not, this is bug 815924.

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

QA can you help to check if this is a duplicate?

Comment 3

[Greg Grisco]

In my test, Data connection was enabled and Wi-Fi was disabled.

Comment 4

[Dylan Oliver [:doliver]]

Evelyn, can you take a look at this one?

Comment 5

[Dylan Oliver [:doliver]]

Moving into C2 due to 802677 blocker.

Comment 6

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Evelyn and I have concluded that this is a back-end Gecko issue. We do nothing special on Gaia here. Settings app do nothing special here besides mutate the values in Settings database.

Chunk, ping Evelyn if she is wrong :)

Comment 7

[Vincent Chang[:vchang][changyihsin]]

Attachment: Patch v1.0

Don't check the validation of password when security type of wifi hotspot is set to "open"

Comment 8

[Greg Grisco]

I verified the fix works for me, thanks.

Comment 9

[Evelyn Hung]

I think we need to grey-out/hide password field when security level sets to open.
Arthur, could you please file a issue and link the number here? Thanks.

Comment 10

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Is this ready to land?

Comment 11

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

Your checkin-needed said yes :).

https://hg.mozilla.org/integration/mozilla-inbound/rev/3e3b0aa05474

Comment 12

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

https://hg.mozilla.org/releases/mozilla-aurora/rev/f6ddbb034d4a
https://hg.mozilla.org/releases/mozilla-beta/rev/55b63c25510a

Comment 13

[Arthur Chen [:arthurcc][inactive after 6/30]]

Bug 818829 has been created for tracking the UI work.

Comment 14

[Dietrich Ayala (:dietrich)]

Is there a m-c changeset URL? Can this be closed now?

Comment 15

[Chris Jones [:cjones] inactive; ni?/f?/r? if you need me]

NB: marking RESO FIXED when these bugs hit beta because it's getting hard for b2g folks to track remaining work for upcoming milestones.

Comment 16

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/3e3b0aa05474