Help needed with 'For each'

As an example (from the Wiki) I made a macro to move all files from the downloads folder to my desktop. Not sure what I am doing wrong, but nothing happens!
Can anyone tell me how I can achieve this?
The macro:

[Move files.kmmacros|attachment]

Move Downloads to Desktop.kmmacros (35 KB)

Macro screenshot

"The Finder's selection" is quite literally "the things selected in the Finder". If you opened your Downloads folder and selected all the items in there, your macro would probably work.

See @noisneil's macro for a better way.

Thanks @Nige_S
Did not know I had to select the files in the finder. Works now.

You don't if you try the macro I just made for you.

Thanks @noisneil for your macro. It's much easier as mine but understand now how for each' is working, i hope, with these two answers