AutoHotkey Community

[ScriptStudent]

Hi all,

Im new to AutoHotKey, Im working on a script to copy files by modification date. This is a straight copy with no source files to compare. Im trying to recurse through C:\ drive and copy all documents older than 10 days. Im running into error messages like the one below :

---------------------------
AHKeyCpyByDate.exe
---------------------------
Error: This date-time string contains at least one invalid component.

Specifically: 10

Line#
---> 008: var1 -= 10

The program will exit.


here is the current script any input would greatly appreciated:


; Copy only files based on the modification date

CopyIfNewer:
FileCreateDir, C:\AHkeyTest
SetWorkingDir, C:\

var1 = ; Make it blank so that the below will use the current time instead.
var1 -= 10, days
Loop, %A_WorkingDir%\*.doc, , 1 ; Recurse into subfolders.

{
FileGetTime, time, %A_WorkingDir%\%A_LoopFileName%
EnvSub, %var1%, %A_LoopFileTimeModified%, days ; Subtract the source file's time in days from the destination's.
if time <= 0 ; Source file is newer than destination file.
copy_it = y

}
if copy_it = y
{
FileCopy, %A_LoopFileFullPath%, C:\AHkeyTest\%A_LoopFileName%, 1 ; Copy with overwrite=yes
if ErrorLevel <> 0
MsgBox, Could not copy "%A_LoopFileFullPath%" to "C:\AHkeyTest\%A_LoopFileName%".
}

[Serenity]

Please use the code tags when posting scripts, it makes it easier for others to read.
_________________
"Anything worth doing is worth doing slowly." - Mae West

[ScriptStudent]

I was not aware of the different tags until your reply...

[Chris]

[ScriptStudent]

I got part of the code working it seems to be copying some of the files correctly however some files are older than the modified date (which should be all files that are ten days old ) this is the current script:

[guivho]

[ScriptStudent]

Thanks for the reply,

The code is still copying files with older Modification dates than ten days (one file was last modified in 1991). Im not sure what its seeing thats validating the files. here is the current script (its basically the same script with the modifications).

[Decarlo110]

[ScriptStudent]

Im new to open source code can you give me the abrdiged version of what your trying to say...??

[guivho]

ScriptStudent,

Your working dir, the root of your loop, is set at c:\
Your target dir is set at C:\AHKeyTest
So within your loop, you will read your target dir and check whether its content needs to be copied to itself. Luckily, the test should always turn negative, but this does not seem a clean approach.

Your loop recurses over your file system, and changes the value of the %copy_it% variable from the moment a positive test is detected. The loop happily continues, without ever resetting the value of %copy_it%, nor copying a file.

After your loop, you check whether there was any file in the filesystem that needed to be copied, and than you copy the last seen file.

You might want to try this one: