Closed Bug 122128 Opened 23 years ago Closed 23 years ago

Page Info | Links shows wrong address

Categories

(SeaMonkey :: Page Info, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: aha, Assigned: db48x)

References

()

Details

Repro:
1. go to URL - http://www.stavocentrum.cz/
2. open Page Info
3. tab Links
4. find 2nd row with 'Vyhledat'

Actual value :
[object HTMLInputElement]

Expected value:
/index.php?none=1

2002012608/WinNT4
Summary: [RFE] Page Info | Links shows wrong address → Page Info | Links shows wrong address
fun. I'll take it, look at it in detail in a few hours
Assignee: blaker → db48x
Blocks: 82059
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows NT → All
Hardware: PC → All
Component: XP Apps: GUI Features → Page Info
QA Contact: sairuh → pmac
This is an odd problem. Adam, does the form actually work correctly in mozilla?
I'm assigning this bug to the parser component to see if they can shed more
light on the issue.

The page info code uses the dom to look up the action of the form, so there
shouldn't be any way for it to get mixed up. The page validates perfectly.
Component: Page Info → Parser
 <FORM ACTION="/index.php?none=1" METHOD="GET">
 <INPUT TYPE="HIDDEN" NAME="action" VALUE="plnytext">

So form.action is the DOM0 way to access the input element...  and you get what
you get.  Life sucks.  Maybe if form.action is an Object we can fall back to
form.getAttribute("action") ?  Or just start off with form.getAttribute (and add
a comment saying why)?
Component: Parser → Page Info
Daniel: Form works fine, page is validated by W3C without error.
Cc'ing jst, but I doubt this will be fixed on the DOM side.
The DOM behavior here is absolutely correct.  We should just be doing what I say
in comment 3 -- if form.action is an Object, then fall back to getAttribute().
I slipped the fix for this into the patch for bug 69295
fixed
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
v=aha (2002040503/Win2K)
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.