Solved

Method 'SaveAs' of object '_Workbook' failed

Posted on 2011-09-09
13
904 Views
Last Modified: 2012-05-12
In Access I use VBA to open a single worksheet Excel file and save it as a text file. I never had a problem with it but it gives the above error on a user's PC in a different location.

    Set myXL = New Excel.Application
    myXL.Visible = False
    Set wb = myXL.Workbooks.Open(strXLfile)
    myXL.DisplayAlerts = False 'don't want dialog box
    With wb
        .SaveAs Filename:=strNewName, FileFormat:=xlTxtFileType
        .Close SaveChanges:=False
    End With
    myXL.DisplayAlerts = True


0
Comment
Question by:TimHudspith
  • 6
  • 6
13 Comments
 
LVL 41

Expert Comment

by:dlmille
ID: 36515952
You might get an error if the saveas file is already opened, or already exists

Dave
0
 
LVL 33

Accepted Solution

by:
Norie earned 500 total points
ID: 36515971
Tim

Are you sure you are using the correct file format?

I've never seen, and can't find a reference to, a format called xlTxtFileType.

Why not try just xlText?
0
 

Author Comment

by:TimHudspith
ID: 36515993
imnorie - I should have pasted in the whole procedure. 'xlTxtFileType' is actually an argument of type Excel.xlFileFormat in this subprocedure. The calling procedure can choose whether to save the the file as CSV (in which case it passes in the constant xlCSVWindows) or TXT (in which case it passes in xlTextWindows).
0
 

Author Comment

by:TimHudspith
ID: 36515998
Additional info - the user is located in Taiwan and I think is using a Chinese-language Windows, if that makes any difference.
0
 
LVL 33

Expert Comment

by:Norie
ID: 36516004
Do you mean it's passed as an argument to the function?

What happens when you don't and just use either of the file format constants you want to use?

Another thing that could be a problem is strFileName not being a valid filename.

How/where are you defining that?
0
 

Author Comment

by:TimHudspith
ID: 36516084
Yes, it is passed in as an argument defined in the caller subprocedure, and strFileName is defined by the location of the original Excel file, which is held in an Access table. This has worked fine on my PC for a long time with no problems. There is no question of the filepath being incorrect - I have eliminated that possibility.

I will try a dedicated procedure without the xlFileFormat argument being passed in and see how that fares.
0
Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

 
LVL 33

Expert Comment

by:Norie
ID: 36516089
Tim

You aren't using the filepath in the code.

What's the full error message anyway?

Is it basically what you posted, or is there something that could possibly point to where the problem is?
0
 

Author Comment

by:TimHudspith
ID: 36516100
The error message is exactly as per this question title: error number 1004, "Method 'SaveAs' of object '_Workbook' failed.
0
 
LVL 33

Expert Comment

by:Norie
ID: 36516116
Tim

Where is the path in the code?

All you appear to be using is the filename, without the path the file will get save in what VBA/Windows regards as the current directory.
0
 

Author Comment

by:TimHudspith
ID: 36516132
The path is held in a table that contains a list of files to import. The import procedure loops down this table's recordset importing the files, most of which are text or csv - no problems there. When it encouters this particular Excel file there is a Yes/No field value in that table that indicates that it must call the subprocedure to convert the Excel workbook to a text file. The file path is read from the field into a string variable which is then passed to the subprocedure.
0
 
LVL 33

Expert Comment

by:Norie
ID: 36516258
Tim

Could you please check the value of strNewName?

Does it include the full path and a valid filename?

That's for the user having the problem.
0
 

Author Comment

by:TimHudspith
ID: 36516281
I'll pick this question up again on Monday when the user is back in the office. Thanks for your help so far.
0
 
LVL 33

Expert Comment

by:Norie
ID: 36516501
No problem.
0

Featured Post

Is Your Active Directory as Secure as You Think?

More than 75% of all records are compromised because of the loss or theft of a privileged credential. Experts have been exploring Active Directory infrastructure to identify key threats and establish best practices for keeping data safe. Attend this month’s webinar to learn more.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Suggested Solutions

Title # Comments Views Activity
Excel VBA - Public declaration of variables 3 45
Excel partial font macro 14 26
Auto populate in Cascade dropdown 3 25
Most Consistent Performer 4 19
Introduction While answering a recent question (http:/Q_27311462.html), I created an alternative function to the Excel Concatenate() function that you might find useful.  I tested several solutions and share the results in this article as well as t…
Freeze panes is an option within all variants of Excel to enable parts of a sheet to remain stationary when the cursor is in another part of the sheet. This is a very useful feature which is overlooked or under used.
This Micro Tutorial demonstrates how to create Excel charts: column, area, line, bar, and scatter charts. Formatting tips are provided as well.
This Micro Tutorial demonstrate the bugs in Microsoft Excel for Mac with Pivot Charts.

929 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question

Need Help in Real-Time?

Connect with top rated Experts

15 Experts available now in Live!

Get 1:1 Help Now