Saving to chosen location then attaching to email

[Maleficent]

Hi all

I have a workbook which will be used by people from a number of different clients. I have VBA code to "submit" the completed workbook which involves saving the xlsm file to an xlsx file, enabling the user to choose a save location and then attaching the saved xlsx file to an email.

I'm quite new to VBA and found this code online which works great if the user wants to save to their desktop but if they choose a folder, the code just exits. Please can you help me modify the code?

Thanks

##other code ensuring mandatory fields have been completed....then

Please Login or Register  to view this content.

UPDATE - it works fine when saving to desktop, but choosing any other folder results in the code exiting and no further action

[LJMetzger]

Hi M,

I never save anything to the Desktop, so I can't comment on that.

However, as you probably have already figured out you need to change the following line:

Please Login or Register  to view this content.

Your code is difficult to debug because you have a concatenation on your 'save as' line.
If you had an intermediate step such as:

Please Login or Register  to view this content.

you would probably find it easier to verify that you did things correctly using a MsgBox or Debug.Print or single stepping through the debugger.

Debugger Secrets:
a. Press 'F8' to single step (goes into subroutines and functions).
b. Press SHIFT 'F8' to single step OVER subroutines and functions.
c. Press CTRL 'F8' to stop at the line where the cursor is.
d. 'Left Click' the margin to the left of a line to set (or clear) a BREAKPOINT.
e. Press CTRL 'G' to open the IMMEDIATE WINDOW. 'debug.print' statements send their
output to the IMMEDIATE WINDOW.
f. Select View > Locals to see all variables while debugging.
g. To automatically set a BREAKPOINT at a certain location put in the line:
'Debug.Assert False'
h. To conditionally set a BREAKPOINT at a certain location put in lines similar to:
if i >= 20 and xTV20 > 99.56 then
Debug.Assert False
endif
i. A variable value will be displayed by putting the cursor over the variable name.

To manually set a breakpoint, see http://www.wiseowl.co.uk/blog/s196/breakpoints.htm

Lewis

[Maleficent]

Hi LJMetzger

i did manage to solve my issue but wanted to thank you for taking the time to respond as you have given me some really helpful advice!

Thanks
Dani

[Maleficent]

For anyone else with the same issue, this is how i solved it:

Please Login or Register  to view this content.