How to check empty messages?

[Rainer]

Hello friends of HMS,

is the following code correct, to check and reject empty messages?

Sub OnAcceptMessage(oClient, oMessage)
....If (Len(oMessage.Body) = 0) And (oMessage.Attachments.Count = 0) Then
........MyResultMessage = "message rejecteced because empty!"
........MyResultValue = 2
....MyMessageBlocked = True
....End If
End Sub

Kind regards

[^DooM^]

Code: Select all

Sub OnAcceptMessage(oClient, oMessage)
    If (Len(oMessage.Body) <= 0) And (oMessage.Attachments.Count <= 0) Then
        Result.Message = "message rejecteced because empty!"
        Result.Value = 2
    End If
End Sub

[Rainer]

Hello DooM, we get every day a lot of empty messages; no subject and no text in body.

Kind regards

[^DooM^]

Tweaked the script. Does it work?

[Rainer]

Hello Doom, now I'm at Home.

My first try only with: If (Len(oMessage.Body) = 0) won't work.
Messages with body-text was rejected!

Tomorrow I will test it with <= 0

Kind regards

[DeanoX]

Rainer,

Are you using Spamassassin?

Curious.


-Dean

[Rainer]

Good Morning DeanoX, Spamassassin is not used.

Kind regards

[Rainer]

@^DooM^ , the script won't work. Messages with content will be rejected too!

Any ideas?

Kind regards

[^DooM^]

Not without debugging.

Stick this line in and see what is contained in the vars you are checking.

EventLog.Write(oMessage.Body)

[Rainer]

Hello ^DooM^, maby this helps:

If IsNull(oMessage.Body) Then
...
ElseIf Len(oMessage.Body) <= 0 Then
...
End If

Kind regards

[^DooM^]

I don't see how that would help. If IsNULL is the same s LEN <= 0 seeing as they would both be empty. I'm not a VB scripter though I mainly code in PHP

I would use if( empty( $oMessage->Body ) ) Like i say you will have to do some debugging, search through old posts in the scripting forum and user contributed sections.

[Rainer]

Hello ^DooM^, IsNull(oMessage.Body) ist not the same like Is Len(oMessage.Body) <= 0.

Len-0 = "" (empty string)
IsNull = nothing

If you work with databases you will know this!

Kind regards

[^DooM^]

Ok "technically" they are different but in this regard they are the same and would do the same thing. And I work with MySQL all day every day