#1814 closed enhancement (fixed)
Use only one entry point for activity launch
Reported by: | alsroot | Owned by: | tomeu |
---|---|---|---|
Priority: | Unspecified by Maintainer | Milestone: | |
Component: | Sugar | Version: | Git as of bugdate |
Severity: | Unspecified | Keywords: | r+ |
Cc: | Distribution/OS: | Unspecified | |
Bug Status: | Unconfirmed |
Description
For now, code that launch/resume activities is scattered. Having only one entry point makes launching related code more robust.
Attachments (2)
Change History (13)
comment:1 follow-up: ↓ 2 Changed 14 years ago by alsroot
- Keywords r? added
Changed 14 years ago by alsroot
comment:2 in reply to: ↑ 1 Changed 14 years ago by alsroot
comment:3 Changed 13 years ago by tomeu
- Keywords r+ added; r? removed
This patch is great, please push if you have already tested each code path that has been made to use misc.launch().
comment:4 Changed 13 years ago by alsroot
- Resolution set to fixed
- Status changed from new to closed
Rechecked all use cases with recent sugar commits.
comment:5 Changed 13 years ago by alsroot
- Keywords r? added; r+ removed
- Resolution fixed deleted
- Status changed from closed to reopened
Changed 13 years ago by alsroot
comment:6 follow-up: ↓ 7 Changed 13 years ago by alsroot
Regression that was introduced by implementation patch.
comment:7 in reply to: ↑ 6 Changed 13 years ago by alsroot
Replying to alsroot:
Regression that was introduced by implementation patch.
At least in one case activity_id could be , from misc.resume().
comment:8 Changed 13 years ago by tomeu
- Keywords r+ added; r? removed
r+, but change the patch to test for both is None and empty string.
comment:9 follow-up: ↓ 10 Changed 13 years ago by alsroot
- Resolution set to fixed
- Status changed from reopened to closed
comment:10 in reply to: ↑ 9 Changed 13 years ago by alsroot
Replying to alsroot:
|TestCase|
- resume image from usb in Imageviewer activity
- there are two new buttons in activities tray
Note: See
TracTickets for help on using
tickets.
Patch contains also #1719 fix because it heavily depends of this ticket fix.