#508756: RIP db_placeholders() started it, so lets remove more stuff we don't need anymore :)

There is only a single call to db_affected_rows(), and that query was already converted to db_update(), so we can just remove that...

The nice thing about that is, that we can remove $lastStatement...

CommentFileSizeAuthor
remove_db_affected_rows.patch2.84 KBberdir

Comments

catch’s picture

Status: Needs review » Reviewed & tested by the community

Nice. I felt a bit bad about posting the final nail for db_placeholders() when Berdir had put so many of the previous ones in.

dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.