Closed
Bug 55659
Opened 25 years ago
Closed 18 years ago
about:licence
Categories
(Core :: Networking, defect, P3)
Core
Networking
Tracking
()
VERIFIED
WORKSFORME
People
(Reporter: BenB, Assigned: BenB)
References
()
Details
(Whiteboard: Fix in hand)
Attachments
(1 file)
44.90 KB,
patch
|
Details | Diff | Splinter Review |
Reproduce:
1. Help|About Mozilla
2. Click on "MPL"
Actual result:
Page fetched from www.mozilla.org displayed
Expected result:
MPL displayed, but no network hit.
Additional comments:
It might be a legal requirement to ship the licence, even with tarballs.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9
Assignee | ||
Comment 1•25 years ago
|
||
Fixed in my N6.0 tree
Assignee | ||
Comment 2•25 years ago
|
||
Getting schizophrenic: Moving all bugs for Beonex <http://www.beonex.com> to my
second Bugzilla identity <ben.bucksch@beonex.com>.
Assignee: mozilla → ben.bucksch
Status: ASSIGNED → NEW
Assignee | ||
Comment 3•24 years ago
|
||
I will attach a patch. It will include the fix for bug 55660. Developed on
Linux. IIRC, it compiled on Windows, not tested on Mac.
gagan, please r= or tell someone to do so.
Assignee | ||
Comment 4•24 years ago
|
||
Assignee | ||
Comment 5•24 years ago
|
||
export POSIXLY_CORRECT=1
cvs diff -uN netwerk/build/nsNetModule.cpp
netwerk/protocol/about/src/Makefile.in netwerk/protocol/about/src/makefile.win
netwerk/protocol/about/src/nsAboutWelcome.cpp
netwerk/protocol/about/src/nsAboutWelcome.h
netwerk/protocol/about/src/nsAboutLicense.cpp
netwerk/protocol/about/src/nsAboutLicense.h xpfe/communicator/jar.mn
xpfe/communicator/resources/locale/en-US/MANIFEST
xpfe/communicator/resources/locale/en-US/welcome.html xpfe/global/jar.mn
xpfe/global/resources/locale/en-US/license.html
xpfe/communicator/resources/locale/en-US/Makefile.in
xpfe/communicator/resources/locale/en-US/makefile.win
xpfe/global/resources/locale/en-US/makefile.win
xpfe/global/resources/locale/en-US/Makefile.in
xpfe/global/resources/locale/en-US/MANIFEST > 55659-1.diff
Ben: I don't think this should be done as separate files. All the about:foo that
only redirect to another URL should be grouped together. And in the NewChannel
we should check for the different cases and present the corresponding URL for
it.
Assignee | ||
Comment 7•24 years ago
|
||
gagan, I completely agree and thought of something similar. But this is another
bug (there are already 5 or so of these classes and I add only 2), which I don't
cannot/don't want to fix right now, and these bugs are fixed already and needed
for Beonex Comm..
I disagree. The 5/6 classes you refer to all do different things. The case in
point (and about:welcome) only deals with the ones that are "redirected" to
another location. And so these cases should just be extensions to the way
about:credits works. What is beonex comm...?
Assignee | ||
Comment 9•24 years ago
|
||
Ok, it's just 3, not 5, existing classes which just redirect:
- <about:>
- <about:credits>
- <about:mozilla>
Still...
> What is beonex comm...?
A product based on Mozilla. <http://www.beonex.com/communicator>
Comment 10•24 years ago
|
||
If anyone files a bug on making the about: redirects share more code, please
make that bug block bug 66607 (about:privacy).
Assignee | ||
Comment 11•24 years ago
|
||
Gagan, I don't see the big difference problem if we have 3 or 5 classes which
essentiall do that same. If you block this bug, all you achieve is that you
force me to keep local changes (-> hassles with patch and new files, conflicts
during merges etc.).
Comment 12•24 years ago
|
||
The difference is not just the additional classes which mean more memory space
but also the extra files added for no benefit at all. And I can assure you the
purpose of denying this checkin is not to make things difficult for you, but to
make it better for the browser. Submit a patch that incorporates my suggestions
of handling redirect cases in one class and then I'll review it.
Assignee | ||
Comment 13•24 years ago
|
||
Nobody reviewd so far :-(, so I have to move to Mozilla 0.9. Please review!
Assignee | ||
Comment 14•24 years ago
|
||
> extra files added for no benefit at all
They sure have benefit for Beonex. How much code is in Mozilla that serves only
Netscape?
> Submit a patch that incorporates my suggestions
I have no time for such archtectural enhancements, at least not in the near
future. Actually, I wouldn't even know how to do it. I tried something like that
for about:<developer> back in 1999, IIRC, but gave up.
> Nobody reviewd so far :-(, so I have to move to Mozilla 0.9. Please review!
Ignore that - mass comment.
Assignee | ||
Comment 15•24 years ago
|
||
I filed bug 68086 for the generic redirector class.
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.8 → mozilla0.9
Assignee | ||
Comment 16•24 years ago
|
||
Gagan, how about |#ifdef|ing the code? This would remove the footprint from the
standard distro and avoid patches and cvs conflicts on my side.
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.9 → mozilla0.9.1
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.9.1 → mozilla0.9.2
Assignee | ||
Updated•24 years ago
|
Assignee | ||
Updated•24 years ago
|
Target Milestone: mozilla0.9.3 → ---
Comment 18•23 years ago
|
||
bug 68086 is fixed, can this be done easily now ?
Comment 19•23 years ago
|
||
note, by now such a redirector does exist (nsAboutRedirector.cpp)
Comment 20•23 years ago
|
||
ben: can you clarify what the status is since bug 68086 is fixed?
If that works, shouldn't you just be able to hook up about: URLs to anything you
want?
Assignee | ||
Comment 21•23 years ago
|
||
This bug is fixed as far as Beonex Communicator is concerned, thanks in part to
gagan's work on bug 68086.
However, I think this is a good idea for Mozilla as well. It is arguable. It
leave it to the netlib module owners to decide, if this should be fixed (in
which case I can contribute the simple fix) or be WONTFIXed.
Comment 22•18 years ago
|
||
Isn't this fixed long ago? About:licence now results in a British English MPL, while about:license produces an American English MPL. Marking WFM.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → WORKSFORME
Assignee | ||
Comment 23•18 years ago
|
||
Are you kidding? There is no British English MPL. about:licence shows a license which spells "license" American. Nobody expected otherwise.
This bug was about shipping the license with the app, not spelling. I think that has been fixed long ago indeed, but can't verify right now.
Comment 24•18 years ago
|
||
V/wfm:
Seamonkey 1.1,2/macos.
Create new profile.
Go offline.
Go to about:license
OBSERVED:
viewed a license. Did not extensively check contents, but it looks like what was requested.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•