error handler when no workbook opened

Hi All,

Can someone help me out with error handler. Have a little code where it goes to different workbook does little editing copy it and paste it in original report based on todays date. I want to add error handler so when no workbook opened it would give a message that nothing opened. It works when nothing is opened but when I have report opened it still tries to error handle it. Probably it is something simple but I just cannot get my head around it. Thanks

Please Login or Register  to view this content.

Hi AlphaFrog, thanks for response.

it works when report is opened but when it`s not it gives twice MsgBox and then goes to runtime error '1004' no data selected to parse

you can use the debugger and follow the code in single steps. You will see when the ErrorHandler is executed.

If you see the MsgBox twice then because you will get an error for both code lines between "On Error .."

Please Login or Register  to view this content.

with the second statement ".. Goto 0" you disable the error handler and now you get the standard error message "runtime error '1004' "

Remove ".. Goto 0" and remove also "Resume Next" in the ErrorHandler section because you want to finish execution and exit the sub

Hi gue2013, thanks for response and advice using debugger. Kind of combined both advices and this is the code:

Please Login or Register  to view this content.

I places diferent handler before "Application.ActiveProtectedViewWindow.Edit" left "goto 0" to disable handlers, removed "Resume Next" and changed it to "Exit Sub"

So it is working how I need. Thanks again for all help!

I never used Application.ActiveProtectedViewWindow (I most probably need not use it) in the past and still not sure when or if to use it.

The "Exit Sub" after "MsgBox" is not required because with "Exit Sub" you jump to "End Sub" which is the next statement after "MsgBox"

Thing is the people who is using the spreadsheet not really tech savvy and keep forgetting to enable editing and then complaining that nothing is working so that`s why I am using Application.ActiveProtectedViewWindow just simplify everything.

And thanks for letting me know on "End Sub" I will declutter the code