We help IT Professionals succeed at work.

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

encryptedheart
on
Medium Priority
534 Views
Last Modified: 2008-02-01
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????
Comment
Watch Question

CERTIFIED EXPERT

Commented:
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

Author

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 ..................
CERTIFIED EXPERT

Commented:
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.

Author

Commented:
Hi,
directly i tried to acces like
MsgBox oHostApp.ActiveExplorer.CommandBars.Item("File").Controls.Item("Open from DM").Tag
still getting same error
CERTIFIED EXPERT

Commented:
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

Author

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.
CERTIFIED EXPERT
Commented:
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

Not the solution you were looking for? Getting a personalized solution is easy.

Ask the Experts
Access more of Experts Exchange with a free account
Thanks for using Experts Exchange.

Create a free account to continue.

Limited access with a free account allows you to:

  • View three pieces of content (articles, solutions, posts, and videos)
  • Ask the experts questions (counted toward content limit)
  • Customize your dashboard and profile

*This site is protected by reCAPTCHA and the Google Privacy Policy and Terms of Service apply.

OR

Please enter a first name

Please enter a last name

8+ characters (letters, numbers, and a symbol)

By clicking, you agree to the Terms of Use and Privacy Policy.