Closed Bug 120282 Opened 23 years ago Closed 22 years ago

javascript error in metadata.js (Link Properties doesn't work)

Categories

(SeaMonkey :: UI Design, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
mozilla1.0

People

(Reporter: bugzilla, Assigned: sicking)

References

Details

Attachments

(1 file, 1 obsolete file)

Right click on a link and select properties. Almost nothing is shown in the
dialog. Could be due to this error in the console:

Error: abbr has no properties
Source File: chrome://navigator/content/metadata.js
Line: 462

build 20020115
-> sicking@bigfoot.com
Assignee: trudelle → sicking
*** Bug 109479 has been marked as a duplicate of this bug. ***
Target Milestone: --- → mozilla1.0
*** Bug 125764 has been marked as a duplicate of this bug. ***
Attached patch Fixes the problem (obsolete) — Splinter Review
The problem is that any page that contains a <link> tag and doesn't contain a
lang="" will screw up.	The patch that I've attached causes the
convertLanguageCode function to automatically return if it recieves a language
of "".

Another way of fixing this would be to add a check at the source, but I figured
it would be best to catch all cases as opposed to just this one.  The "" string
is being sent from the checkForLink function.  Right now it doesn't check what
it's sending before it sends it.
Keywords: patch
Attachment #72661 - Attachment is obsolete: true
Attachment #72663 - Flags: review+
Comment on attachment 72663 [details] [diff] [review]
Moved it up a line.

sr=alecf
Attachment #72663 - Flags: superreview+
Comment on attachment 72663 [details] [diff] [review]
Moved it up a line.

a=asa (on behalf of drivers) for checkin to the 1.0 trunk
Attachment #72663 - Flags: approval+
The was checked into the trunk yesterday so it should be fixed.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
*** Bug 130039 has been marked as a duplicate of this bug. ***
vrfy'd fixed using 2002.03.13 comm bits on linux rh7.2, win2k and mac 10.1.3.
Status: RESOLVED → VERIFIED
Product: Core → Mozilla Application Suite
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: