Error when sending mail using GUI

[grimmlock]

Good Morning,

For some reason, and I am sure it is something very simple, this code will not send mail. When I run this code using the Beta Run feature, I get this message...

The server rejected the senders address. What I think is going on is that the names in $hCombo2 are not being set as the $Username (which is used as the senders address). I am hoping that someone might be able to tell me where or what I am doing wrong.

The goal of this is to be able to select a user from the drop down list ($hCombo2) and use it as the username in username@domain.com.

expandcollapse popup

#NoTrayIcon
#include <File.au3>
#include <ButtonConstants.au3>
#include <EditConstants.au3>
#include <Constants.au3>
#include <GUIConstantsEx.au3>
#include <GUIListBox.au3>
#include <StaticConstants.au3>
#include <WindowsConstants.au3>
#include <Array.au3>
    Global $goMyRet[2]
    Global $goMyError = ObjEvent('AutoIt.Error', 'MyErrFunc')
;~  Global $MyManager = GetManager(GuiCtrlRead($hCombo1)


Local $sText1 = ''; Output text for message box.
Local $sText2 = ''; Output text for message box.
Local $sTest3 = ''; Output text for message box.
Local $combo1 = ''; Output text for combo box.
Const $sDomain = 'domain.com'
Const $sServer = 'mail' & '.' & $sDomain
Const $sSupportEmail = 'bob' & '@' & $sDomain
    ClipPut($sServer)

Opt('TrayOnEventMode',1)
Opt('TrayMenuMode',1)

TraySetOnEvent($TRAY_EVENT_PRIMARYUP,'SpecialEvent')
;TraySetState(2) ; hide --> not needed

GUICreate('IT HelpDesk App', 300, 620, Default, Default) ;, BitAND($WS_CAPTION, $WS_POPUP, $WS_SYSMENU), $WS_EX_CONTEXTHELP)
;~ GUISetStyle

    local $hShinkButton = GUICtrlCreateButton('&Hide', 115, 560, 80) ;*Commented Out; Would like to learn how to create a form and clear the data
    Local $hOkButton = GUICtrlCreateButton('&Send', 200, 560, 80)

    _AddHorzSep(10, 70, 280)

    GUICtrlCreateLabel('Helpdesk Request Form', 30, 20, 270, 50)
    GUICtrlSetFont(-1, 15, 800); bold

    GUICtrlCreateLabel('Request &Type:', 25, 273, 75)
    Local $hCombo = GUICtrlCreateCombo('', 100, 270, 160) ; create first item
    GUICtrlSetData(-1, 'Keyboard/Mouse|Monitor|Desktop', 'Keyboard/Mouse') ; add other item snd set a new default

    GUICtrlCreateLabel('&Subject:', 25, 303, 75)
    Local $hInputText = GUICtrlCreateInput('', 100, 300, 160)

    GUICtrlCreateLabel('&Details:', 25, 330, 75)
    Local $hInputText2 = GUICtrlCreateEdit('', 25, 345, 235, 200, 0)

Local $hCombo2 = GUICtrlCreateCombo('', 100, 100, 160)
GUICtrlSetData(-1, 'sam|bob', '')

Local $hInputText3 = GUICtrlCreateInput('', 100, 150, 160)

   Local $sUsername = GuictrlRead($hCombo2) ; User Name
  Local $sComputername = GuiCtrlRead($hInputText3) ; Computer Name

;~  $sText1 = 'Username = ' & $sUsername & @CRLF ; The & character concentrates (joins) two values together.
;~  $sText2 = 'Computer Name = ' & $sComputername & @CRLF ; The &= adds to the variable. The same as doing $var = $var & 'extra stuff'

GUICtrlCreateLabel($sText1, 25, 100, 75)
GUICtrlCreateLabel($sText2, 25, 273, 75)

    GUISetState()

    While 1
        Switch GUIGetMsg()
Case $GUI_EVENT_CLOSE
Exit

Case $GUI_EVENT_MINIMIZE
  GuiSetState(@SW_HIDE)
             TraySetState(1) ; show
             TraySetToolTip ('click here to restore')

            Case $hOkButton
                $msg = 'Subject: ' & GUICtrlRead($hInputText) & @CRLF & @CRLF & 'Details: ' & GUICtrlRead($hInputText2) & @CRLF & @CRLF & '' & $sText1 & $sText2 & _wmigetcompinfo()

                OKButton($sServer, $sUsername, $sUsername & '@' & $sDomain, $sSupportEmail, 'Issue: ' & GUICtrlRead($hCombo), $msg)

       EndSwitch
   WEnd


Func SpecialEvent()
   GUISetState(@SW_SHOW)
   TraySetState(2) ; hide
EndFunc


Func _AddHorzSep($iX, $iY, $iW)

    GUICtrlCreateLabel('', $iX, $iY, $iW, 1)
    GUICtrlSetBkColor(-1, 0x000000)


EndFunc   ;==>_AddHorzSep


Func _wmigetcompinfo($strComputer = 'localhost')
    ; Generated by AutoIt Scriptomatic November 30, 2012
    $wbemFlagReturnImmediately = 0x10
    $wbemFlagForwardOnly = 0x20
    $colItems = ''
    $sOutput = ''
    ;- $sOutput &= 'Computer: ' & $strComputer & @CRLF
    $sOutput &= ''
    $objWMIService = ObjGet('winmgmts:\\' & $strComputer & '\root\CIMV2')
    $colItems = $objWMIService.ExecQuery('SELECT * FROM Win32_ComputerSystem', 'WQL', _
            $wbemFlagReturnImmediately + $wbemFlagForwardOnly)


    If IsObj($colItems) Then
        For $objItem In $colItems
            $sOutput &= 'Computer Status: ' & $objItem.Status & @CRLF


        Next
    Else
        Return ('WMI not functional')
    EndIf

    $colItems = $objWMIService.ExecQuery('SELECT * FROM Win32_ComputerSystemProduct', 'WQL', _
            $wbemFlagReturnImmediately + $wbemFlagForwardOnly)


;-    If IsObj($colItems) Then
;-       For $objItem In $colItems
;-            MsgBox(0, '', 'SerialNumber' & $objItem.IdentifyingNumber)
;-            $sOutput &= 'SerialNumber: ' & $objItem.IdentifyingNumber & @CRLF
;-        Next

        Return ($sOutput)

;-    EndIf

EndFunc   ;==>_wmigetcompinfo

Func WMIDateStringToDate($dtmDate)

    Return (StringMid($dtmDate, 5, 2) & '/' & _
            StringMid($dtmDate, 7, 2) & '/' & StringLeft($dtmDate, 4) _
             & ' ' & StringMid($dtmDate, 9, 2) & ':' & StringMid($dtmDate, 11, 2) & ':' & StringMid($dtmDate, 13, 2))
EndFunc   ;==>WMIDateStringToDate

Func OKButton($s_SmtpServer, $s_FromName, $s_FromAddress, $s_ToAddress, $s_Subject, $s_Body)

    $rc = _INetSmtpMailCom($s_SmtpServer, $s_FromName, $s_FromAddress, $s_ToAddress, $s_Subject, $s_Body)
    ProgressOn('Progress', 'Submitting your ticket...', '0 percent')
    For $i = 0 To 100 Step 20
        Sleep(500)
        ProgressSet($i, $i & ' percent')
    Next
    ProgressSet(100, 'Done', 'Complete')
    Sleep(500)
    ProgressOff()
    If @error Then
        MsgBox(0, 'Error sending message', 'Error code:' & @error & ' Description:' & $rc)
   Else
   SplashTextOn('YVL HelpDesk App', 'Your ticket has been submitted successfully', 300, 100)
Sleep (2000)
SplashOff()

;~ else
    EndIf

