Other posters have pointed out problems. Here's another, which jjhorner also alluded to.

You wrote:

    open(shortcuts,"location.txt" || die"Can't open the stinkin' file"
+);
[download]

This is wrong. Why do you have the || there? The || has higher precedence than the comma, so you're actually OR-ing "location.txt" with the die statement. "location.txt" will always be true, so that die will never get invoked, even if the open fails. Which is, I presume, when you wanted it to be invoked.

Instead, use something like this:

    open SH,"location.txt" or die"Can't open the stinkin' file";
[download]

This will die if the open fails.