2 mindre problemer med DOM

Hov, der er faktisk en fejl til, som falder inden for fejl nr. 2.

Funtkionen "updateDayName()", som står for at opdatere ugenavnet ud fra den valgte dato, opdaterer det faktisk ikke, hvis det første sæt af dropdown-boxe som blevet påvirket (onclick-eventen). Så det er endnu en fejl (ud over at den skriver "invalid argument" ved anden gennemløb).

Mon ikke, du bare mangler nogle form-tags - eller har sat dem forkert?

Prøv at poste hele koden til siden

Det med checkbox'en skyldes nok, den skal findes i siden, før det giver mening, at den checkes(?). Prøv at appende først:

    newTR.appendChild(newTD);
    newCheck.checked = true;

- og den anden fejl, kan jeg ikke forstå, du opfatter som en fejl  :)

Når du har kørt denne én gang:
  var replaced = para.replaceChild(newSpan,spanElm);

- eksisterer spanElm jo ikke næste gang, du forsøger at køre den - og skal derfor smide en fejl  :)

>Det med checkbox'en skyldes nok...
Rettet nu virker det. Takker :-)

>eksisterer spanElm jo ikke næste gang, du forsøger at køre den - og skal derfor smide en fejl  :)
Men kan du ikke hjælpe mig til at ændre min kode, så den ikke smider nogle fejl? Samt at den også kan opdatere det første sæt af dropdown-boxe (hvilket den ikke gør).

Replace-funktionen har jeg stjålet herfra: http://developer.apple.com/internet/webcontent/dom2i.html (Se under "Replacing nodes".)

Jeg har sådan set bare kopieret koden derfra, så det "burde" jo også virke hos mig (altså at den ikke kommer med en fejl ved anden gennemløb).

Replace-fejlen er faktisk større end før antaget. Det virker slet ikke i Firefox, hvor den skriver "undefined" som ugedage, når man ændre på dropdown-boxene, mens den i IE virker... somme tider op til 3. gennemløb... hvorefter den også fejler.

Her kommer hele koden:

<!--#INCLUDE virtual = "/forum/config.asp"-->
<!--#INCLUDE virtual = "/common/inc_dsn_open.asp" -->
<!--#INCLUDE virtual = "/forum/inc_header.asp" -->
<!--#include virtual = "/common/showAlert.asp"-->
<!--#INCLUDE virtual="/common/inc_checkLogin.asp" --><%
if strDBNTUserName = "" then
    response.redirect "index.asp"
end if
'************************************************
'DER SKAL VÆRE ET CHECK SÅ MAN IKKE KAN OPDATERE EN ANDENS ARRANGEMENT - KOMMER SENERE

if not request.querystring("ID") = "" then
    strSQL = "Select * FROM KALENDER_EVENTS WHERE ID = " & request.querystring("ID")
    Set RS = DataConn.Execute(strSQL)
   
    if NOT RS.EOF then   
        if rs("Author") = memberId or mlev = 4 then
            strCalenderID = rs("id")
            strCalenderEvent_Name = rs("Event_Name")
            strCalenderDate_Day = left(rs("Date"), 2)
            strCalenderDate_Month = right(left(rs("Date"), 5), 2)
            strCalenderDate_Year = right(left(rs("Date"), 10), 4)
            strCalenderTime_Hour = left(right(rs("Date"), 8), 2)
            strCalenderTime_Minute = left(right(rs("Date"), 5), 2)
            strCalenderCategory = rs("Category")
            strCalenderLocation = rs("Location")
            strCalenderDescription = rs("Description")
            strCalenderUrl = rs("Url")
        else
            'dataforbindelsen skal lukkes inden viderestilling
            DataConn.Close
            response.redirect "kalender_oversigt.asp"
        end if
    else
        'dataforbindelsen skal lukkes inden viderestilling
        DataConn.Close
        response.redirect("admin_oversigt.asp?message=Aktiviteten er tidligere blevet slettet og kan derfor ikke opdateres.")
    end if
end if

%>
<!--#include virtual = "/common/inc_dsn_open.asp"-->
<!--#INCLUDE virtual ="/common/inc_writeSubmenu.asp" -->
<!--#INCLUDE virtual ="/common/inc_writeTable.asp" -->
<script type="text/javascript">

var intSelects = 0;

