This looks weird. One of these lines has ```$url . "/" . $file_basename``` and the other one just ```$url . $file_basename```.

That's a subtle, but very significative difference. This is *exactly* why we must *never* duplicate code. If you're calculting something, put it in a variable *once*, and use that variable as many times as you need.