Closed Bug 129574 Opened 22 years ago Closed 16 years ago

Autosubscribe & remove expired message links open in new window

Categories

(SeaMonkey :: MailNews: Message Display, defect)

defect
Not set
major

Tracking

(Not tracked)

VERIFIED WORKSFORME

People

(Reporter: stephend, Unassigned)

References

(Blocks 2 open bugs)

Details

(Whiteboard: see comment 42 and 43 for why this bug was resolved)

Build ID: 2002-03-07, All OSs.

Summary: Autosubscribe URLs should use/focus the currently open mail window.

Type of URLS this covers:

[Expired Articles]:
news://nospam@news/netscape.public.mozilla.mail-news?list-ids

[Autosubscribe]
news://news.mozilla.org/netscape.public.mozilla.mail-news
news:netscape.public.mozilla.mail-news

Testcase URL:

http://www.mozilla.org/quality/mailnews/tests/sea-mn-newsgroup-function.html#URI

Steps to Reproduce:

1.  Have a mail window open already.  In browser, type:
    news://news.mozilla.org/netscape.public.mozilla.mail-news [OR]
    news://netscape.public.mozilla.mail-news (should spawn Account Wizard)

Expected Results:

Mail window gets focused, account setup begins (or, confirmation if you used the
complete URL with the server info).

Actual Results:

A new mail window is spawned, and the correct events occur.  But we shouldn't be
spawing a new window.  This is 4xp.
Keywords: 4xp
QA Contact: esther → stephend
Summary: Autosubscribe URLs should use/focus the currently open mail window. → Autosubscribe/expired message URLs should use/focus the currently open mail window.
*** Bug 72820 has been marked as a duplicate of this bug. ***
Something seem to have changed - now list-ids URL goes to existing browser window.
Sorry, forgot - BuildID 2002032109, RedHat Linux 7.2
*** Bug 133563 has been marked as a duplicate of this bug. ***
*** Bug 134133 has been marked as a duplicate of this bug. ***
I just spent the last few hours investigating this, and I have come to the
conclusion that we - in the case where a 3pane is already open - need to pass
the arguments in OpenMessengerWindowWithUri() into the 3pane *and* focus it at
the same time.

Unfortunately, this is impossible with the existing APIs -- either you open a
new window with arguments, or you just focus an existing one.  Bienvenu, am I
correct in my understanding that the default-folder selection things depend on
us passing these args to OpenWindow()?
Hakan, if you can get a msgWindow, you can call msgWindow->SelectFolder() to
select the folder, without opening a new window. However, you need to make sure
that the msg window is a 3-pane window, which requires a bit of work. 
*** Bug 139167 has been marked as a duplicate of this bug. ***
*** Bug 145351 has been marked as a duplicate of this bug. ***
Summary: Autosubscribe/expired message URLs should use/focus the currently open mail window. → Autosubscribe/expired message links open in new window
*** Bug 156032 has been marked as a duplicate of this bug. ***
*** Bug 159147 has been marked as a duplicate of this bug. ***
*** Bug 162594 has been marked as a duplicate of this bug. ***
*** Bug 164811 has been marked as a duplicate of this bug. ***
Mozilla 1.1b nightly build 
Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.1b) Gecko/20020901
If only in a new window, type  
news://news.mozilla.org/netscape.public.mozilla.mail-news
then mozilla will alert for autoscribe, but then it does nothing but makes the
full usage of CPU(almost 100%). 
 Isn't bug 80972 functionally the same as this?
*** Bug 167648 has been marked as a duplicate of this bug. ***
*** Bug 167857 has been marked as a duplicate of this bug. ***
*** Bug 170092 has been marked as a duplicate of this bug. ***
*** Bug 170290 has been marked as a duplicate of this bug. ***
*** Bug 171974 has been marked as a duplicate of this bug. ***
*** Bug 178030 has been marked as a duplicate of this bug. ***
*** Bug 184635 has been marked as a duplicate of this bug. ***
*** Bug 188731 has been marked as a duplicate of this bug. ***
Bug 47981 reportedly fixed a similar problem.
2003040808 win32, xp-pro.  Still happening.
*** Bug 202897 has been marked as a duplicate of this bug. ***
*** Bug 216229 has been marked as a duplicate of this bug. ***
Summary: Autosubscribe/expired message links open in new window → Autosubscribe & remove expired message links open in new window
Problem always there with Mozilla 1.6b
*** Bug 246594 has been marked as a duplicate of this bug. ***
Product: Browser → Seamonkey
*** Bug 254782 has been marked as a duplicate of this bug. ***
Assignee: sspitzer → mail
*** Bug 300658 has been marked as a duplicate of this bug. ***
QA Contact: stephend
I just noticed (in the latest Thunderbird2 builds) that clicking on "Click here to remove all expired articles" no longer opens a new window. I tested it in SeaMonkey 1.1.1, and the bug does not occur. Can anyone confirm, that "Click here to remove all expired articles" no longer opens a new window? I'm on WinXP.
Chris, there were some fixes in this area...
Yes, confirmed as WFM with
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.2) Gecko/20070221 SeaMonkey/1.1.1

