Deleted duplicate home screens can be selected

VERIFIED FIXED

Status

P3
normal
VERIFIED FIXED
3 years ago
3 years ago

People

(Reporter: twen, Assigned: gmarty)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(blocking-b2g:2.5+)

Details

(Whiteboard: [systemsfe])

Attachments

(1 attachment)

(Reporter)

Description

3 years ago
When there are two same home screens available, delete one of them will delete both, but a radio button remains. Selecting the radio button will create a blank home screen. 

[Repro Steps]:
1. Install a home screen twice from webIDE
2. Go to Settings -> Home Screens -> Change Home Screen
3. Tap the installed home screen name for details
4. Tap delete button
5. Select the empty radio button

[Actual Result]: 
Radio button available for selection

[Expected Result]:
No radio button 

[Build Info]:
Build ID               20151023005002
Gaia Revision          29ce8ec8606e59f582375234440812b046346513
Gaia Date              2015-10-22 05:31:38
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/1f03a14106e59280761ac53904340f389674337f
Gecko Version          44.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151023.001128
Firmware Date          Fri Oct 23 00:11:35 UTC 2015
Bootloader             s1
(Reporter)

Comment 1

3 years ago
Nominate for blocking. Broken function.
Blocks: 1180666
blocking-b2g: --- → 2.5?
QA Whiteboard: [COM=New Homescreen]
See Also: → bug 1217730

Comment 2

3 years ago
This bug is very clearly, going on the STR, manifesting in Settings, not Homescreen.
Component: Gaia::Homescreen → Gaia::Settings
Whiteboard: [systemsfe]
blocking-b2g: 2.5? → 2.5+
gmarty, can you take a look?
Flags: needinfo?(gmarty)
(Assignee)

Updated

3 years ago
Assignee: nobody → gmarty
Flags: needinfo?(gmarty)
Created attachment 8678961 [details] [review]
[gaia] gmarty:Bug-1217734-Deleted-duplicate-home-screens-can-be-selected > mozilla-b2g:master
(Assignee)

Comment 5

3 years ago
Comment on attachment 8678961 [details] [review]
[gaia] gmarty:Bug-1217734-Deleted-duplicate-home-screens-can-be-selected > mozilla-b2g:master

This patch makes sure that all apps with the same manifest URL are removed on uninstall. Fred, can you take a look?
Attachment #8678961 - Flags: review?(gasolin)
Priority: -- → P3

Comment 6

3 years ago
Comment on attachment 8678961 [details] [review]
[gaia] gmarty:Bug-1217734-Deleted-duplicate-home-screens-can-be-selected > mozilla-b2g:master

Thanks for the patch. As my test result the install related code seems need some tweaking as well.

I create an empty homescreen from WebIDE `previleged empty app`, add "role": "homescreen" in manifest.webapp, then:

1. go to Settings > Home Screens > Change Home Screen
2. Install a home screen twice from webIDE
there will be 2 dup entries on list

3. kill settings then re-enter the Change Home Screen panel
the dup entries are gone and leave correct single entry


And please add some test on https://github.com/mozilla-b2g/gaia/blob/master/apps/settings/test/unit/modules/apps_cache_test.js to make sure others wont break it next time
Attachment #8678961 - Flags: review?(gasolin)
(Assignee)

Comment 7

3 years ago
Comment on attachment 8678961 [details] [review]
[gaia] gmarty:Bug-1217734-Deleted-duplicate-home-screens-can-be-selected > mozilla-b2g:master

Thanks for your feedback Fred. I removed the old comment and added a test for that fix. Can you review it again?
Attachment #8678961 - Flags: review?(gasolin)

Comment 8

3 years ago
Comment on attachment 8678961 [details] [review]
[gaia] gmarty:Bug-1217734-Deleted-duplicate-home-screens-can-be-selected > mozilla-b2g:master

The install related issue is addressed in bug 1217730, so this patch looks good to me. Thanks!
Attachment #8678961 - Flags: review?(gasolin) → review+
(Assignee)

Comment 9

3 years ago
Landed in master in https://github.com/mozilla-b2g/gaia/commit/a498f875bb2fd248f1a46f3dfdd7695aadcd232f
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Reporter)

Comment 10

3 years ago
Verified fixed. No duplicate home screen can be installed or deleted. 

Build ID               20151101012023
Gaia Revision          91cac94948094cfdcd00cba5c6483e27e80cb3b0
Gaia Date              2015-10-28 20:32:15
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/96377bdbcdf3e444a22aeaa677da696243b00d98
Gecko Version          45.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20151101.004008
Firmware Date          Sun Nov  1 00:40:17 UTC 2015
Bootloader             s1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.