Macro to move E-mail

[Douglas Littlefield]

I have a macro that works great in office 2010, however we recently upgraded our computers and now have office 2013. The macro still works but it can take up 4 or 5 mins to finish the macro where before it happened in about 5-10 seconds.

Details:
Before Office 2010
Current Office 2013

Load Time Before: 5-10 seconds
Load Time 2013: 4-5 mins

Using Outlook with an exchange server and retention folders

I have attached the macro I'm using along with a screen shot of what my outlook folder looks like.

What I'm trying to simply accomplish is to move ALL e-mail that is in my inbox & sent mail folder to my 7 year retention folder if it is older than 60 days.

This also loads at start up of outlook 2013

-----
Is there a way to speed this back up?

 

[Diane Poremsky]

if you run it manually, is it still slow?

 

[Douglas Littlefield]

Yes. I have tried to run it manually and auto start

 

[Diane Poremsky]

i thought maybe it was slow as an autostart. i use a similar macro in 2013 and its speed is acceptable, but when i run it, i'm moving half the messages in the folder (around 2000 get moved). If you run it on start, there shouldn't be a lot of messages to move every day. How is the speed if you do each folder individually?

 

[Douglas Littlefield]

I have tried only doing one folder (inbox) and its the same. It typically only moves 1-3 messages out of each folder when it does run. It would be no issue if it ran slowly and still allowed outlook to function but it will not receive any e-mail until it has completed.

 

[Diane Poremsky]

The first time, with the first message box and moving 30 and almost 2000 messages, it took 4 minutes start to finish. (I had 3000 and 5000 messages in each folder).

If I remove the first msgbox, i can move between 50 and 100 total in under 17 seconds (Now around 3000 in each folder. )

if you want the count, do something like this for the inbox instead of the msgbox then add strInboxCount to the sent folder's msgbox - that saves me 5 seconds (yes, I'm slow!)
strInboxCount = "Moved " & lngMovedItems & " messages(s) From your Inbox."
[DOUBLEPOST=1418926256][/DOUBLEPOST]BTW, are you using cached or online mode? I'm testing with cached.

 

[Douglas Littlefield]

Cached mode, and i don't understand what is going on now, when i try to run the script (that worked previously) im getting an error

Any thoughts?

 

[Diane Poremsky]

That is an odd error - it looks like its saying the macro is in a class module. It should be in ThisOutlookSession.
Did you change any references in Tools, references?
We know the code is fine, it works here.

 

[Douglas Littlefield]

I have changed nothing... About 2 weeks ago i upgraded to office 2013, it was so slow that i deleted it, i re pasted it back in to try the suggestion you gave but even with the old code im getting the error. I try the same thing on another computer and it works fine... i will try and repair office to see if that resolves the issue

 

[Douglas Littlefield]

so i have no idea why it works now and didnt 2 mins ago but anyways...

I tried to enter the new code you gave me, but it does not display any message once complete...

did i enter it wrong?

Private Sub Application_Startup()
Dim objOutlook As Outlook.Application
    Dim objNamespace As Outlook.NameSpace
    Dim objSourceFolder As Outlook.MAPIFolder
    Dim objDestFolder As Outlook.MAPIFolder
    Dim objVariant As Variant
    Dim lngMovedItems As Long
    Dim intCount As Integer
    Dim intDateDiff As Integer
    Dim strDestFolder As String
    
'''''''''' '''''''''' '''''''''' ''''''''''
'''''''''' Inbox Folder  ''''''''''
'''''''''' '''''''''' '''''''''' ''''''''''

    Set objOutlook = Application
    Set objNamespace = objOutlook.GetNamespace("MAPI")
    Set objSourceFolder = objNamespace.GetDefaultFolder(olFolderInbox)
    
    ' use a subfolder under Inbox
    Set objParent = Session.GetDefaultFolder(olFolderInbox)
    Set objManaged = objParent.Parent.Folders("Managed Folders")
    Set objDestFolder = objManaged.Folders("7 Year Retention")

    For intCount = objSourceFolder.Items.Count To 1 Step -1
        Set objVariant = objSourceFolder.Items.Item(intCount)
        DoEvents
        If objVariant.Class = olMail Then
            
             intDateDiff = DateDiff("d", objVariant.SentOn, Now)
             
            ' I'm using 60 days, adjust as needed.
            If intDateDiff > 60 Then
              objVariant.Move objDestFolder
              
              'count the # of items moved
               lngMovedItems = lngMovedItems + 1
            End If
        End If
    Next
    
    ' Display the number of items that were moved.
strInboxCount = "Moved " & lngMovedItems & " messages(s) From your Inbox."
Set objDestFolder = Nothing

'''''''''' '''''''''' '''''''''' ''''''''''
'''''''''' Sent Mail Folder  ''''''''''
'''''''''' '''''''''' '''''''''' ''''''''''

      Set objOutlook = Application
    Set objNamespace = objOutlook.GetNamespace("MAPI")
    Set objSourceFolder = objNamespace.GetDefaultFolder(olFolderSentMail)
    
    ' use a subfolder under Inbox
Set objParent = Session.GetDefaultFolder(olFolderInbox)
Set objManaged = objParent.Parent.Folders("Managed Folders")
Set objDestFolder = objManaged.Folders("7 Year Retention")

    For intCount = objSourceFolder.Items.Count To 1 Step -1
        Set objVariant = objSourceFolder.Items.Item(intCount)
        DoEvents
        If objVariant.Class = olMail Then
            
             intDateDiff = DateDiff("d", objVariant.SentOn, Now)
             
            ' I'm using 7 days, adjust as needed.
            If intDateDiff > 10 Then
              objVariant.Move objDestFolder
              
              'count the # of items moved
               lngMovedItems = lngMovedItems + 1
            End If
        End If
    Next
    
    ' Display the number of items that were moved.
   strInboxCount = "Moved " & lngMovedItems & " messages(s) From your Inbox."
Set objDestFolder = Nothing

End Sub

 

[Michael Bauer]

I wouldn't run this code in Application_Startup, but start a timer to run it after Outlook has completely startet up. See this sample for how to start the timer from the Application_Startup event. Add DisableTimer to the Timer procedure at the top so you'd get only one signal and then stop the timer. And move all your other code from Application_Startup to the Timer procedure.

Play with the timer interval if one minute is too short in your case.

Calling DoEvents is slowing it down. It allows Outlook to do other stuff and halt the execution of your code. I guess OL 2013 is doing more work than OL 2010 when startin up.