Cannot get code to work for API Save Dialog Box

I have an access database (2000) where I want the user to click on a button
to save the current form (with subform) to excel. This event will open a
dialog box so the user will be able to save.

I have seen several links to http://www.mvps.org/access/api/api0001.htm when
creating dialog box to save. However, I cannot get this code to work. What
am I doing wrong?

I copied the code exactly like shown on the website and then started making
my changes to make it work. I did not understand how to put in my own
coding, if this should be a substitute for testit or my own "on click" or
function. Here is what I did by deleting the function testit():

Private Sub cmdTest_Click()
Dim strFilter As String
Dim strSaveFileName As String
strFilter = ahtAddFilterItem(mystrFilter, "Excel Files (*.xls)", "*.xls")
strSaveFileName = ahtCommonFileOpenSave(OpenFile:=False,
Filter:=strFilter, Flags:=ahtOFN_OVERWRITEPROMPT Or ahtOFN_READONLY)
End Sub

I keep getting the error message: "only comments may appear after end sub,
end function or end property."

First, don't put anything in the module you downloaded. It could cause
problems. Here is an example of how I use that exact same API module:

'Set filter to show only Excel spreadsheets
strfilter = ahtAddFilterItem(strfilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY

strCurrMonth = Me.cboPeriod.Column(1)
strCurrYear = Me.txtCurrYear
'Get the File Name To Save
strDefaultDir = "\\rsltx1-bm01\busmgmt\Vought " & strCurrYear & "\" &
strCurrYear _
& " Actuals\" & strCurrMonth & "\"
varGetFileName = "Vought Invoice " & strCurrMonth & " " & strCurrYear &
".xls"

varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT, _
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , _
"xls", varGetFileName, "Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If


"Elleve" wrote:

I have an access database (2000) where I want the user to click on a button
to save the current form (with subform) to excel. This event will open a
dialog box so the user will be able to save.

I have seen several links to http://www.mvps.org/access/api/api0001.htm when
creating dialog box to save. However, I cannot get this code to work. What
am I doing wrong?

I copied the code exactly like shown on the website and then started making
my changes to make it work. I did not understand how to put in my own
coding, if this should be a substitute for testit or my own "on click" or
function. Here is what I did by deleting the function testit():

Private Sub cmdTest_Click()
Dim strFilter As String
Dim strSaveFileName As String
strFilter = ahtAddFilterItem(mystrFilter, "Excel Files (*.xls)", "*.xls")
strSaveFileName = ahtCommonFileOpenSave(OpenFile:=False,
Filter:=strFilter, Flags:=ahtOFN_OVERWRITEPROMPT Or ahtOFN_READONLY)
End Sub

I keep getting the error message: "only comments may appear after end sub,
end function or end property."

Will I have to include the comboboxes in the coding? For your information,
the listbox I mentioned last time consist of two columns that are contain a
relationship from a separate query. They are neccessary in order to pick up
the correct information. This is done by
=[Forms]![frmSupportScheduleMain]![List54].[column](0). Why can't I just
export what is at the current form to excel since this information is already
specified?

Also, as for the coding you provided me - where exactly do I input this?
Under the click event? Will I still need to keep the downloaded coding from
the website as is or make changes to it. I apologize if I ask stupid
questions, but that coding just got me lost.


"Klatuu" wrote:

First, don't put anything in the module you downloaded. It could cause
problems. Here is an example of how I use that exact same API module:

'Set filter to show only Excel spreadsheets
strfilter = ahtAddFilterItem(strfilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY

strCurrMonth = Me.cboPeriod.Column(1)
strCurrYear = Me.txtCurrYear
'Get the File Name To Save
strDefaultDir = "\\rsltx1-bm01\busmgmt\Vought " & strCurrYear & "\" &
strCurrYear _
& " Actuals\" & strCurrMonth & "\"
varGetFileName = "Vought Invoice " & strCurrMonth & " " & strCurrYear &
".xls"

varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT, _
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , _
"xls", varGetFileName, "Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If


"Elleve" wrote:

I have an access database (2000) where I want the user to click on a button
to save the current form (with subform) to excel. This event will open a
dialog box so the user will be able to save.

I have seen several links to http://www.mvps.org/access/api/api0001.htm when
creating dialog box to save. However, I cannot get this code to work. What
am I doing wrong?

I copied the code exactly like shown on the website and then started making
my changes to make it work. I did not understand how to put in my own
coding, if this should be a substitute for testit or my own "on click" or
function. Here is what I did by deleting the function testit():

Private Sub cmdTest_Click()
Dim strFilter As String
Dim strSaveFileName As String
strFilter = ahtAddFilterItem(mystrFilter, "Excel Files (*.xls)", "*.xls")
strSaveFileName = ahtCommonFileOpenSave(OpenFile:=False,
Filter:=strFilter, Flags:=ahtOFN_OVERWRITEPROMPT Or ahtOFN_READONLY)
End Sub

I keep getting the error message: "only comments may appear after end sub,
end function or end property."

Not stupid questions at all. See answers below in your post:

"Elleve" wrote:

Will I have to include the comboboxes in the coding? For your information,
No, what I sent was just an example. You can change it as necessary to use
the values you need for your situation. If they come from a list box, just
reference the column in the list box you need to use.

the listbox I mentioned last time consist of two columns that are contain a
relationship from a separate query. They are neccessary in order to pick up
the correct information. This is done by
=[Forms]![frmSupportScheduleMain]![List54].[column](0). Why can't I just
export what is at the current form to excel since this information is already
specified?
You can. This code does not do the export, it only returns a path and file
where you want to open or save the data. If you are exporting to Excel, you
would use varGetFileName as the File argument in your TransferSpreadsheet.


Also, as for the coding you provided me - where exactly do I input this?
Under the click event? Will I still need to keep the downloaded coding from
If you want this to happen as the result of a click event, then that would
be the place to put it.
You still need the downloaded code. It should be kept by itself in a
Standard module. Mine is named modCommonDialog. The sample I provided
calles the API routines in modCommonDialog.
the website as is or make changes to it. I apologize if I ask stupid
Do Not make changes to it. Calling APIs is very powerful, but also somewhat
dangerous. If you make changes in the downloaded code, you could get some
very weird unexpected results.

API stands for Application Program Interface. What you are doing is making
calls to DLL's in the Windows operating system. If you pass bad values, bad
things can happen. Don't let this discourage you. The alternative is an
ActiveX control. ActiveX controls are not that easy to deal with. This API
is very useful. My current application uses it about 20 different places.

questions, but that coding just got me lost.


"Klatuu" wrote:

First, don't put anything in the module you downloaded. It could cause
problems. Here is an example of how I use that exact same API module:

'Set filter to show only Excel spreadsheets
strfilter = ahtAddFilterItem(strfilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY

strCurrMonth = Me.cboPeriod.Column(1)
strCurrYear = Me.txtCurrYear
'Get the File Name To Save
strDefaultDir = "\\rsltx1-bm01\busmgmt\Vought " & strCurrYear & "\" &
strCurrYear _
& " Actuals\" & strCurrMonth & "\"
varGetFileName = "Vought Invoice " & strCurrMonth & " " & strCurrYear &
".xls"

varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT, _
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , _
"xls", varGetFileName, "Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If


"Elleve" wrote:

I have an access database (2000) where I want the user to click on a button
to save the current form (with subform) to excel. This event will open a
dialog box so the user will be able to save.

I have seen several links to http://www.mvps.org/access/api/api0001.htm when
creating dialog box to save. However, I cannot get this code to work. What
am I doing wrong?

I copied the code exactly like shown on the website and then started making
my changes to make it work. I did not understand how to put in my own
coding, if this should be a substitute for testit or my own "on click" or
function. Here is what I did by deleting the function testit():

Private Sub cmdTest_Click()
Dim strFilter As String
Dim strSaveFileName As String
strFilter = ahtAddFilterItem(mystrFilter, "Excel Files (*.xls)", "*.xls")
strSaveFileName = ahtCommonFileOpenSave(OpenFile:=False,
Filter:=strFilter, Flags:=ahtOFN_OVERWRITEPROMPT Or ahtOFN_READONLY)
End Sub

I keep getting the error message: "only comments may appear after end sub,
end function or end property."

You're great in your feedback and speediness. I seem to be on the right
track now, I only have one more question regarding modules as I have never
used them before - how do you set up the module? Is it like private sub-end
sub?

By the way, here is the code I have for the event itself:

Private Sub cmdExportSupportSchedule_Click()
'Set filter to show only Excel spreadsheets
strFilter = ahtAddFilterItem(strFilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY
'Get the File Name To Save
strDefaultDir = "c:/"
varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT,
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , "xls", varGetFileName,
"Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If
DoCmd.TransferSpreadsheet acExport, acSpreadsheetTypeExcel9,
"frmSupportScheduleGLMainForm", varGetFileName, True
End Sub

Does this look like it would work?

"Klatuu" wrote:

Not stupid questions at all. See answers below in your post:

"Elleve" wrote:

Will I have to include the comboboxes in the coding? For your information,
No, what I sent was just an example. You can change it as necessary to use
the values you need for your situation. If they come from a list box, just
reference the column in the list box you need to use.

the listbox I mentioned last time consist of two columns that are contain a
relationship from a separate query. They are neccessary in order to pick up
the correct information. This is done by
=[Forms]![frmSupportScheduleMain]![List54].[column](0). Why can't I just
export what is at the current form to excel since this information is already
specified?
You can. This code does not do the export, it only returns a path and file
where you want to open or save the data. If you are exporting to Excel, you
would use varGetFileName as the File argument in your TransferSpreadsheet.


Also, as for the coding you provided me - where exactly do I input this?
Under the click event? Will I still need to keep the downloaded coding from
If you want this to happen as the result of a click event, then that would
be the place to put it.
You still need the downloaded code. It should be kept by itself in a
Standard module. Mine is named modCommonDialog. The sample I provided
calles the API routines in modCommonDialog.
the website as is or make changes to it. I apologize if I ask stupid
Do Not make changes to it. Calling APIs is very powerful, but also somewhat
dangerous. If you make changes in the downloaded code, you could get some
very weird unexpected results.

API stands for Application Program Interface. What you are doing is making
calls to DLL's in the Windows operating system. If you pass bad values, bad
things can happen. Don't let this discourage you. The alternative is an
ActiveX control. ActiveX controls are not that easy to deal with. This API
is very useful. My current application uses it about 20 different places.

questions, but that coding just got me lost.


"Klatuu" wrote:

First, don't put anything in the module you downloaded. It could cause
problems. Here is an example of how I use that exact same API module:

'Set filter to show only Excel spreadsheets
strfilter = ahtAddFilterItem(strfilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY

strCurrMonth = Me.cboPeriod.Column(1)
strCurrYear = Me.txtCurrYear
'Get the File Name To Save
strDefaultDir = "\\rsltx1-bm01\busmgmt\Vought " & strCurrYear & "\" &
strCurrYear _
& " Actuals\" & strCurrMonth & "\"
varGetFileName = "Vought Invoice " & strCurrMonth & " " & strCurrYear &
".xls"

varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT, _
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , _
"xls", varGetFileName, "Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If


"Elleve" wrote:

I have an access database (2000) where I want the user to click on a button
to save the current form (with subform) to excel. This event will open a
dialog box so the user will be able to save.

I have seen several links to http://www.mvps.org/access/api/api0001.htm when
creating dialog box to save. However, I cannot get this code to work. What
am I doing wrong?

I copied the code exactly like shown on the website and then started making
my changes to make it work. I did not understand how to put in my own
coding, if this should be a substitute for testit or my own "on click" or
function. Here is what I did by deleting the function testit():

Private Sub cmdTest_Click()
Dim strFilter As String
Dim strSaveFileName As String
strFilter = ahtAddFilterItem(mystrFilter, "Excel Files (*.xls)", "*.xls")
strSaveFileName = ahtCommonFileOpenSave(OpenFile:=False,
Filter:=strFilter, Flags:=ahtOFN_OVERWRITEPROMPT Or ahtOFN_READONLY)
End Sub

I keep getting the error message: "only comments may appear after end sub,
end function or end property."

"Elleve" wrote:

You're great in your feedback and speediness. I seem to be on the right
track now, I only have one more question regarding modules as I have never
used them before - how do you set up the module? Is it like private sub-end
sub?

By the way, here is the code I have for the event itself:

Private Sub cmdExportSupportSchedule_Click()
'Set filter to show only Excel spreadsheets
strFilter = ahtAddFilterItem(strFilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY
'Get the File Name To Save
strDefaultDir = "c:/"
varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT,
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , "xls", varGetFileName,
"Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If
DoCmd.TransferSpreadsheet acExport, acSpreadsheetTypeExcel9,
"frmSupportScheduleGLMainForm", varGetFileName, True
End Sub

Does this look like it would work?

"Klatuu" wrote:

Not stupid questions at all. See answers below in your post:

"Elleve" wrote:

Will I have to include the comboboxes in the coding? For your information,
No, what I sent was just an example. You can change it as necessary to use
the values you need for your situation. If they come from a list box, just
reference the column in the list box you need to use.

the listbox I mentioned last time consist of two columns that are contain a
relationship from a separate query. They are neccessary in order to pick up
the correct information. This is done by
=[Forms]![frmSupportScheduleMain]![List54].[column](0). Why can't I just
export what is at the current form to excel since this information is already
specified?
You can. This code does not do the export, it only returns a path and file
where you want to open or save the data. If you are exporting to Excel, you
would use varGetFileName as the File argument in your TransferSpreadsheet.


Also, as for the coding you provided me - where exactly do I input this?
Under the click event? Will I still need to keep the downloaded coding from
If you want this to happen as the result of a click event, then that would
be the place to put it.
You still need the downloaded code. It should be kept by itself in a
Standard module. Mine is named modCommonDialog. The sample I provided
calles the API routines in modCommonDialog.
the website as is or make changes to it. I apologize if I ask stupid
Do Not make changes to it. Calling APIs is very powerful, but also somewhat
dangerous. If you make changes in the downloaded code, you could get some
very weird unexpected results.

API stands for Application Program Interface. What you are doing is making
calls to DLL's in the Windows operating system. If you pass bad values, bad
things can happen. Don't let this discourage you. The alternative is an
ActiveX control. ActiveX controls are not that easy to deal with. This API
is very useful. My current application uses it about 20 different places.

questions, but that coding just got me lost.


"Klatuu" wrote:

First, don't put anything in the module you downloaded. It could cause
problems. Here is an example of how I use that exact same API module:

'Set filter to show only Excel spreadsheets
strfilter = ahtAddFilterItem(strfilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY

strCurrMonth = Me.cboPeriod.Column(1)
strCurrYear = Me.txtCurrYear
'Get the File Name To Save
strDefaultDir = "\\rsltx1-bm01\busmgmt\Vought " & strCurrYear & "\" &
strCurrYear _
& " Actuals\" & strCurrMonth & "\"
varGetFileName = "Vought Invoice " & strCurrMonth & " " & strCurrYear &
".xls"

varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT, _
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , _
"xls", varGetFileName, "Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If


"Elleve" wrote:

I have an access database (2000) where I want the user to click on a button
to save the current form (with subform) to excel. This event will open a
dialog box so the user will be able to save.

I have seen several links to http://www.mvps.org/access/api/api0001.htm when
creating dialog box to save. However, I cannot get this code to work. What
am I doing wrong?

I copied the code exactly like shown on the website and then started making
my changes to make it work. I did not understand how to put in my own
coding, if this should be a substitute for testit or my own "on click" or
function. Here is what I did by deleting the function testit():

Private Sub cmdTest_Click()
Dim strFilter As String
Dim strSaveFileName As String
strFilter = ahtAddFilterItem(mystrFilter, "Excel Files (*.xls)", "*.xls")
strSaveFileName = ahtCommonFileOpenSave(OpenFile:=False,
Filter:=strFilter, Flags:=ahtOFN_OVERWRITEPROMPT Or ahtOFN_READONLY)
End Sub

I keep getting the error message: "only comments may appear after end sub,
end function or end property."

First thing is, Private Sub End Sub is not a module, it is a procedure.
There are two types of procedures. A Sub is called to execute code but does
not return any value to the calling procedure. You usually use a Sub for
events or for tasks that may be called from more than one place. Basically,
if you have a procedure you may call from more than one form or form multiple
places in a form, you should put that code in a Sub. If it is called from
more than one form or report, then it should be in a Standard module. If it
is only used by one form, then it can go in the General section of the form
module.

A Function is another type of procedure. It is used to perform some
evaluation or calculation and returns a value to the calling procedure. The
same rules as for Subs applys on how to use it .

A module is a collection of Subs and or Functions. Forms and reports can
have, but are not required to have modules. The code you put in your form
and report events are in the form or report module. A Standard module is a
collection of code that is usually called from forms, reports, or other
modules. For example, the code you downloaded for the API is a module. I
usually group procedures into modules that have something in common. I have
one module that is nothing more that date functions - Calculating the number
of working days between two dates, Finding the Friday of a given week, etc.

As to your code below. There are two things that need attention. First
there is a problem with this line:

DoCmd.TransferSpreadsheet acExport, acSpreadsheetTypeExcel9,
"frmSupportScheduleGLMainForm", varGetFileName, True

The next issue is declaring variables. Although not required, it is best to
establish the habit of always declaring and typing your variables. You can
always tell what data type my variables are by the prefix on the name. For
example var is for a Variant data type, str is for a String data type, lng is
Long Integer, etc. The reason you did not see the variable declarations the
code I sent is because they were declared at the module level because I need
them in more than this sub. Read up on varialbe Scoping.

So, add these lines right after the Sub statement:

Dim strFilter as String
Dim lngFlags as Long
Dim strDefaultDir as String
Dim varGetFileName as Variant

And, good luck!

frmSupportScheduleGLMainForm is a form name. What you want here is the name
of the table or query you are exporting.
"Elleve" wrote:

You're great in your feedback and speediness. I seem to be on the right
track now, I only have one more question regarding modules as I have never
used them before - how do you set up the module? Is it like private sub-end
sub?

By the way, here is the code I have for the event itself:

Private Sub cmdExportSupportSchedule_Click()
'Set filter to show only Excel spreadsheets
strFilter = ahtAddFilterItem(strFilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY
'Get the File Name To Save
strDefaultDir = "c:/"
varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT,
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , "xls", varGetFileName,
"Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If
DoCmd.TransferSpreadsheet acExport, acSpreadsheetTypeExcel9,
"frmSupportScheduleGLMainForm", varGetFileName, True
End Sub

Does this look like it would work?

"Klatuu" wrote:

Not stupid questions at all. See answers below in your post:

"Elleve" wrote:

Will I have to include the comboboxes in the coding? For your information,
No, what I sent was just an example. You can change it as necessary to use
the values you need for your situation. If they come from a list box, just
reference the column in the list box you need to use.

the listbox I mentioned last time consist of two columns that are contain a
relationship from a separate query. They are neccessary in order to pick up
the correct information. This is done by
=[Forms]![frmSupportScheduleMain]![List54].[column](0). Why can't I just
export what is at the current form to excel since this information is already
specified?
You can. This code does not do the export, it only returns a path and file
where you want to open or save the data. If you are exporting to Excel, you
would use varGetFileName as the File argument in your TransferSpreadsheet.


Also, as for the coding you provided me - where exactly do I input this?
Under the click event? Will I still need to keep the downloaded coding from
If you want this to happen as the result of a click event, then that would
be the place to put it.
You still need the downloaded code. It should be kept by itself in a
Standard module. Mine is named modCommonDialog. The sample I provided
calles the API routines in modCommonDialog.
the website as is or make changes to it. I apologize if I ask stupid
Do Not make changes to it. Calling APIs is very powerful, but also somewhat
dangerous. If you make changes in the downloaded code, you could get some
very weird unexpected results.

API stands for Application Program Interface. What you are doing is making
calls to DLL's in the Windows operating system. If you pass bad values, bad
things can happen. Don't let this discourage you. The alternative is an
ActiveX control. ActiveX controls are not that easy to deal with. This API
is very useful. My current application uses it about 20 different places.

questions, but that coding just got me lost.


"Klatuu" wrote:

First, don't put anything in the module you downloaded. It could cause
problems. Here is an example of how I use that exact same API module:

'Set filter to show only Excel spreadsheets
strfilter = ahtAddFilterItem(strfilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY

strCurrMonth = Me.cboPeriod.Column(1)
strCurrYear = Me.txtCurrYear
'Get the File Name To Save
strDefaultDir = "\\rsltx1-bm01\busmgmt\Vought " & strCurrYear & "\" &
strCurrYear _
& " Actuals\" & strCurrMonth & "\"
varGetFileName = "Vought Invoice " & strCurrMonth & " " & strCurrYear &
".xls"

varGetFileName = ahtCommonFileOpenSave(ahtOFN_OVERWRITEPROMPT, _
strDefaultDir, "Excel Spreadsheets (*.xls) *.xls", , _
"xls", varGetFileName, "Import Adjusted Actuals", , True)
Me.Repaint
If varGetFileName = "" Then 'User Clicked CANCEL
GoTo LoadAdjustedActuals_Exit
End If


"Elleve" wrote:

I have an access database (2000) where I want the user to click on a button
to save the current form (with subform) to excel. This event will open a
dialog box so the user will be able to save.

I have seen several links to http://www.mvps.org/access/api/api0001.htm when
creating dialog box to save. However, I cannot get this code to work. What
am I doing wrong?

I copied the code exactly like shown on the website and then started making
my changes to make it work. I did not understand how to put in my own
coding, if this should be a substitute for testit or my own "on click" or
function. Here is what I did by deleting the function testit():

Private Sub cmdTest_Click()
Dim strFilter As String
Dim strSaveFileName As String
strFilter = ahtAddFilterItem(mystrFilter, "Excel Files (*.xls)", "*.xls")
strSaveFileName = ahtCommonFileOpenSave(OpenFile:=False,
Filter:=strFilter, Flags:=ahtOFN_OVERWRITEPROMPT Or ahtOFN_READONLY)
End Sub

I keep getting the error message: "only comments may appear after end sub,
end function or end property."

Which column is it you want to enter the data in?

And for your next lesson
Okay, Here are some samples.


First, here is how you open an Excel Spreadsheet for Automation:
'Open Excel
On Error Resume Next ' Defer error trapping.
Set xlApp = GetObject(, "Excel.Application")
If Err.Number 0 Then
blnExcelWasNotRunning = True
Set xlApp = CreateObject("excel.application")
Else
DetectExcel
End If
Err.Clear ' Clear Err object in case error occurred.
On Error GoTo LoadAdjustedActuals_Err
DoEvents
xlApp.DisplayAlerts = False
xlApp.Interactive = False
xlApp.ScreenUpdating = False
Set xlBook = xlApp.Workbooks.Open(varGetFileName, 0, True)
xlBook.Worksheets("Actuals_res_export").Activate
*******************
The above code uses this code. The code below should go it it's own module
just like you did for the Common Dialog API. I call mine modExcelRoutines

Option Compare Database
Option Explicit

' Declare necessary API routines:
Declare Function FindWindow Lib "user32" Alias _
"FindWindowA" (ByVal lpClassName As String, _
ByVal lpWindowName As Long) As Long

Declare Function SendMessage Lib "user32" Alias _
"SendMessageA" (ByVal Hwnd As Long, ByVal wMsg As Long, _
ByVal wParam As Long, _
ByVal lParam As Long) As Long

Sub DetectExcel()
' Procedure dectects a running Excel and registers it.
Const WM_USER = 1024
Dim Hwnd As Long
' If Excel is running this API call returns its handle.
Hwnd = FindWindow("XLMAIN", 0)
If Hwnd = 0 Then ' 0 means Excel not running.
Exit Sub
Else
' Excel is running so use the SendMessage API
' function to enter it in the Running Object Table.
SendMessage Hwnd, WM_USER + 18, 0, 0
End If
End Sub
***********************
Here is a formatting example
With xlSheet
For intX = 2 To lngItmCount + 1
strLeftRange = "C" & Trim(str(intX))
strRightRange = "S" & Trim(str(intX))
For Each cell In xlSheet.Range(strLeftRange, strRightRange)
cell.Font.Size = 10
cell.Font.Name = "Arial"
cell.Font.Bold = True
cell.Interior.Color = conLightBlue
cell.NumberFormat = "##,###,##0_);[Red](##,###,##0)"
Next
Next intX
*********************************
Here is putting in formulas. You could use the Excel Sum function for your
totals
With xlSheet
.Cells(30, 2).Formula = "=+B29"
.Cells(30, 3).Formula = "=+B30+C29"
.Cells(30, 4).Formula = "=+C30+D29"
.Cells(30, 5).Formula = "=+D30+E29"
.Cells(30, 6).Formula = "=+E30+F29"
.Cells(30, 7).Formula = "=+F30+G29"
.Cells(30, 8).Formula = "=+G30+H29"
.Cells(30, 9).Formula = "=+H30+I29"
.Cells(30, 10).Formula = "=+I30+J29"
.Cells(30, 11).Formula = "=+J30+K29"
.Cells(30, 12).Formula = "=+K30+L29"
.Cells(30, 13).Formula = "=+L30+M29"
End With
*******************
Here's how you create a new worksheet
Sub CreateNewSheet(xlApp As Object, xlBook As Object, xlSheet As Object)
'Create a new worksheet
xlApp.Worksheets.Add.Move after:=xlApp.Worksheets(xlApp.Worksheets.Count)
xlBook.Worksheets(xlBook.Worksheets.Count).Activat e
Set xlSheet = xlBook.ActiveSheet
xlSheet.Name = strChartName
End Sub
***************************
Then, once you are done:
xlBook.Save
xlBook.Close
If blnExcelWasNotRunning = True Then
xlApp.Quit
Else
xlApp.DisplayAlerts = True
xlApp.Interactive = True
xlApp.ScreenUpdating = True
End If
Set xlSheet = Nothing
Set xlBook = Nothing
Set xlApp = Nothing

The column I want to edit data in is for qrySupporScheduleNotSettle under
'Approved' (last column on the right).

Thanks, I'll keep working on my lessons for now

"Klatuu" wrote:

Which column is it you want to enter the data in?

And for your next lesson
Okay, Here are some samples.


First, here is how you open an Excel Spreadsheet for Automation:
'Open Excel
On Error Resume Next ' Defer error trapping.
Set xlApp = GetObject(, "Excel.Application")
If Err.Number 0 Then
blnExcelWasNotRunning = True
Set xlApp = CreateObject("excel.application")
Else
DetectExcel
End If
Err.Clear ' Clear Err object in case error occurred.
On Error GoTo LoadAdjustedActuals_Err
DoEvents
xlApp.DisplayAlerts = False
xlApp.Interactive = False
xlApp.ScreenUpdating = False
Set xlBook = xlApp.Workbooks.Open(varGetFileName, 0, True)
xlBook.Worksheets("Actuals_res_export").Activate
*******************
The above code uses this code. The code below should go it it's own module
just like you did for the Common Dialog API. I call mine modExcelRoutines

Option Compare Database
Option Explicit

' Declare necessary API routines:
Declare Function FindWindow Lib "user32" Alias _
"FindWindowA" (ByVal lpClassName As String, _
ByVal lpWindowName As Long) As Long

Declare Function SendMessage Lib "user32" Alias _
"SendMessageA" (ByVal Hwnd As Long, ByVal wMsg As Long, _
ByVal wParam As Long, _
ByVal lParam As Long) As Long

Sub DetectExcel()
' Procedure dectects a running Excel and registers it.
Const WM_USER = 1024
Dim Hwnd As Long
' If Excel is running this API call returns its handle.
Hwnd = FindWindow("XLMAIN", 0)
If Hwnd = 0 Then ' 0 means Excel not running.
Exit Sub
Else
' Excel is running so use the SendMessage API
' function to enter it in the Running Object Table.
SendMessage Hwnd, WM_USER + 18, 0, 0
End If
End Sub
***********************
Here is a formatting example
With xlSheet
For intX = 2 To lngItmCount + 1
strLeftRange = "C" & Trim(str(intX))
strRightRange = "S" & Trim(str(intX))
For Each cell In xlSheet.Range(strLeftRange, strRightRange)
cell.Font.Size = 10
cell.Font.Name = "Arial"
cell.Font.Bold = True
cell.Interior.Color = conLightBlue
cell.NumberFormat = "##,###,##0_);[Red](##,###,##0)"
Next
Next intX
*********************************
Here is putting in formulas. You could use the Excel Sum function for your
totals
With xlSheet
.Cells(30, 2).Formula = "=+B29"
.Cells(30, 3).Formula = "=+B30+C29"
.Cells(30, 4).Formula = "=+C30+D29"
.Cells(30, 5).Formula = "=+D30+E29"
.Cells(30, 6).Formula = "=+E30+F29"
.Cells(30, 7).Formula = "=+F30+G29"
.Cells(30, 8).Formula = "=+G30+H29"
.Cells(30, 9).Formula = "=+H30+I29"
.Cells(30, 10).Formula = "=+I30+J29"
.Cells(30, 11).Formula = "=+J30+K29"
.Cells(30, 12).Formula = "=+K30+L29"
.Cells(30, 13).Formula = "=+L30+M29"
End With
*******************
Here's how you create a new worksheet
Sub CreateNewSheet(xlApp As Object, xlBook As Object, xlSheet As Object)
'Create a new worksheet
xlApp.Worksheets.Add.Move after:=xlApp.Worksheets(xlApp.Worksheets.Count)
xlBook.Worksheets(xlBook.Worksheets.Count).Activat e
Set xlSheet = xlBook.ActiveSheet
xlSheet.Name = strChartName
End Sub
***************************
Then, once you are done:
xlBook.Save
xlBook.Close
If blnExcelWasNotRunning = True Then
xlApp.Quit
Else
xlApp.DisplayAlerts = True
xlApp.Interactive = True
xlApp.ScreenUpdating = True
End If
Set xlSheet = Nothing
Set xlBook = Nothing
Set xlApp = Nothing

I still cannot get the sum to display in the spreadsheet. Here is my code:

Private Sub cmdExportSupportSchedule_Click()
Dim strFilter As String
Dim lngFlags As Long
Dim strDefaultDir As String
Dim varGetFileName As Variant

'Set filter to show only Excel spreadsheets
strFilter = ahtAddFilterItem(strFilter, "Excel Files (*.xls)")
'Hides the Read Only Check Box on the Dialog box
lngFlags = ahtOFN_HIDEREADONLY
'Get the File Name To Save
strDefaultDir = "c:\"
varGetFileName = ahtCommonFileOpenSave( _
OpenFile:=False, _
InitialDir:=strDefaultDir, _
Filter:=strFilter, _
FileName:=strDefaultFileName, _
Flags:=lngFlags, _
DialogTitle:="Save Report")
Me.Repaint
If varGetFileName "" Then
DoCmd.TransferSpreadsheet acExport, acSpreadsheetTypeExcel9,
"qrySupportScheduleUnionqry1and2", varGetFileName, True
End If

'Open Excel
On Error Resume Next ' Defer error trapping.
Set xlApp = GetObject(, "Excel.Application")
If Err.Number 0 Then
blnExcelWasNotRunning = True
Set xlApp = CreateObject("excel.application")
Else
DetectExcel
End If
Err.Clear ' Clear Err object in case error occurred.
'On Error GoTo LoadAdjustedActuals_Err
DoEvents
xlApp.DisplayAlerts = False
xlApp.Interactive = False
xlApp.ScreenUpdating = False
Set xlBook = xlApp.Workbooks.Open(varGetFileName, 0, True)
xlBook.Worksheets("Actuals_res_export").Activate

' Format output
With xlSheet
For intX = 2 To lngItmCount + 1
strLeftRange = "C" & Trim(Str(intX))
strRightRange = "S" & Trim(Str(intX))
For Each cell In xlSheet.Range(strLeftRange, strRightRange)
cell.Font.Size = 10
cell.Font.Name = "Arial"
cell.Font.Bold = True
cell.Interior.Color = conLightBlue
cell.NumberFormat = "##,###,##0_);[Red](##,###,##0)"
Next
Next intX
End With

'Formulas to add each column of amount
With xlSheet
.Cells(25, 6).Formula = "=sum(F2:F24)"
.Cells(25, 7).Formula = "=sum(G2:G24)"
.Cells(25, 8).Formula = "=sum(H2:H24)"
.Cells(25, 9).Formula = "=sum(I2:I24)"
End With

'Done and save
xlBook.Save
xlBook.Close
If blnExcelWasNotRunning = True Then
xlApp.Quit
Else
xlApp.DisplayAlerts = True
xlApp.Interactive = True
xlApp.ScreenUpdating = True
End If

Set xlSheet = Nothing
Set xlBook = Nothing
Set xlApp = Nothing

End Sub

*****************************************

As for the module I created this:

' Declare necessary API routines:
Declare Function FindWindow Lib "user32" Alias "FindWindowA" (ByVal
lpClassName As String, ByVal lpWindowName As Long) As Long
Declare Function SendMessage Lib "user32" Alias "SendMessageA" (ByVal Hwnd
As Long, ByVal wMsg As Long, ByVal wParam As Long, ByVal lParam As Long) As
Long

Sub DetectExcel()
' Procedure dectects a running Excel and registers it.
Const WM_USER = 1024
Dim Hwnd As Long
' If Excel is running this API call returns its handle.
Hwnd = FindWindow("XLMAIN", 0)
If Hwnd = 0 Then
' 0 means Excel not running.
Exit Sub
Else
' Excel is running so use the SendMessage API
' function to enter it in the Running Object Table.
SendMessage Hwnd, WM_USER + 18, 0, 0
End If
End Sub

Sub CreateNewSheet(xlApp As Object, xlBook As Object, xlSheet As Object)
'Create a new worksheet
xlApp.Worksheets.Add.Move after:=xlApp.Worksheets(xlApp.Worksheets.Count)
xlBook.Worksheets(xlBook.Worksheets.Count).Activat e
Set xlSheet = xlBook.ActiveSheet
xlSheet.Name = strChartName
End Sub

What should I do to make it work?

"Klatuu" wrote:

Which column is it you want to enter the data in?

And for your next lesson
Okay, Here are some samples.


First, here is how you open an Excel Spreadsheet for Automation:
'Open Excel
On Error Resume Next ' Defer error trapping.
Set xlApp = GetObject(, "Excel.Application")
If Err.Number 0 Then
blnExcelWasNotRunning = True
Set xlApp = CreateObject("excel.application")
Else
DetectExcel
End If
Err.Clear ' Clear Err object in case error occurred.
On Error GoTo LoadAdjustedActuals_Err
DoEvents
xlApp.DisplayAlerts = False
xlApp.Interactive = False
xlApp.ScreenUpdating = False
Set xlBook = xlApp.Workbooks.Open(varGetFileName, 0, True)
xlBook.Worksheets("Actuals_res_export").Activate
*******************
The above code uses this code. The code below should go it it's own module
just like you did for the Common Dialog API. I call mine modExcelRoutines

Option Compare Database
Option Explicit

' Declare necessary API routines:
Declare Function FindWindow Lib "user32" Alias _
"FindWindowA" (ByVal lpClassName As String, _
ByVal lpWindowName As Long) As Long

Declare Function SendMessage Lib "user32" Alias _
"SendMessageA" (ByVal Hwnd As Long, ByVal wMsg As Long, _
ByVal wParam As Long, _
ByVal lParam As Long) As Long

Sub DetectExcel()
' Procedure dectects a running Excel and registers it.
Const WM_USER = 1024
Dim Hwnd As Long
' If Excel is running this API call returns its handle.
Hwnd = FindWindow("XLMAIN", 0)
If Hwnd = 0 Then ' 0 means Excel not running.
Exit Sub
Else
' Excel is running so use the SendMessage API
' function to enter it in the Running Object Table.
SendMessage Hwnd, WM_USER + 18, 0, 0
End If
End Sub
***********************
Here is a formatting example
With xlSheet
For intX = 2 To lngItmCount + 1
strLeftRange = "C" & Trim(str(intX))
strRightRange = "S" & Trim(str(intX))
For Each cell In xlSheet.Range(strLeftRange, strRightRange)
cell.Font.Size = 10
cell.Font.Name = "Arial"
cell.Font.Bold = True
cell.Interior.Color = conLightBlue
cell.NumberFormat = "##,###,##0_);[Red](##,###,##0)"
Next
Next intX
*********************************
Here is putting in formulas. You could use the Excel Sum function for your
totals
With xlSheet
.Cells(30, 2).Formula = "=+B29"
.Cells(30, 3).Formula = "=+B30+C29"
.Cells(30, 4).Formula = "=+C30+D29"
.Cells(30, 5).Formula = "=+D30+E29"
.Cells(30, 6).Formula = "=+E30+F29"
.Cells(30, 7).Formula = "=+F30+G29"
.Cells(30, 8).Formula = "=+G30+H29"
.Cells(30, 9).Formula = "=+H30+I29"
.Cells(30, 10).Formula = "=+I30+J29"
.Cells(30, 11).Formula = "=+J30+K29"
.Cells(30, 12).Formula = "=+K30+L29"
.Cells(30, 13).Formula = "=+L30+M29"
End With
*******************
Here's how you create a new worksheet
Sub CreateNewSheet(xlApp As Object, xlBook As Object, xlSheet As Object)
'Create a new worksheet
xlApp.Worksheets.Add.Move after:=xlApp.Worksheets(xlApp.Worksheets.Count)
xlBook.Worksheets(xlBook.Worksheets.Count).Activat e
Set xlSheet = xlBook.ActiveSheet
xlSheet.Name = strChartName
End Sub
***************************
Then, once you are done:
xlBook.Save
xlBook.Close
If blnExcelWasNotRunning = True Then
xlApp.Quit
Else
xlApp.DisplayAlerts = True
xlApp.Interactive = True
xlApp.ScreenUpdating = True
End If
Set xlSheet = Nothing
Set xlBook = Nothing
Set xlApp = Nothing