• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 522
  • Last Modified:

Error in MyButton.Delete() in OnBeginShutdown event when using outlook comaddins

Hi,
I developed small com-addins(VB6) to create 2 buttons on Startup & to remove the button on OnBeginShutdown.

Code Snippet:
Dim WithEvents cmdSaveinDM As Office.CommandBarButton
Set mnuOpenfrmDM = oHostApp.CommandBars.FindControl(Tag:="Open from DM")
cmdOpenfrmDM.Delete

above code removes Menu button when exit the application for Word,Excel,Powerpoint etc but except for Outlook.

even i tried with
Dim WithEvents cmdSaveinDM As Office.CommandBarButton
Set mnuOpenfrmDM = oHostApp.ActiveInspector.CommandBars.FindControl(Tag:="Open from DM")
cmdOpenfrmDM.Delete

still am getting error says object doesnt support.

Can anyone help me in this case????
0
encryptedheart
Asked:
encryptedheart
  • 4
  • 3
1 Solution
 
mvidasCommented:
Hi,

You don't specify which line the object doesn't support, but if you're just removing the buttons, why use WithEvents? Are you in a class module?

When I add my buttons from a COM add-in, I just set the Temporary argument to True while adding them.  And at the top of my RemoveButtons sub (called from OnDisconnect) I just put On Error Resume Next (since the buttons will be removed when it closes, and this way I won't get the random error like this when outlook disagrees with my code).

Lastly, are you sure you want ActiveInspector and not ActiveExplorer? Both have the commandbars object, though the Explorer is the more common place to put buttons, especially if you'll be calling them from OnBeginShutdown.  If you're adding them to an inspector object, just make them temporary as inspectors are closed frequently.

My opinion at least :)
Matt
0
 
encryptedheartAuthor Commented:
Hi,
Am getting error in the line
Set mnuOpenfrmDM = oHostApp.CommandBars.FindControl(Tag:="Open from DM")
I declared this button with events for different purpose.
Even i tried to set temporary:true as arguement while creating new button, it doesnt works.
I tried
Set mnuOpenfrmDM = oHostApp.ActiveExplorer.CommandBars.FindControl(Tag:="Open from DM")
 no use.
still getting same ..................
0
 
mvidasCommented:
Hmm! Interesting..

I just added one to the Tools menu using:

 With oHostApp.ActiveExplorer.CommandBars("Tools").Controls.Add(msoControlButton, , , , True)
  .Tag = "Open from DM"
  .Caption = "Hello!"
 End With

Then used

 Dim mnuOpenfrmDM As CommandBarControl
 Set mnuOpenfrmDM = oHostApp.ActiveExplorer.CommandBars.FindControl(Tag:="Open from DM")
 If Not mnuOpenfrmDM Is Nothing Then
  mnuOpenfrmDM.Delete
 Else
  MsgBox "mnuOpenfrmDM is nothing"
 End If

And it did get deleted.. when I ran it a second time I got the message box.
Since you're the one adding the control, why use FindControl? Why not specify where it is and delete it that way? In my case (in the tools menu) :

 Set mnuOpenfrmDM = oHostApp.ActiveExplorer.CommandBars("Menu Bar").Controls("Tools").Controls("Hello!")

Though usually in my com add-ins, I add a separate menu after Help for any buttons, and then just delete the whole new menu.
0
Free Tool: Subnet Calculator

The subnet calculator helps you design networks by taking an IP address and network mask and returning information such as network, broadcast address, and host range.

One of a set of tools we're offering as a way of saying thank you for being a part of the community.

 
encryptedheartAuthor Commented:
Hi,
directly i tried to acces like
MsgBox oHostApp.ActiveExplorer.CommandBars.Item("File").Controls.Item("Open from DM").Tag
still getting same error
0
 
mvidasCommented:
You would still have to reference the "Menu Bar" commandbar.  If you want to use the Item property like that, it would be:

 MsgBox oHostApp.ActiveExplorer.CommandBars.Item("Menu Bar").Controls.Item("File").Controls.Item("Open from DM").Tag

Though it could also be written shorter, as

 MsgBox oHostApp.ActiveExplorer.CommandBars("Menu Bar").Controls("File").Controls("Open from DM").Tag

Matt
0
 
encryptedheartAuthor Commented:
Hey i got the solution by using temporary flag to true.
actually my code written in a way if button already exist then not to insert the button,
so earlier i created button without temporary flag is set to true.
for the i should manually remove it.
now when i tried with that, its working fine.
thanks for your effort to help me.
0
 
mvidasCommented:
Glad to have helped! At the beginning of all my "InsertCommandButtons" subs, I always call "RemoveCommandButtons" in case they're still resident in there.  Thats the main reason I use On Error Resume Next in my remove sub, since often they won't be there when it is called
Matt
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.

Join & Write a Comment

Featured Post

Upgrade your Question Security!

Your question, your audience. Choose who sees your identity—and your question—with question security.

  • 4
  • 3
Tackle projects and never again get stuck behind a technical roadblock.
Join Now