function appendSelect(f){

    var isMonth = intSelects==0?false:f.elements["isMonth"][1].checked;
    var actDate = intSelects==0?new Date():new Date(f.elements["Date_Year"+intSelects].value, f.elements["Date_Month"+intSelects].selectedIndex, f.elements["Date_Day"+intSelects].selectedIndex + (isMonth?29:8));
     
    var thisDay = actDate.getDate();
    var thisMonth = actDate.getMonth();
    var thisYear = actDate.getFullYear();

    var thisDayName = getDayName(actDate);   
    var intSelectsTemp = null;
 
    intSelects++; 
   
    var dash = document.createTextNode(" ");
    var mother = document.getElementById('motherDiv');
 
    var newTabel = document.createElement("table");
    newTabel.style.width = "100%";
    newTabel.style.border = "0px";
    mother.appendChild(newTabel);

    var newTBody = document.createElement("tbody");
    newTabel.appendChild(newTBody);

    var newTR = document.createElement("tr");
    newTBody.appendChild(newTR);

    var newTD = document.createElement("td");
    newTD.style.width = "15px";
    newTD.style.align = "left";
     
    newCheck = document.createElement("input");
    newCheck.type = 'checkbox';
    newCheck.name = 'Cancel_date' + intSelects;
    newCheck.id = 'Cancel_date' + intSelects; 
    newCheck.value = intSelects; 
    newCheck.onclick = function () { doCancelEvent(this.form, this); }
    newTD.appendChild(newCheck);
 
    // ************** 1. CELLE AFSLUTNING ***********************
    newTR.appendChild(newTD);

    newCheck.checked = true;
   
    // ************** 2. CELLE START **************************** 
    newTD = document.createElement("td");
    newTD.style.width = "50px";

    newPara = document.createElement("p");
    newPara.id = 'para_day_name' + intSelects;
    newTD.appendChild(newPara);
   
      newSpan = document.createElement("span");
    newSpan.id = 'span_day_name' + intSelects;
    newPara.appendChild(newSpan);
   
    var spanTxt = document.createTextNode(thisDayName);
    newSpan.appendChild(spanTxt);

    // ************** 2. CELLE AFSLUTNING ***********************
    newTR.appendChild(newTD);

    // ************** 3. CELLE START **************************** 
    newTD = document.createElement("td");

    var newSel = document.createElement("select");
    newSel.className = 'inputbox';
    newSel.name = 'Date_Day' + intSelects;
    newSel.id = 'Date_Day' + intSelects;
    intSelectsTemp = intSelects;
    newSel.onclick = function () { updateDayName(this.form, intSelectsTemp); }
    newTD.appendChild(newSel);

    for(i=1;32>i;i++){
        newSel.options[newSel.length] = new Option(i, i);
    }

    newSel.selectedIndex = thisDay - 1;

    newTD.appendChild(dash);

    newSel = document.createElement("select");
    newSel.className = 'inputbox';
    newSel.name = 'Date_Month' + intSelects;
    newSel.id = 'Date_Month' + intSelects;
    intSelectsTemp = intSelects;
    newSel.onclick = function () { updateDayName(this.form, intSelectsTemp); }

    newTD.appendChild(newSel);
     
    var mName = ['','januar','februar','marts','april','maj','juni','juli','august','september','oktober','november','december'];
    for(i=1;13>i;i++){
        newSel.options[newSel.length] = new Option(mName[i],i);
    }

    newSel.selectedIndex = thisMonth;

    dash = dash.cloneNode(true);
    newTD.appendChild(dash);

    newSel = document.createElement("select");
    newSel.className = 'inputbox';
    newSel.name = 'Date_Year' + intSelects;
    newSel.id = 'Date_Year' + intSelects;
    intSelectsTemp = intSelects;
    newSel.onclick = function () { updateDayName(this.form, intSelectsTemp); }
    newTD.appendChild(newSel);

    //3 år frem i tiden
    for(i=0;3>i;i++){
        newSel.options[newSel.length] = new Option(thisYear + i, thisYear + i);
    }

    newSel.selectedIndex = 0;
 
    // ************** 3. CELLE AFSLUTNING ***********************
    newTR.appendChild(newTD);
}