Great! :-)
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → WORKSFORME
Please reopen


It does not work at all (Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9a3pre) Gecko/20070228 MultiZilla/1.8.3.0a Mnenhy/0.7.4.0 SeaMonkey/1.5a)

1.  news://news.mozilla.org/netscape.public.mozilla.mail-news

- mail-news window is not focused 
- news.mozilla.org does not appear in accounts

news:netscape.public.mozilla.mail-news

- mail-news is not focused
- a ghost account named news is created and does not appear in account list (my "default" news-server account is a local leafnode that do fetch news.mozilla.org. It should be able to accept.
(this problem has been observed in thunderbird too)
- account wizard is not launched
Sorry for spam

1.  news://news.mozilla.org/netscape.public.mozilla.mail-news

- mail-news window is not focused 
- news.mozilla.org *do* appear in accounts
the 2) is still a problem.
Funny, should this be fixed on branch but not on trunk?
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
I just tried Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a3pre) Gecko/20070312 SeaMonkey/1.5a, and clicking on "Click here to remove all expired articles" does not open a new window. Clicking on a news link in the browser does not bring up the mailnews window. This looks like two separate issues to me.
No new comments in a year.

Using
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9pre) Gecko/2008040201 SeaMonkey/2.0a1pre

1. news://news.mozilla.org/netscape.public.mozilla.mail-news

A popup opens: "Would you like to subscribe to <group-name> [ Cancel ] [[ OK ]]". The mailnews window doesn't come to the front.

After hitting Enter (selecting OK), the mailnews windows still doesn't come to the front. After Alt-tab (bringing it to the front) here's what I see: No new window is opened. The group in question is added under my "news.mozilla.org" server. No error, no new messages in it, the group is not selected.

Selecting the group shows: 24011 unread messages. Popup (download so-many or download all) comes up. I hit Cancel and unsubscribe since I think my test has gone far enough.

2. news:netscape.public.mozilla.mail-news

Same popup as in case 1.

Not only mailnews window doesn't come to the front, but _browser_ tab contents are replaced by XUL error page "Address not found" for URL "news://news:119/netscape.public.mozilla.mail-news" as shown in URL bar.

A new account has been created in MailNews with name "news", server name "news" port 119, etc. This server doesn't appear in the left pane of the 3-pane window. I have to "Remove Account" in the account settings dialog.

Case 1 is approx. OK but 2 is obviously in error. Compared to comment #36, case 1 seems better but case 2 seems worse to me.

Assignee is set to mail@seamonkey.bugs and QAC to empty. These look weird to me; I'm resetting them to defaults.
Status: REOPENED → NEW
Morphing summary to describe current erroneous behaviour
Summary: Autosubscribe & remove expired message links open in new window → "news" URI without server is not handled correctly
Restoring bug summary. Contrary to comment 0, this bug is not about behavior from typing a news URL in a browser urlbar. It's about what happens when one clicks a "remove expired articles" link in a news message pane for a message which is claimed to be expired. This when done does not remove the expired articles, but does open an additional mailnews window. To test requires one find a newsgroup message in which such a message appears.
Summary: "news" URI without server is not handled correctly → Autosubscribe & remove expired message links open in new window
Bug that change their meaning should be forbidden. Too late for this one (I cannot even track down with which comment the meaning of this one changed).

Exactly the behavior described by Felix in comment 42 was fixed on trunk a year ago. I just tried again and it is fixed. [Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9pre) Gecko/2008032901 SeaMonkey/2.0a1pre] I think I was mistaken when I reopened because the not-working part was about the URL problem while this bug was already morphed to the expired articled problem. So I'm resolving once again.

Please do not morph this again but open a new bug, if one on the problem of comment 0 doesn't exist yet (if one does exist it is probably a relative of bug 108948).
Status: NEW → RESOLVED
Closed: 17 years ago16 years ago
Resolution: --- → WORKSFORME
Whiteboard: see comment 42 and 43 for why this bug was resolved
(In reply to comment #42)
> Restoring bug summary. Contrary to comment 0, this bug is not about behavior
> from typing a news URL in a browser urlbar. It's about what happens when one
> clicks a "remove expired articles" link in a news message pane for a message
> which is claimed to be expired. This when done does not remove the expired
> articles, but does open an additional mailnews window. To test requires one
> find a newsgroup message in which such a message appears.
> 

I typed nothing in the URL bar, I clicked the link. Clicking a news link should never change the contents of the current browser tab. This link (number 2 in comment #40) did it, which is wrong.
In reply to comment #43
OK, I just reported bug 426868 about the incorrect behaviour described at step 2 of comment #40
No longer blocks: 106267
Status: RESOLVED → VERIFIED
Duplicate of this bug: 1860308
You need to log in before you can comment on or make changes to this bug.