EndFunc   ;==>OKButton

Func OnEmail($sSupportEmail)
    Run(@ComSpec & ' /c ' & 'start mailto:' & $sSupportEmail, '', @SW_HIDE)
EndFunc   ;==>OnEmail

Func CLOSEClicked()
    ;Note: at this point @GUI_CTRLID would equal $GUI_EVENT_CLOSE,
    ;and @GUI_WINHANDLE would equal $hMainwindow

    Exit
 EndFunc   ;==>CLOSEClicked

Func GetManager($Username)
Global $objConnection = ObjCreate('ADODB.Connection')  ; Create COM object to AD
$objConnection.ConnectionString = 'Provider=ADsDSOObject'
$objConnection.Open ('Active Directory Provider')  ; Open connection to AD
Global $objRootDSE = ObjGet('LDAP://RootDSE')
Global $strDNSDomain = $objRootDSE.Get ('defaultNamingContext')  ; Retrieve the current AD domain name
Global $strHostServer = $objRootDSE.Get ('dnsHostName') ; Retrieve the name of the connected DC
Global $strConfiguration = $objRootDSE.Get ('ConfigurationNamingContext') ; Retrieve the Configuration naming context
$object=$Username
$strQuery = '<LDAP://' & $strHostServer & '/' & $strDNSDomain & '>;(sAMAccountName=' & $object & ');ADsPath;subtree'
$objRecordSet = $ObjConnection.Execute ($strQuery)  ; Retrieve the FQDN for the object
$ldap_entry = $objRecordSet.fields (0).value
$oObject = ObjGet($ldap_entry)  ; Retrieve the COM Object for the object
$oObject.GetInfo
$mailad = $oObject.Get ('Manager')
$oObject.PurgePropertyList
$oObject = 0

;- msgbox (4096, 'Mail', $mailad)
$manageruser=''
If $mailad <> '' Then
$mgrsplit1 = StringSplit ( ''& $mailad, ',')
$managerfn = StringTrimLeft ( ''& $mgrsplit1[1], 3 )
$mgrsplit2 = StringSplit ( ''& $managerfn, ' ')
$firstnm=$mgrsplit2[1]
If $mgrsplit2[2] <> '' Then
$lastnm=$mgrsplit2[2]
Else
$lastnm=$mgrsplit2[1]
EndIf

$manageruser=StringLeft($firstnm,1)&$lastnm&'@yvl.org'
EndIf
;- msgbox (0, 'Test', $manageruser)
;- $manageruser=StringReplace($manangeruser, '-', '')
Return($manageruser)

  EndFunc

