Solved

VBS issue

Posted on 2011-09-23
10
341 Views
Last Modified: 2012-05-12
I have VBS that I mashed together that is supposed to look in a folder and take .bkf extensions and change them to .txt. Then it is supposed to delete older files.

It gives me an error on line 9.

Can someone help me on this??

Thanks
Dim fso, startFolder, OlderThanDate

strComputer = "."
Set objWMIService = GetObject("winmgmts:" _
    & "{impersonationLevel=impersonate}!\\" & strComputer & "\root\cimv2")
Set FileList = objWMIService.ExecQuery _
    ("ASSOCIATORS OF {Win32_Directory.Name='\\server\itdept\adbackup'} Where " _
        & "ResultClass = CIM_DataFile")
For Each objFile In FileList
    If objFile.Extension = "bkf" Then
        strNewName = objFile.Drive & objFile.Path & _
            objFile.FileName & "." & "txt"
            errResult = objFile.Rename(strNewName)
        Wscript.Echo errResult
    End If
Next


 Set fso = CreateObject("Scripting.FileSystemObject")
 startFolder = "\\server\itdept\adbackup"
 OlderThanDate = DateAdd("d", -2, Date) ' 2 days 
 DeleteOldFiles startFolder, OlderThanDate 
 
  
 Function DeleteOldFiles(folderName, BeforeDate) 
 Dim folder, file, fileCollection, folderCollection, subFolder 
 Set folder = fso.GetFolder(folderName) 
 Set fileCollection = folder.Files 
 For Each file In fileCollection 
 	If file.DateLastModified < BeforeDate Then 
 		fso.DeleteFile(file.Path) 
 	End If 
 Next 
 Set folderCollection = folder.SubFolders 
 For Each subFolder In folderCollection 
 	DeleteOldFiles subFolder.Path, BeforeDate 
 Next 
 End Function

Open in new window

0
Comment
Question by:clynch302
  • 4
  • 3
  • 2
  • +1
10 Comments
 
LVL 28

Expert Comment

by:sammySeltzer
ID: 36589013
I don't think it likes computer name being define as "."

strComputer = "."

Are you using this for more than one computer or are you planning on passing computer name as param?

Trying sticking actual computer name there and see what happens.

Also, do you have a different file name other than bkf on that folder?
0
 
LVL 54

Expert Comment

by:Bill Prew
ID: 36589151
What error are you getting?

~bp
0
 
LVL 28

Expert Comment

by:sammySeltzer
ID: 36589233
The error says (null) with some error code. That's what I got when I attempted to test it to see if I could figure out what could be wrong.

I wanted to clarify that defining your computer as strComputer="." doesn't mean there is something wrong with that declaration.

I would try using this to diagnose vmi to see if something is amiss with it.

http://windowsxp.mvps.org/repairwmi.htm

0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 

Author Comment

by:clynch302
ID: 36589935
This is used on my computer only.  I have a folder that files are copied to that are .bkf. I need them to be .txt. So this is basically all ran from 1 system. I have a server unc in that I was testing.
0
 

Author Comment

by:clynch302
ID: 36589964
Also, when I add my computer name it does not error out but it also does not rename the .bkf to .txt.
0
 
LVL 65

Expert Comment

by:RobSampson
ID: 36591064
Hi, CIM_DataFile won't allow you to enumerate a remote folder.  If you have admin rights, you should set strComputer to the remote computer, then make the path relative to that computer.

strComputer = "server"
Set objWMIService = GetObject("winmgmts:" _
    & "{impersonationLevel=impersonate}!\\" & strComputer & "\root\cimv2")
Set FileList = objWMIService.ExecQuery _
    ("ASSOCIATORS OF {Win32_Directory.Name='D:\itdept\adbackup'} Where " _
        & "ResultClass = CIM_DataFile")

Open in new window


Regards,

Rob.
0
 

Author Comment

by:clynch302
ID: 36600274
Ya, no dice.n I am still getting the same error. Basically all I need to do is rename bkf to txt and then delete the the oldest files. In my script the delete portion works just not the extension rename.
0
 
LVL 65

Accepted Solution

by:
RobSampson earned 500 total points
ID: 36614685
OK, try it by deleting the target file (if it exists), and then using Move instead.  Also, this doesn't use the CIM_DataFile query, so it might be a bit quicker.

Make sure you test in on a test set of files first.

Regards,

Rob.
Dim objFSO, startFolder, OlderThanDate
Set objFSO = CreateObject("Scripting.FileSystemObject")
startFolder = "\\server\itdept\adbackup"
OlderThanDate = DateAdd("d", -2, Date) ' 2 days 
DeleteOldFiles startFolder, OlderThanDate 

RenameFiles startFolder

Sub DeleteOldFiles(folderName, BeforeDate) 
	For Each file In objFSO.GetFolder(folderName).Files
		If file.DateLastModified < BeforeDate Then
			fso.DeleteFile(file.Path), True
		End If 
	Next 
	For Each subFolder In objFSO.GetFolder(folderName).SubFolders
		DeleteOldFiles subFolder.Path, BeforeDate 
	Next 
End Sub

Sub RenameFiles(folderName)
	For Each objFile In objFSO.GetFolder(startFolder).Files
		If objFile.Extension = "bkf" Then
			strNewName = objFile.Path & ".txt"
			If objFSO.FileExists(strNewName) = True Then objFSO.DeleteFile strNewName, True
			objFSO.MoveFile objFile.Path, strNewName
		End If
	Next
	For Each subFolder In objFSO.GetFolder(folderName).SubFolders
		RenameFiles subFolder.Path
	Next 
End Sub

Open in new window

0
 

Author Closing Comment

by:clynch302
ID: 36710994
Thank you for taking the time to assist me.
0
 
LVL 65

Expert Comment

by:RobSampson
ID: 36713576
No problem. Thanks for the grade.

Regards,

Rob.
0

Featured Post

Free Tool: SSL Checker

Scans your site and returns information about your SSL implementation and certificate. Helpful for debugging and validating your SSL configuration.

One of a set of tools we are providing to everyone as a way of saying thank you for being a part of the community.

Question has a verified solution.

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

In this post we will learn how to connect and configure Android Device (Smartphone etc.) with Android Studio. After that we will run a simple Hello World Program.
Although it can be difficult to imagine, someday your child will have a career of his or her own. He or she will likely start a family, buy a home and start having their own children. So, while being a kid is still extremely important, it’s also …
In this fourth video of the Xpdf series, we discuss and demonstrate the PDFinfo utility, which retrieves the contents of a PDF's Info Dictionary, as well as some other information, including the page count. We show how to isolate the page count in a…

685 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