pludselig fejl i script

I tilfælde af SQL-fejl: Prøv at udskrive din sql før at du forsøger at udføre den. På den måde får man meget lettere en ide om hvad der eventuelt kunne være galt:

sql1 = "insert into traad(navn, gruppe) values('" & strNavn & "',"&request.form("gruppenr")&")"

Response.Write "DEBUG: " & sql1 & "<br>"

Myconn.execute(sql1)
sqlID = "select @@identity as newID from traad"
set myID = Myconn.execute(sqlID)

hmmm... ja jeg bliv ikke klogere :-)
men det her var hvad den skrev

DEBUG: insert into traad(navn, gruppe) values('test',)

Microsoft JET Database Engine error '80040e14'

Syntax error in INSERT INTO statement.

/strnewtraad.asp, line 16

Der skal vel stå en værdi på plads nr. 2 i din values...

Og sagt på en anden måde: request.form("gruppenr") er tom

Det er forhåbentlig bare noget kode du leger med for din egen fornøjelses skyld, og ikke noget du lægger synligt på nettet. Der er direkte åbent for en række standard sikkerhedshuller. Læs og lær om det inden andre kan bruge din kode, ik'os'?

Mit problem blev ikke løst, men lukker tråden idet jeg ikke fatter en bjælde :-)
tak for hjælpen ser om jeg få det løst

Er du ikke lidt hurtig om at give op? Hvad er det du ikke fatter en bjælde af?

koden er en jeg har fået lavet idet jeg ikke er den store asp haj, den har virket so far. og så virker den lige pludselig ikke mere, det er så her jeg står af :-)

Det sted hvor koden fejler er i virkeligheden på den foregående side. På den har du en formular, og i denne formular har du endvidere et input-felt som hedder "gruppenr".

Eller rettere sagt det har der været tidligere ... nu er der åbenbart sket et eller andet med feltet sådan at det ikke bliver sendt med når form'en submittes.

hmm..okay det lyder ikke ret godt gad vide om jeg har været inde og pille ved noget jeg ikke skulle havde gjort..jeg har jo leget lidt med skidtet. Men nu må jeg hellere lade være med at smide en hel masse kode herind kan jeg så forstå

Du er velkommen til at smide koden fra den foregående side ind - eller til at gove os et link.

her er koden


<!--#include file="top.inc"-->
      <td class="td_info">
      <div class="div_info">
      <!--#include file="info_main.inc"-->
      <!--#include file="login.inc"-->
     
      </div>
      </td>
     
      <td class="td_body">
      <center>
    <%
Set data = Server.CreateObject ("ADODB.Recordset")
data.Open "SELECT * From gruppe Where id ="& Request.QueryString("gruppenr") &"", MyConn

%>
<div class="font" style="width: 550px;"

function validering(f){
  if(f.titel.value==''){     
    alert('Du skal skrive en overskrift');
    f.navn.focus();
    return false;     
  }
  if(f.besked.value.length < 20){
    alert("Dit indlæg skal være på mindst 20 tegn")
    f.besked.focus();
    return false;
  }
  return true; 
  }
</script>

      <h3>Lav ny tråd i <% = data("navn")%></h3><br/>
      <a href="forum.asp">Tilbage</a>

    <center>
    <table width="55%">
    <tr>
      <td valign="top">
     

<form name="form" action="strnewtraad.asp"gruppenr=<% = data("id") %>" method="post" onsubmit="return validering(this);">
<table class="tbl_cont">
<tr>
  <td>Overskrift</td><td><input type="text" name="navn" size="45"></td>
</tr>
<tr>
  <td>Tekst</td><td><textarea name="besked" rows="7" cols="40"></textarea></td>
</tr>
<tr>

  <td><input type="submit" value="send"></td><td>&nbsp;</td>
</tr>
<script language="JavaScript" src="preload.js"></script>

<script language=javascript type=text/javascript>

function gemtekst(Teksten)

{

if(Teksten.createTextRange)

Teksten.caretPos = document.selection.createRange().duplicate();

}

<tr>
</table>
<input type="hidden" name="gruppenr" value="<% = data("id") %>"/>
<input type="hidden" name="ip" value="<% request.servervariables("REMOTE_ADDR") %>"/>

</form>
  </td>
</tr>
</table>
</center>



        </div>
        </center>
        </td>
<!--#include file="bottom.inc"-->

Du ska ændre:

<form name="form" action="strnewtraad.asp"gruppenr=<% = data("id") %>" method="post" onsubmit="return validering(this);">

til:

<form name="form" action="strnewtraad.asp?gruppenr=<% = data("id") %>" method="post" onsubmit="return validering(this);">

PS: Det er i øvrigt en rigtig dårlig ide at give sin form navnet "form".

Eller endnu bedre - flytte den ind i selve formen:

<form name="form" action="strnewtraad.asp" method="post" onsubmit="return validering(this);">
<input type="hidden" name="gruppenr" value="<% = data("id") %>">
... og så resten af din form.

Hej

Jamen mange tak skal du havde..Nu virker det igen. perfekt

Hvordan giver jeg point ? og lukker tråden

Det kan du ikke gøre før nogen har lagt et svar.

... så det får du her :^)

Så mangler du bare at acceptere mit svar :^)

det er hermed gjort

Nej... du skal markere mit nick i den lille boks i bunden før at du kan acceptere et svar. :^)