Moderators Melba23 Posted September 16, 2010 Moderators Posted September 16, 2010 Fran,The file copying code produces the same results for me as it did before, so there is no reason I can see why the files no longer copy. Have you changed the folder structure to match the new code? You were copying the files to:"@ScriptDir & "\config\mphoto_exe\"&GUICtrlRead($theme[$i])&" - Mphoto.exe"and you are now using:"@ScriptDir & "\config\mphoto_exe\PHOTOBOOKS\" & GUICtrlRead($theme[$i]) & " - Mphoto.exe"Do you need to use the "8 = Create destination directory structure if it doesn't exist" flag? M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area
Fran Posted September 16, 2010 Author Posted September 16, 2010 My PC is acting up. I think that might be the problem I just added the flag out of pure frustration - I'm reformatting my pc tonight ... hold thumbs
Fran Posted September 17, 2010 Author Posted September 17, 2010 (edited) @M23 ...Still not working. I've added a message box to the code, and the paths echo 100%. Why won't it copy though? The directory structure is correct. Fran For $i = 0 To $iMax - 1 ;Photobooks copy files If $change[$i] = 1 Then $srcFile = GUICtrlRead($input[$i]) $destFile = @ScriptDir & "\config\mphoto_exe\PHOTOBOOKS\" & GUICtrlRead($theme[$i]) & " - Mphoto.exe" MsgBox(4096, "", "Copy from:" & @CRLF & $srcFile & @CRLF & @CRLF & "to:" & @CRLF & $destFile) FileCopy($srcFile, $destFile, 1) EndIf Next I'm going off my rocket a bit. Edited September 17, 2010 by Fran
Moderators Melba23 Posted September 17, 2010 Moderators Posted September 17, 2010 Fran,Are you absolutely sure it copied the files before? Another thought - you have spaces in the $destfile name:$destFile = @ScriptDir & "\config\mphoto_exe\PHOTOBOOKS\" & GUICtrlRead($theme[$i]) & " - Mphoto.exe"Perhaps if you were to enclose the whole name in double quotes like this:$destFile = '"' & @ScriptDir & '\config\mphoto_exe\PHOTOBOOKS\' & GUICtrlRead($theme[$i]) & ' - Mphoto.exe' & '"'Other than that I am right out of ideas too - sorry. M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area
Fran Posted September 17, 2010 Author Posted September 17, 2010 (edited) I figured out what the problem was!! There wasn't enough disk space. How crazy is that? Anyway... all set and working 100% now. And my pc feels a lot better after the format as well Enjoy your weekend! Fran Edited September 17, 2010 by Fran
Moderators Melba23 Posted September 17, 2010 Moderators Posted September 17, 2010 Fran, I thought that problem went out with floppy disks! You must watch an awful lot of movies! Glad you got it sorted. M23 Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind Open spoiler to see my UDFs: Spoiler ArrayMultiColSort ---- Sort arrays on multiple columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now