Solved

User login form not functioning properly

Posted on 2011-02-19
5
448 Views
Last Modified: 2012-05-11
Hi Experts,

I am attempting to create a user login form in access '07.  The code that I have so far is not erroring out, however, it is not doing what I want it to do.  It is a relatively simple login form.  The user opens the db and a login form is loaded.  The user will select their username from the combo box and then focus is placed on the password field.  The user enters in an assigned pwd given to them by the administrator.  When the user clicks on login, the code should compare the pwd string in the textbox to the adminassigned pwd on the users table.  The code is not doing that and it is not erroring out.  The 'Users' table has the following columns: username, password, assignedpassword and confirm password.  Can someone take a look at my code and tell me what I am missing?  Thanks!!
Option Compare Database
Const NumLoginAttempts = 3
Dim Attempts As Integer
Private Sub Cbousername_AfterUpdate()
'If a username is selected, then place the focus on the password field.
If Not IsNull(Me.Cbousername) Then
    Me.txtPwd.SetFocus
    Me.CmdLogin.Enabled = True
End If
    
End Sub
Private Sub CmdLogin_Click()

'Check to make sure that there are characters in the txtpwd field
'If not, display message telling user that a password must be entered and place focus back on the password box
If IsNull(Me.txtPwd) Then
    MsgBox "You must enter a password.", vbExclamation + vbOnly, "Required Entry"
    Me.txtPwd.SetFocus
End If
'Validate password for user
If Me.txtPwd.Value = DLookup("Password", "Users", "[ID]=" & Me.Cbousername.Value) Then
    ID = Me.Cbousername.Value
'Close the login form and open frmMain
    DoCmd.Close acForm, "FrmLogin", acSaveNo
    DoCmd.OpenForm "frmMain"
End If

If Me.txtPwd.Value <> DLookup("Password", "Users", "[ID]=" & Me.Cbousername.Value) Then
    MsgBox "The password you entered does not match the username selected.", vbExclamation + vbOnly, "Try Again"
    Me.txtPwd = ""
    Me.txtPwd.SetFocus
    Attempts = Attempts + 1
    'After 3 tries, the db will close
        If Attempts > 3 Then
            MsgBox "Maximum number of attempts reached. Please contact administrator to reset your password.", vbExclamation + vbOKOnly, "Database Shutdown"
            DoCmd.CloseDatabase
            Exit Sub
Else
'Compare txtpwd field value to the assignedpassword value for the user that is logging in
'If the password that the user types in matches the assignedpassword in tblusers, then have the user create a new password
If Me.txtPwd.Value = DLookup("AssignedPassword", "Users", "[ID]=" & Me.Cbousername & " Or [Password] Is null") Then
    MsgBox "You must enter a new password.", vbExclamation + vbOKOnly, "Invalid Password"
    Me.LblPwd.Visible = False
    Me.txtPwd.Visible = False
    Me.LblNewPwd.Visible = True
    Me.TxtNewPwd.Visible = True
    Me.TxtNewPwd.SetFocus

End If
End If
End If
End Sub

Open in new window

0
Comment
Question by:daintysally
5 Comments
 
LVL 21
ID: 34935999
Try:

Option Compare Database
Const NumLoginAttempts = 3
Dim Attempts As Integer
Private Sub Cbousername_AfterUpdate()
'If a username is selected, then place the focus on the password field.
If Not IsNull(Me.Cbousername) Then
    Me.txtPwd.SetFocus
    Me.CmdLogin.Enabled = True
End If
    
End Sub
Private Sub CmdLogin_Click()

'Check to make sure that there are characters in the txtpwd field
'If not, display message telling user that a password must be entered and place focus back on the password box
If IsNull(Me.txtPwd) Then
    MsgBox "You must enter a password.", vbExclamation + vbOnly, "Required Entry"
    Me.txtPwd.SetFocus
