If - flere ting skal være opfyldt

Udskift denne SQL-sætning:

  SQL = "select * from nyhedsmail where bruger = '" &bruger& "'"

med denne:

  SQL = "select * from nyhedsmail where bruger = '" &bruger& "' AND brugerpassword = '" & password & "'"

Det forudsætter naturligvis at feltet brugerpassword findes i tabellen nyhedsmail... :-)

tak softspot, det prøver jeg lige :o)

softspot> Det var den forkerte kode jeg har sat ind :o) - - - Vil du prøve at kigge på denne istedet?

Mit problem er at den sætter aktiv til 1, uanset hvilken kode det tastes i formularen.
Jeg vil gerne have det sådan, at den KUN skal opdatere hvis 1. brugeren(mobil) er i DB, og 2. hvis der er indtastet den korrekte kode(password). Kan du se hvad der er galt med koden?

pft.

<%
  Set Conn = Server.CreateObject("ADODB.Connection")
DSN = "DRIVER={Microsoft Access Driver (*.mdb)}; "
DSN = DSN & "DBQ=" & Server.MapPath ("/log/skosms.mdb")
Conn.Open DSN

  mobil = replace(Request.Form("mobil"),"'","")
  password = replace(Request.Form("password"),"'","")

  'tjek om bruger findes
SQL = "select * from nyhedsmail where mobil = '" &mobil& "' AND userpass = '" & password & "'"

  Set rs=Conn.Execute(SQL)
  if not rs.EOF then
    SQL = "update nyhedsmail set aktiv=1 where mobil = '" &mobil& "'"
    Conn.Execute(SQL)
    Response.Redirect("default1.asp?m=tak_tilmeld")
Else
Response.Redirect("default1.asp?m=forkert_kode")
  end if
 
  ' Luk databaseforbindelse
Conn.Close
Set Conn = Nothing

%>

Hvis det ikke er den rigtige kode der er indtastet skal brugeren sendes til findes_ikke.asp.

Umiddelbart ser det korrekt ud. Jeg går ud fra at din Response.Redirect("default.asp?m=forkert_kode") svarer til findes_ikke.asp, ellers skal du naturligvis omdirigere til findes_ikke.asp i stedet for default.asp i else-delen af din if-sætningen...

Dog vil jeg påpege at din forbindelse ikke bliver lukket da du omdirigerer (redirect) brugeren, inden conn.close kaldes (udførsel af kode på siden stoppes ved bla. Response.Redirect). Det kan få konsekvenser for tilgængeligheden af forbindelser til databasen, hvis der kommer lidt trafik på din server. Du bør altså kalde Conn.Close inden Response.Redirect (begge steder).

Hvis det stadig ikke fungerer, så tjek lige om felterne i din formular hedder mobil og password, altså noget i stil med:

<input type="text" name="mobil"...>
<input type="password" name="password"...>

det kan sagtens være sådan noget der driller. Du kan evt. prøve at udskrive SQL-sætningen inden du udfører den. Det afslører typisk årsagen til problemer med SQL der ikke opfører sig som det skal :-)

Altså noget i stil med:

SQL = "select * from nyhedsmail where mobil = '" &mobil& "' AND userpass = '" & password & "'"

Response.Write SQL
Response.End

Set rs=Conn.Execute(SQL)

Så kan du evt. se om feltere mobil og password indeholder det du forventer...

Takker - Det var en tastefejl i formular-siden. Jeg siger mange tak for hjælpen. Smider du er svar?

Velbekomme :)

Tak for point :)