When I tried to rebuild inbox I got this error:

An AJAX HTTP error occurred. HTTP Result Code: 500 Debugging information follows. Path: /batch?id=8&op=do StatusText: Service unavailable (with message) ResponseText: PDOException: SQLSTATE[42S02]: Base table or view not found: 1146 Table 'drupal_drupal7.pm_index' doesn't exist: SELECT pmi.thread_id, pmi.recipient, pmi.type FROM {pm_index} pmi WHERE thread_id IN (:threads_0) GROUP BY pmi.thread_id, pmi.recipient, pmi.type HAVING ((SELECT pmf.author FROM {pm_message} pmf WHERE pmf.mid = pmi.thread_id) = pmi.recipient AND pmi.type IN ('user', 'hidden') AND COUNT(pmi.thread_id) > 1) OR (SELECT COUNT(*) FROM {pm_message} pmf INNER JOIN pm_index pmif ON (pmf.mid = pmif.mid) WHERE pmif.thread_id = pmi.thread_id AND pmf.author <> pmi.recipient AND pmi.type IN ('user', 'hidden')) > 0; Array ( [:threads_0] => 1 ) in privatemsg_filter_inbox_rebuild_process() (line 263 of D:\_localhost\htdocs\drupal7\htdocs\sites\all\modules\privatemsg\privatemsg_filter\privatemsg_filter.admin.inc).

I fix it in the attached patch.

Best,
Petrica

Comments

berdir’s picture

Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, privatemsg_filter.admin_.patch, failed testing.

berdir’s picture

Version: 7.x-1.0-alpha1 » 7.x-1.x-dev

Fix looks good, any chance that you can create a proper patch, the easiest way is to use cvs diff. See http://drupal.org/node/3279/cvs-instructions/DRUPAL-7--1 or http://drupal.org/patch/create

Petrica’s picture

StatusFileSize
new1.53 KB

How about this one. I don't have a CVS client, but it worked applying on my computer.

berdir’s picture

Status: Needs work » Needs review

You need to patch from the privatemsg main directory and the file names need to be identical.You should be able to correct this manually directly in the patch file (prepend privatemsg_filter/ and remove _new).

Anyway, it's not that problematic, this is just for the testbot and we don't have tests for this specific functionality anyway (or they wouldn't work). It would however help you for you next patches, especially when you start writing patches for core.

Status: Needs review » Needs work

The last submitted patch, 2.privatemsg_filter.admin_.patch, failed testing.

berdir’s picture

Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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