Fatal errors, fatal errors everywhere!

CommentFileSizeAuthor
#10 forum.patch2.53 KBmfb
#1 613278-forum-borked-D7.patch1.11 KBdave reid

Comments

dave reid’s picture

Status: Active » Needs review
StatusFileSize
new1.11 KB
chrishaslam’s picture

Prior to this patch upon replying to a forum post I was getting the following error

Recoverable fatal error: Argument 4 passed to db_query_range() must be an array, integer given, called in /var/www/html/chris/drupal-7/modules/forum/forum.module on line 1113 and defined in db_query_range() (line 1897 of /var/www/html/chris/drupal-7/includes/database/database.inc).

After applying the patch the error is gone and I can successfully reply to a forum post. Did this error occur elsewhere and is further testing needed?

heather’s picture

Tested.

Me too. Before I applied the patch, I got same error as chrisred when trying to reply with a comment on a forum topic.

Applied the patch and it works! Well done Dave Reid :)

jim0203’s picture

Status: Needs review » Reviewed & tested by the community

This gets a pass from me too. Fixes a misconstructed db_query_range() call. RTBC.

damien tournoud’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs tests

Sorry for the error, that was me...

Let's extend the code coverage a little bit.

pwolanin’s picture

Do you have a suggestion for the added test coverage? Adding a comment with forum module enabled might be enough. If not, let's just fix this critical error asap and come back to it.

jim0203’s picture

Agree with @pwolanin - I'm sure I've seen Dries commit a few crucial patches to HEAD and then mark the issue as "Needs Tests".

mfb’s picture

Status: Needs work » Needs review
StatusFileSize
new2.53 KB

Added one additional bugfix and tests for 1) adding comment to forum topic and 2) editing forum topic that has a comment.

dries’s picture

Status: Needs review » Fixed

Confirmed that this is a bug, and that the patch solves it. Committed.

Status: Fixed » Closed (fixed)
Issue tags: -Needs tests

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