function updateDayName(form, strSelected){

    alert(form);
    alert(strSelected);
    var newSpan = document.createElement("span");
    strNewDateName = getDayName(form.elements["Date_Day"+strSelected].value + "/" + form.elements["Date_Month"+strSelected].value + "-" + form.elements["Date_Year"+strSelected].value);
    var newText = document.createTextNode(strNewDateName);
    newSpan.appendChild(newText);

    var para = document.getElementById("para_day_name" + strSelected);
    var spanElm = document.getElementById("span_day_name" + strSelected);
    var replaced = para.replaceChild(newSpan,spanElm);
}

function getDayName(dato) {
    dato = new Date(dato);
    strDays = new Array("Søndag","Mandag","Tirsdag","Onsdag","Torsdag","Fredag","Lørdag");
    return strDays[dato.getDay()];
}

function doCancelEvent(form, element) {

    form.elements["Date_Day"+element.value].disabled = element.checked;
    form.elements["Date_Month"+element.value].disabled = element.checked;
    form.elements["Date_Year"+element.value].disabled = element.checked;   
}

function submitForm() {

    form = document.frmUser;
   
    form.Description.value = form.Description.value.replace( /'/g, "\264");
    form.Description.value = form.Description.value.replace( /"/g, "\42");
   
    if (form.Event_Name.value == "") {
        alert("Du mangler at vælge en titel til arrangementet");
        return false;
    }
   
    if (form.Event_Name.value.length > 0 && form.Event_Name.value == document.frmUser.Event_Name.value.toUpperCase()){
        alert("Det er ikke tilladt at skrive titlen kun med store bogstaver");
        return false;
    }
   
    indlaeg = form.Event_Name.value;
    txt = indlaeg;
    indlaeg = indlaeg.replace(/([\wæøå]{14})/gi,"\$1\-");
    indlaeg = indlaeg.replace(/--/gi,"-");
    if(txt!=indlaeg){
        form.Event_Name.value = indlaeg;
          alert('Du har i titlen indtastet et langt ord uden at anvende en bindestreg - og det vil skabe ravage på kalenderens oversigt.\n\nDerfor er der blevet indsat en bindestreg i det lange ord. Denne bindestreg flytter du blot derhen, hvor ordet naturligt kan deles.');
          return false;
      }
   
    if (form.Date_Day.value == "Blank") {
        alert("Du mangler at vælge en dato (dag) til arrangementet");
        return false;
    }
   
    if (form.Date_Month.value == "Blank") {
        alert("Du mangler at vælge en dato (måned) til arrangementet");
        return false;
    }
   
    if (form.Date_Year.value == "Blank") {
        alert("Du mangler at vælge en dato (år) til arrangementet");
        return false;
    }
   
    if (isDate(form.Date_Day.value, form.Date_Month.value, form.Date_Year.value) == false) {
        return false;
    }
   
    if (form.Time_Hour.value == "Blank") {
        alert("Du mangler at vælge et klokkeslæt (time) til arrangementet");
        return false;
    }
   
    if (form.Time_Minute.value == "Blank") {
        alert("Du mangler at vælge et klokkeslæt (minut) til arrangementet");
        return false;
    } 
   
    if (isTime(form.Time_Hour.value, form.Time_Minute.value) == false) {
        return false;
    }
   
    if (form.Category.value == "") {
        alert("Du mangler at vælge en kategori til arrangementet");
        return false;
    }
   
    if (form.Location.value == "") {
        alert("Du mangler at vælge et sted til arrangementet");
        return false;
    }
       
    form.onsubmit(); // workaround browser bugs.
    form.submit();
}

function daysInFebruary(ar){
    dage = ((ar % 4 == 0)&&((!(ar % 100 == 0))||(ar % 400 == 0))) ? 29 : 28;
    return (dage);
}
   
function isDate(dd,mm,aaaa){

    dd = parseInt(dd,10);
    mm = parseInt(mm,10);
    aaaa = parseInt(aaaa);
   
    daysInMonth = new Array(12);
    daysInMonth[1] = 31;
    daysInMonth[2] = daysInFebruary(aaaa);
    daysInMonth[3] = 31;
    daysInMonth[4] = 30;
    daysInMonth[5] = 31;
    daysInMonth[6] = 30;
    daysInMonth[7] = 31;
    daysInMonth[8] = 31;
    daysInMonth[9] = 30;
    daysInMonth[10] = 31;
    daysInMonth[11] = 30;
    daysInMonth[12] = 31;
   
    if(dd > daysInMonth[mm]||mm > 12){
        alert("Du har valgt en ugyldig dato");
        return false;
    }
       
    return true;
    }

function isTime(tt, mm){

    if (tt > 23 && mm > 0) {
        alert("Du har valgt et ugyldig tidspunkt");
        return false;
    }
   
    return true;
}

</script>
<%
WriteSubMenu "index.asp", "Kalender", "Opret/opdater&nbsp;aktivitet"
WriteTableStart "Opret/opdater en aktivitet", strHeadCellColor, "100%", "/grafik/layout/wavyb.gif"

%> <center>
<form action="!admin_aktivitet2.asp" name="frmUser" method="POST">
    <p><font size="7" color="#FF0000">P.t. ude af drift</font></p>
    <table width="80%" border="0" cellpadding="3" cellspacing="0" bordercolor="#111111">
        <tbody>
        <tr>
            <td valign="baseline" align="left"><b>
            <font face="<%= strDefaultFontFace%>" size="<%=strDefaultFontSize %>" color="<%=strForumFontColor %>">
            Titel</font> </b></td>
            <td align="left" valign="top">
            <input type="text" name="Event_Name" maxlength="50" class="inputbox" size="44" />
            </td>
        </tr>
        <tr>
            <td valign="baseline" align="left"><b>
            <font face="<%= strDefaultFontFace%>" size="<%=strDefaultFontSize %>" color="<%=strForumFontColor %>">
            Dato</font> </b></td>
            <td align="left" valign="top">
            <div id="motherDiv">
            </div>
            <table border="0" width="100%" id="table1" cellspacing="0" cellpadding="0">
                <tbody>
                <tr>
                    <td width="28%">
                    <input type="radio" value="false" name="isMonth" checked="" />gentag
                    en uge senere<br />
                    <input type="radio" value="true" name="isMonth" />gentag fire
                    uger senere</td>
                    <td width="72%">
                    <input onclick="appendSelect(this.form)" type="button" value=" Ok " /></td>
                </tr>
            </tbody>
            </table>
            <br />
            </td>
        </tr>
        <tr>
            <td valign="baseline" align="left"><b>
            <font face="<%=strDefaultFontFace%>" size="<%=strDefaultFontSize %>" color="<%=strForumFontColor%>">
            Tidspunkt</font> </b></td>
            <td align="left" valign="top">
            <select name="Time_Hour" class="inputbox">
            <option selected="" value="Blank"></option>
            <%
          hourCounter = 1
          Do WHILE hourCounter <= 23
                if hourCounter > 9 then
                    %>
            <option value="<%=hourCounter%>"><%=hourCounter%></option>
            <%
                else
                    %>
            <option value="0<%=hourCounter%>">0<%=hourCounter%></option>
            <%
                end if
                hourCounter = hourCounter + 1
          Loop
          %></select> <select name="Time_Minute">
            <option selected="" value="Blank"></option>
            <%
          minuteCounter = 0
          Do WHILE minuteCounter <= 55
                if minuteCounter < 9 then
                    %>
            <option value="0<%=minuteCounter%>">0<%=minuteCounter%></option>
            <%
                else
                    %>
            <option value="<%=minuteCounter%>"><%=minuteCounter%></option>
            <%
                end if
                minuteCounter = minuteCounter + 5
          Loop
          %></select> </td>
        </tr>
        <tr>
            <td valign="baseline" align="left"><b>
            <font face="<%= strDefaultFontFace%>" size="<%=strDefaultFontSize %>" color="<%=strForumFontColor %>">
            Kategori</font> </b></td>
            <td align="left" valign="top"><select name="Category">
            <option value="" selected=""></option>
            <%
          strQuery="SELECT category FROM KALENDER_CATEGORIES"
          Set rs=dataConn.Execute(strQuery)

          Do While Not rs.eof
                %>
            <option value="<%=rs("Category")%>"><%=rs("Category")%></option>
            <%
                  rs.movenext
          Loop
          set RS = nothing
          %></select> </td>
        </tr>
        <tr>
            <td valign="baseline" align="left"><b>
            <font face="<%= strDefaultFontFace%>" size="<%=strDefaultFontSize %>" color="<%=strForumFontColor %>">
            Sted</font> </b></td>
            <td align="left" valign="top"><select name="Location">
            <option value="" selected=""></option>
            <%
            strQuery="SELECT Location FROM KALENDER_LOCATIONS"
            Set rs=dataConn.Execute(strQuery)
   
            Do While Not rs.eof
                %>
            <option value="<%=rs("Location")%>"><%=rs("Location")%></option>
            <%
                rs.movenext
            Loop
            set RS = nothing
            %></select> </td>
        </tr>
        <tr>
            <td valign="baseline" align="left"><b>
            <font face="<%= strDefaultFontFace%>" size="<%=strDefaultFontSize %>" color="<%=strForumFontColor %>">
            Link</font> </b></td>
            <td align="left" valign="top">
            <input type="text" name="Url" size="44" /> </td>
        </tr>
        <tr>
            <td valign="top" align="left"><b>
            <font face="<%=strDefaultFontFace%>" size="<%=strDefaultFontSize %>" color="<%=strForumFontColor %>">
            Beskrivelse</font> </b></td>
            <td align="left" valign="top">
            <table border="0" cellpadding="0" cellspacing="0" width="100%">
                <tbody>
                <tr>
                    <td width="100%">
                    <textarea id="HTMLArea" name="Description" rows="28" cols="75" style="border-style: outset; border-width: 1"><%=strCalenderDescription%></textarea>
                    </td>
                </tr>
                <tr>
                    <td align="left" valign="bottom" colspan="2">
                    <font face="Verdana" size="1">
                    </font><p align="center"><font align="center" face="Verdana" size="1">&nbsp;<br />
                    <input type="button" onclick="window.location.href='index.asp'" value="Annuller" size="20" />
                    <input type="button" onclick="submitForm();" value="    Ok    " /></font>
                    </p>
                    </td>
                </tr>
            </tbody>
            </table>
            </td>
        </tr>
    </tbody>
    </table>
    <input type="hidden" name="hiddenID" value="<%=strCalenderID%>" />
</form>
</center>
<script type="text/javascript">

if ("<%=strCalenderID%>" != "") {

    form = document.frmUser;
    form.Event_Name.value = "<%=strCalenderEvent_name%>";
    form.Date_Day.value = "<%=strCalenderDate_Day%>";
    form.Date_Month.value = "<%=strCalenderDate_month%>";
    form.Date_Year.value = "<%=strCalenderDate_Year%>";
    form.Time_Hour.value = "<%=strCalenderTime_Hour%>";
    form.Time_Minute.value = "<%=strCalenderTime_Minute%>";
    form.Category.value = "<%=strCalenderCategory%>";
    form.Location.value = "<%=strCalenderLocation%>";
    form.Url.value = "<%=strCalenderUrl%>";

}

appendSelect();

</script>
<%
WriteTableEnd
WriteFooter
%><!--#INCLUDE virtual = "/common/inc_dsn_close.asp" -->

Kunne du hjælpe mig?

Nå løste det selv ved at indsætte name og id, så jeg kan kalde span'et igen ved næste opdatering:

function updateDayName(form, strSelected){

    var newSpan = document.createElement("span");
    strNewDateName = getDayName(form.elements["Date_Day"+strSelected].value + "/" + form.elements["Date_Month"+strSelected].value + "-" + form.elements["Date_Year"+strSelected].value);
    var newText = document.createTextNode(strNewDateName);
    newSpan.name = "span_day_name" + strSelected;
    newSpan.id = "span_day_name" + strSelected;
    newSpan.appendChild(newText);

    var para = document.getElementById("para_day_name" + strSelected);
    var spanElm = document.getElementById("span_day_name" + strSelected);
    var replaced = para.replaceChild(newSpan,spanElm);
}

Lukker og slukker - vil du have nogle points, så tryk svar :-)

Lukketid - tryk svar :-)

...

Sorry ... jeg har vist ikke fået mails om dette - og nej, her ønskede jeg absolut ikke points  :)

Takker, alligevel for din fine rådgivning, som jeg sagtens sadig kan huske. Jeg er nu i fuld gang med DOM :-)

sadig = stadig

Selvtak ... det lyder godt. DOM er en herlig ting  :)

Det er det, men det er bare så problematisk, når IE og FF tolker forskelligt, rettelse Bill og co. forsøger at lave sine egne standarder, som de tvinger ned over resten af verden.