Learn More
Resend Email
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.

undefined

php legacy shootme
Favorite
Ranter
Comments
  • 2
    8y
    that real_escape thing is just funny 😂
  • 6
    8y
    As someone who's never worked on a real life project, may I ask what are the wrong behaviors here?
  • 2
    8y
    @oscarascal SQL injection in the file name field and 2 sql-escaped variables that aren't even part of the query
  • 1
    8y
    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
    8y
    I love the arbitrary "count() + 5" to break out of the loop!
  • 1
    8y
    ++ not only for shit code but for using PhpStorm
  • 2
    8y
    this made me want to punch the author... with a chair...
  • 0
    8y
    Update: I resigned.
  • 1
    8y
    That's a sweet color scheme.
  • 0
    8y
    @ninjatini one of the dark themes in Jetbrains PHPStorm
devRant © 2021 Hexical Labs LLC
Privacy Policy  |  Terms of Service
Add Comment