End If
'Validate password for user
If Me.txtPwd = DLookup("Password", "Users", "[ID]=""" & Me.Cbousername & """") Then
    ID = Me.Cbousername
'Close the login form and open frmMain
    DoCmd.Close acForm, "FrmLogin", acSaveNo
    DoCmd.OpenForm "frmMain"
End If

If Me.txtPwd <> DLookup("Password", "Users", "[ID]=""" & Me.Cbousername & """") Then
    MsgBox "The password you entered does not match the username selected.", vbExclamation + vbOnly, "Try Again"
    Me.txtPwd = ""
    Me.txtPwd.SetFocus
    Attempts = Attempts + 1
    'After 3 tries, the db will close
        If Attempts > 3 Then
            MsgBox "Maximum number of attempts reached. Please contact administrator to reset your password.", vbExclamation + vbOKOnly, "Database Shutdown"
            DoCmd.CloseDatabase
            Exit Sub
Else
'Compare txtpwd field value to the assignedpassword value for the user that is logging in
'If the password that the user types in matches the assignedpassword in tblusers, then have the user create a new password
If Me.txtPwd = DLookup("AssignedPassword", "Users", "[ID]=""" & Me.Cbousername & """) Or DLookup("Password", "Users", "[ID]=""" & Me.Cbousername & """") = "" Then
    MsgBox "You must enter a new password.", vbExclamation + vbOKOnly, "Invalid Password"
    Me.LblPwd.Visible = False
    Me.txtPwd.Visible = False
    Me.LblNewPwd.Visible = True
    Me.TxtNewPwd.Visible = True
    Me.TxtNewPwd.SetFocus

End If
End If
End If
End Sub

Open in new window

0
 
LVL 6

Expert Comment

by:Gugro
ID: 34936877
> "The code is not doing that"
What exactly is the code not doing?
- will the sub cmdLogin_Click not get called ?
- will the code not ask you for a password, when you entered none ?
- will the code not open "frmMain", when you gave the correct password ?
- will the code not warn you if you gave a wrong password ?

The best way to find the culprit: set a breakpoint on the start of Sub  cmdLogin_Click and step through your code ;-)

PS:
I would guess that "frmMain" will not open,  as you  are closing this form first, so it will never reach the second line.
    DoCmd.Close acForm, "FrmLogin", acSaveNo
    DoCmd.OpenForm "frmMain"
0
 

Author Comment

by:daintysally
ID: 34937480
the code does not work as it should at this point:

'Compare txtpwd field value to the assignedpassword value for the user that is logging in
'If the password that the user types in matches the assignedpassword in tblusers, then have the user create a new password
If Me.txtPwd = DLookup("AssignedPassword", "Users", "[ID]=""" & Me.Cbousername & """) Or DLookup("Password", "Users", "[ID]=""" & Me.Cbousername & """") = "" Then
    MsgBox "You must enter a new password.", vbExclamation + vbOKOnly, "Invalid Password"
    Me.LblPwd.Visible = False
    Me.txtPwd.Visible = False
    Me.LblNewPwd.Visible = True
    Me.TxtNewPwd.Visible = True
    Me.TxtNewPwd.SetFocus
0
 

Author Comment

by:daintysally
ID: 34937493
when I enter in the password that is equivalent to the assigned password, the code should prompt me to enter a new password and it does not.  Everything else works as it should.
0
 
LVL 26

Accepted Solution

by:
Nick67 earned 500 total points
ID: 35023376
Looking at the later part of your code, you have an OR statement in there.
You also have a lot of nested ifs and double quotes.
All of those are a bugger to debug.
Dlookup isn't the world's happiest thing either.

First, does you users table have a numeric UserID as a primary key?
If it does, and it should, get your combo box to return 2 columns, UserID and Users
Format the combo box column widths for 0, 1"--or whatever width the single column has now.
Now you've done away with all the nasty doublequotes
I use Chr(34) & SomeStringVariable & Chr(34) instead of doublequotes when I do have to use them, becuase they are so ugly to debug.

Now, the logic
Option Compare Database
Option Explicit
Const NumLoginAttempts = 3
Dim Attempts As Integer
Private Sub Cbousername_AfterUpdate()
'If a username is selected, then place the focus on the password field.
If Not IsNull(Me.Cbousername) Then
    Me.txtPwd.SetFocus
    Me.CmdLogin.Enabled = True
End If
    
End Sub

Private Sub CmdLogin_Click()
dim db as database
dim rs as recordset

'Check to make sure that there are characters in the txtpwd field
'If not, display message telling user that a password must be entered and place focus back on the password box
If IsNull(Me.txtPwd) Then
    MsgBox "You must enter a password.", vbExclamation + vbOnly, "Required Entry"
    Me.txtPwd.SetFocus
End If

'Validate password for user
set db = currebtdb
set rs = db.openrecordset("select * from Users where UserID = " & me.Cbousername & ";", dbopendynaset, dbseechanges)

'this works REALLY well to replace a lot of nested ifs
'only the first true statement gets processed
Select case true
    case Me.txtPwd = rs!Password
	'I assume ID is some global variable not listed in your code
        ID = Me.Cbousername
        'Close the login form and open frmMain
         DoCmd.Close acForm, "FrmLogin", acSaveNo
         DoCmd.OpenForm "frmMain"
    case Me.txtPwd <> rs!Password and Me.txtPwd <> rs!AssignedPassword
        MsgBox "The password you entered does not match the username selected.", vbExclamation + vbOnly, "Try Again"
        Me.txtPwd = ""
        Me.txtPwd.SetFocus
        Attempts = Attempts + 1
        'After 3 tries, the db will close
        If Attempts > 3 Then
            MsgBox "Maximum number of attempts reached. Please contact administrator to reset your password.", vbExclamation + vbOKOnly, "Database Shutdown"
            DoCmd.CloseDatabase ' bail the db entirely
            Exit Sub
        else
            exit sub 'just bail the sub on attempts 1 and 2
	end if
    case Me.txtPwd <> rs!Password and Me.txtPwd = rs!AssignedPassword
    'if the password doesn't match, but the assigned password does, then have the user create a new password
        MsgBox "You must enter a new password.", vbExclamation + vbOKOnly, "Invalid Password"
        Me.LblPwd.Visible = False
        Me.txtPwd.Visible = False
        Me.LblNewPwd.Visible = True
        Me.TxtNewPwd.Visible = True
        Me.TxtNewPwd.SetFocus
end select

rs.close
db.close

End Sub

Open in new window

0

Featured Post

Live: Real-Time Solutions, Start Here

Receive instant 1:1 support from technology experts, using our real-time conversation and whiteboard interface. Your first 5 minutes are always free.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

In a multiple monitor setup, if you don't want to use AutoCenter to position your popup forms, you have a problem: where will they appear?  Sometimes you may have an additional problem: where the devil did they go?  If you last had a popup form open…
Over the years I have built up my own little library of code snippets that I refer to when programming or writing a script.  Many of these have come from the web or adaptations from snippets I find on the Web.  Periodically I add to them when I come…
Polish reports in Access so they look terrific. Take yourself to another level. Equations, Back Color, Alternate Back Color. Write easy VBA Code. Tighten space to use less pages. Launch report from a menu, considering criteria only when it is filled…
With Secure Portal Encryption, the recipient is sent a link to their email address directing them to the email laundry delivery page. From there, the recipient will be required to enter a user name and password to enter the page. Once the recipient …

776 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