Felder mit JavaScript füllen.

[Neoclin]

Hallo zusammen,

was ist daran falsch ???

function layex(lname)
        {
          if(document.layform.all[lname].value == "off")
          {
             action
          }
        }

aufruf über layex('a')
das Formular ist mit name="layform" richtig benannt.

Danke im Voraus.

MfG

PS: hier nochmal das ganze Script ausführlich:

<script language="JavaScript">
        <!--
        function layex(lname)
        {
          if(document.layform.all[lname].value == "off")
          {
            document.all[lname].style.display = "block";
            document.layform.all[lname].value = "on";
            alert(lname);
          }
          else
          {
            document.all[lname].style.display = "none";
            document.layform.all[lname].value = "off";
          }
        }
        //-->
        </script>

<form name="layform">
        <input type="text" name="winamp2lay" value="off">
        <input type="hidden" name="winamp3lay" value="off">
        <input type="hidden" name="mp3lay" value="off">
        <input type="hidden" name="midilay" value="off">
        <input type="hidden" name="wallpaperlay" value="off">
        <input type="hidden" name="screensaverlay" value="off">
        <input type="hidden" name="promotionlay" value="off">
        <input type="hidden" name="andereslay" value="off">
        </form>

<a href="JavaScript:layex('winamp2lay');">TipTap</a>

winamp2lay ist Aufruf vonmehreren.

[wahsaga]

hi,

was ist daran falsch ???

if(document.layform.all[lname].value == "off")

all[] ist MS-syntax.

du solltest lieber die schreibweise von http://de.selfhtml.org/javascript/objekte/elements.htm#allgemeines verwenden - für # kannst du auch den namen des feldes einsetzen.

gruß,
wahsaga