Closed
Bug 340006
Opened 18 years ago
Closed 18 years ago
general.startup.* prefs don't work for mailnews items in suiterunner
Categories
(SeaMonkey :: General, defect)
SeaMonkey
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: standard8, Assigned: neil)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
standard8
:
review+
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
Bug 335550 implemented a default commandline handler for suiterunner. Part of this was to handle the general.startup.* prefs. However some of these work:
general.startup.chat
general.startup.browser
general.startup.editor
but most of them won't:
general.startup.addressbook
general.startup.mail
general.startup.calendar
general.startup.news
general.startup.compose
The problem is that the contract id to try to startup these items is defined as:
@mozilla.org/commandlinehandler/general-startup;1?type=
plus the end of the pref name. For xpfe SeaMonkey this is fine, as the mailnews items use the old commandlinehander and define the contract ids as above. For the new suiterunner, this doesn't work, the contact ids appear to now be as follows:
general.startup.addressbook - @mozilla.org/addressbook;1
general.startup.mail - @mozilla.org/appshell/component/messenger;1
general.startup.news - @mozilla.org/messenger/nntpservice;1
general.startup.compose - @mozilla.org/messengercompose;1
What a nice mixture ;-)
I couldn't quite work out a relevant one for calendar - that's if we still want to support it as it currently stands.
Assignee | ||
Comment 1•18 years ago
|
||
Reporter | ||
Comment 2•18 years ago
|
||
(In reply to comment #1)
> Created an attachment (id=224194) [edit]
> Compiles, but untested as yet
>
This doesn't seem to work, at least for address book anyway.
Assignee | ||
Comment 3•18 years ago
|
||
Note: this doesn't work with the general.startup.mail preference yet but that's unrelated to this patch as the -mail command line argument doesn't work either.
Assignee: general → neil
Attachment #224194 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #224265 -
Flags: review?(bugzilla)
Reporter | ||
Comment 4•18 years ago
|
||
Comment on attachment 224265 [details] [diff] [review]
Proposed patch
Looks good to me.
Attachment #224265 -
Flags: review?(bugzilla) → review+
Assignee | ||
Comment 5•18 years ago
|
||
Comment on attachment 224265 [details] [diff] [review]
Proposed patch
Asking David for sr as I hear Scott is on holiday(?)
Attachment #224265 -
Flags: superreview?(bienvenu)
Updated•18 years ago
|
Attachment #224265 -
Flags: superreview?(bienvenu) → superreview+
Assignee | ||
Comment 6•18 years ago
|
||
Fix checked in.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•