Can it be this simple?

CommentFileSizeAuthor
x.patch1.29 KBchx
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Mis-using this to test the tests...

Status: Needs review » Needs work

The last submitted patch, x.patch, failed testing.

ptmkenny’s picture

Status: Needs work » Needs review

x.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, x.patch, failed testing.

ptmkenny’s picture

Category: task » feature

I applied the patch manually but I was unable to get it to work. As is, I had to change 'options list' => _privatemsg_rules_filters, to 'options list' => _privatemsg_rules_filters(),, which gives this error: Fatal error: Unsupported operand types in /Volumes/Bullfrog/om/includes/form.inc on line 2623.

When I removed the line return $options and placed a dpm() instead, I can see that the list of input filters is correctly generated; however, for a reason I do not understand, they are not prefilled as options correctly (I get the error with the return statement in place).

In any case, is "format" a property of the private message, or a property of the body of the privatemsg? (Because in theory, couldn't a privatemsg have multiple fields, some of which allow configurable formats and some that do not)?

chx’s picture

Assigned: chx » Unassigned
Issue summary: View changes
ivnish’s picture

Status: Needs work » Closed (outdated)
andypost’s picture

Status: Closed (outdated) » Needs review