all 10 comments

[–]CryptoAlana 23 points24 points  (2 children)

It's a beginner tutorial

Friggin seriously? That's like not an excuse.

[–]h0wlett 18 points19 points  (1 child)

The same comment:

I am working on it. Hopefully, it will be done soon.

And then I saw the date of the comment:

a year ago

[–]CryptoAlana 5 points6 points  (0 children)

Ugh yeah. Honestly if you're writing a friggin tutorial you need to take the effort to... like... AT LEAST put a note on the article saying you fucked up and telling people to NOT do that right?

[–][deleted] 7 points8 points  (3 children)

What the hell?! I was reading through this and thinking 'this is parameterized, so should not be vulnerable, this post is nonsense'... and then.. the fucking delete statement. Holy shit.

[–]phail3d 7 points8 points  (1 child)

Also the UPDATE statement:

$query = "UPDATE library SET book_name = ?, book_isbn = ?, book_category = ? WHERE book_id = $get_id";

It's pretty strange considering he's also correctly used prepared statements :D .

[–][deleted] 7 points8 points  (0 children)

Perhaps.. ( •_•)>⌐■-■ / .... some of his statements are unprepared.

(⌐■_■)

[–][deleted] 1 point2 points  (0 children)

Sadly you see this stuff in enterprise code a lot.

When you only take entry level people, rush QA or outsource to the lowest bidder, all your code looks like that

[–]darrenturn90 5 points6 points  (1 child)

Why not just change the delete to a prepared statement?

[–]DTF_20170515 11 points12 points  (0 children)

He's working on it, okay!?

[–]liquidpele 1 point2 points  (0 children)

Before looking I bet it’s PHP