Solved

Two hidden input with the same name within different forms on the same page bugs !

Posted on 2008-06-23
6
458 Views
Last Modified: 2008-07-02
Hi experts,

I have a webpage which there is two generated forms on it where both have a field named notype.

When i try to get the value of this hidden input from javascript, I got undefined. Both forms have different names and ids.

If you want to try it, go to :
http://www.cints.net/PHP/affpage.php
Connect yourselft using the two textboxes at the top with :
Username (firstbox) : TE1
Password (secondbox) : t

From there, you can click on this link :
http://www.cints.net/PHP/affpage.php?file=searchemploye

The lastest button called "Rechercher" is the second form on the page with the Sondage form.
So, if you click on this button "Rechercher", you will get a first alert, then a second which tells you the value of the notype input.

If I remove the first form (Sondage form), then the problem is fixed. I'd like to be able to keep the same name, which I think can usually be achieve due to the existance of the function getElementsByName (Elements, the s is key). So, could someone tells me where I'm wrong.

This testing has been done with IE7.
0
Comment
Question by:djon2003
  • 4
  • 2
6 Comments
 
LVL 1

Assisted Solution

by:mudskipperw
mudskipperw earned 20 total points
Comment Utility
give your forms with various names and use each notype is under one form which will be unique to any others. then you can have such as
alert(document.FormName1.notype.value);
alert(document.FormName2.notype.value);

more accurate than using getElementByName.
0
 
LVL 9

Author Comment

by:djon2003
Comment Utility
The forms already have two differents names and ids : they are "sondageform" & "searchform".

The button "Rechercher" is calling this javascript function :
function searchUnite(ObjID) {
   objUnitesRequest = new Request();
   SaveUniteFromForm(ObjID,true);

   alert(curUnite.jsonData.toJSONString());
   objUnitesRequest.httpRequest("POST", "UnitesManager.php", false , uniteSearched, "page=searchunite&searchingUnite=" + encodeURIComponent(curUnite.jsonData.toJSONString()));
}

The bug is happening in the other function called SaveUniteFromForm. I'll give you only the beginning because this is something quiete big, but the problem happens at the beginning anyway.

So as you can see in the code above & below, the button calls a javascript function with his button Id as parameter. Then this param is passed to the function SaveUniteFromForm and this function create a new var called Obj which is the HTML element represented by this ID. (The button).

More, as said in the first post, If I remove the "Sondage" form (the other one) than my problem is solved. (just a reminder)
function SaveUniteFromForm(ObjID,ForSearch) {

   if (ForSearch == undefined) ForSearch = false;

    var Obj = document.getElementById(ObjID);

 

    curUnite.jsonData = new Object();

    alert(1);

    alert(Obj.form.notype.value);

    curUnite.jsonData.NoUnite = Obj.form.nounite.value;

    curUnite.jsonData.NoType = Obj.form.notype.value;

    alert(curUnite.jsonData);
 

..... not useful for the problem.

Open in new window

0
 
LVL 9

Author Comment

by:djon2003
Comment Utility
Excuse me for this useless question, but I discovered that I faulty not correctly closed the sondage form tag.
0
How to improve team productivity

Quip adds documents, spreadsheets, and tasklists to your Slack experience
- Elevate ideas to Quip docs
- Share Quip docs in Slack
- Get notified of changes to your docs
- Available on iOS/Android/Desktop/Web
- Online/Offline

 
LVL 9

Accepted Solution

by:
djon2003 earned 0 total points
Comment Utility
Only 20 points because what mudskipperw gave me was already stated in the problem has done.
0
 
LVL 1

Expert Comment

by:mudskipperw
Comment Utility
alright, 20 points then.

this msg is not an objection, but when i looked at your page and the source code, you didn't give any form any names.
in your SaveUniteFromForm, ObjID would have to be document, to be able to make the following line
    alert(Obj.form.notype.value);

still when you didn't name a form as "form", that line is going to fail the program. alternatively, you can call forms as an array
document.forms

--- "points" is not what a program is after but getting the program working. hehehe agree?

//your function...
function SaveUniteFromForm(ObjID,ForSearch) {
   if (ForSearch == undefined) ForSearch = false;
    var Obj = document.getElementById(ObjID);
 
    curUnite.jsonData = new Object();
    alert(1);
    alert(Obj.form.notype.value);
0
 
LVL 9

Author Comment

by:djon2003
Comment Utility
Two things :
1- Up to now, I always gave all the points, but now the problem was not located there and I mostly sure that the forms were named. Anyhow, if you have the chance to answer another question I'll have in the future, for you'll certainly have all the points.

2- Obj is the button and using Obj.form is accessing the form of this button, so no name is required there. It is a special property of the inputs of a form.
0

Featured Post

6 Surprising Benefits of Threat Intelligence

All sorts of threat intelligence is available on the web. Intelligence you can learn from, and use to anticipate and prepare for future attacks.

Join & Write a Comment

Showing your events from Google Calendar in Google Maps Why? I travel all week and I thought it would be ideal if staff in office knew where I was based on my calendar. (OK real reason: my son wanted to see where I would be working, and I thoug…
Have you tried to learn about Unicode, UTF-8, and multibyte text encoding and all the articles are just too "academic" or too technical? This article aims to make the whole topic easy for just about anyone to understand.
In this tutorial viewers will learn how to style a corner ribbon overlay for an image using CSS Create a new class by typing ".Ribbon":  Define the class' "display:" as "inline-block": Define its "position:" as "relative": Define its "overflow:" as …
In this tutorial viewers will learn how to embed videos in a webpage using HTML5. Ensure your DOCTYPE declaration is set to HTML5: "<!DOCTYPE html>": Use the <video> tag to insert a video. Define the src as the URL of your video; this is similar to …

771 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

11 Experts available now in Live!

Get 1:1 Help Now