41
erik404
8y

"Our last lead-developer was a real smart guy, he modernized the legacy code and we all learned a lot from him"

I need a beer, and a shotgun.

Comments
  • 2
    that real_escape thing is just funny 😂
  • 6
    As someone who's never worked on a real life project, may I ask what are the wrong behaviors here?
  • 2
    @oscarascal SQL injection in the file name field and 2 sql-escaped variables that aren't even part of the query
  • 1
    Using $update before assigning a value... Also, why not using PDO with PreparedStatements?
    Incrementing $i instead of breaking the cicle? Pls give me the shotgun
  • 1
    I love the arbitrary "count() + 5" to break out of the loop!
  • 1
    ++ not only for shit code but for using PhpStorm
  • 2
    this made me want to punch the author... with a chair...
  • 0
    Update: I resigned.
  • 1
    That's a sweet color scheme.
  • 0
    @ninjatini one of the dark themes in Jetbrains PHPStorm
Add Comment