;
; The UDF
Func _INetSmtpMailCom($s_SmtpServer, $s_FromName, $s_FromAddress, $s_ToAddress, $s_Subject = '', $sBody = '', $s_AttachFiles = '', $s_CcAddress = '', $s_BccAddress = '', $s_Importance = 'Normal', $s_Username = '', $s_Password = '', $IPPort = 25, $ssl = 0)

    Local $objEmail = ObjCreate('CDO.Message')
    $objEmail.From = ''' & $s_FromName & '' <' & $s_FromAddress & '>'
    $objEmail.To = $s_ToAddress
   Local $i_Error = 0
    Local $i_Error_desciption = ''
;~    If $MyManager <> '' Then
;~    $objEmail.Cc = $MyManager
;~    EndIf
    If $s_BccAddress <> '' Then $objEmail.Bcc = $s_BccAddress
    $objEmail.Subject = $s_Subject
    If StringInStr($sBody, '<') And StringInStr($sBody, '>') Then
        $objEmail.HTMLBody = $sBody
    Else
        $objEmail.Textbody = $sBody & @CRLF
    EndIf
    If $s_AttachFiles <> '' Then
        Local $S_Files2Attach = StringSplit($s_AttachFiles, ';')
        For $x = 1 To $S_Files2Attach[0]
            $S_Files2Attach[$x] = _PathFull($S_Files2Attach[$x])
;~ ConsoleWrite('@@ Debug : $S_Files2Attach[$x] = ' & $S_Files2Attach[$x] & @LF & '>Error code: ' & @error & @LF) ;### Debug Console
            If FileExists($S_Files2Attach[$x]) Then
                ConsoleWrite('+> File attachment added: ' & $S_Files2Attach[$x] & @LF)
                $objEmail.AddAttachment($S_Files2Attach[$x])
            Else
                ConsoleWrite('!> File not found to attach: ' & $S_Files2Attach[$x] & @LF)
                SetError(1)
                Return 0
            EndIf
        Next
    EndIf
    $objEmail.Configuration.Fields.Item('http://schemas.microsoft.com/cdo/configuration/sendusing') = 2
    $objEmail.Configuration.Fields.Item('http://schemas.microsoft.com/cdo/configuration/smtpserver') = $s_SmtpServer
    If Number($IPPort) = 0 Then $IPPort = 25
    $objEmail.Configuration.Fields.Item('http://schemas.microsoft.com/cdo/configuration/smtpserverport') = $IPPort
    ;Authenticated SMTP
    If $s_Username <> '' Then
        $objEmail.Configuration.Fields.Item('http://schemas.microsoft.com/cdo/configuration/smtpauthenticate') = 1
        $objEmail.Configuration.Fields.Item('http://schemas.microsoft.com/cdo/configuration/sendusername') = $s_Username
        $objEmail.Configuration.Fields.Item('http://schemas.microsoft.com/cdo/configuration/sendpassword') = $s_Password
    EndIf
    If $ssl Then
        $objEmail.Configuration.Fields.Item('http://schemas.microsoft.com/cdo/configuration/smtpusessl') = True
    EndIf
    ;Update settings
    $objEmail.Configuration.Fields.Update
    ; Set Email Importance
    Switch $s_Importance
        Case 'High'
            $objEmail.Fields.Item('urn:schemas:mailheader:Importance') = 'High'
        Case 'Normal'
            $objEmail.Fields.Item('urn:schemas:mailheader:Importance') = 'Normal'
        Case 'Low'
            $objEmail.Fields.Item('urn:schemas:mailheader:Importance') = 'Low'
    EndSwitch
    $objEmail.Fields.Update
    ; Sent the Message
    $objEmail.Send
    If @error Then
        SetError(2)
        Return $goMyRet
    EndIf
    $objEmail = ''
EndFunc   ;==>_INetSmtpMailCom
;
;
; Com Error Handler

Func MyErrFunc()
    $HexNumber = Hex($goMyError.number, 8)
    $goMyRet[0] = $HexNumber
    $goMyRet = StringStripWS($goMyError.description, 3)
    ConsoleWrite('### COM Error ! Number: ' & $HexNumber & ' ScriptLine: ' & $goMyError.scriptline & ' Description:' & $goMyRet & @LF)
    SetError(1); something to check for when this function returns
    Return
EndFunc   ;==>MyErrFunc

Thanks

Grimm

[BrewManNH]

You're reading the comboboxes before anything is in them. You have to read them after the user presses the Send button not right after you've made them. The script is read from the top to the bottom of the text until it hits a loop or jumps somewhere else by calling a function. Your GUICtrlRead statements are not inside the While loop, so they're only read once.

[grimmlock]

Your a genius. Thank you!!!!!!

I moved the $sUsername = GuictrlRead($hCombo2) ; User Name from just above the while to under the $msg (under Case $hOKButton)

It works!!!!

Thank you

Grimm

[BrewManNH]

Glad I could help.

[grimmlock]

I need some help with this again. I wanted to add the ability to add attachments. Since I already have this tread open with the code, I was wondering how easy it would be to allow the form to have the ability to add and send an attachment.

I have looked and it got overwhelming and kind of confusing.

Thanks

Grimm

Edited February 7, 2013 by grimmlock