I have gone over all 4.7/cvs issues and collected together all pending patches, plus fixed a couple more bugs. New privatemsg.module file attached.

  • 46362 - Fix write to author links
  • 46623 - Fix display on user pages
  • 63208 - Fix warning on move to folder
  • 40626 - Fix msgto/{id} so username appears
  • 51378 - Fix hook_user problem, default _allow to on, _mailalert to off
  • 64757 - Fix if recipient can't use input format of msg
  • 49367 - Don't allow subject to be just whitespace
  • 47554 - Cache message count to avoid repeat queries
  • 47918 - Fix error handling (in HEAD now but not DRUPAL-4-7)
CommentFileSizeAuthor
privatemsg_6.module25.74 KBmindless

Comments

faruk’s picture

faruk commented

works great on my site :)

jlmeredith’s picture

jlmeredith
Location Denver, CO
commented

Installed on version 4.7.2 and still get the two p's stacked on top of each other in the user profile.

Any suggestions?

jlmeredith’s picture

jlmeredith
Location Denver, CO
commented

Here is an error message I get whenever browsing a node in the site:


    * warning: array_keys(): The first argument should be an array in /hsphere/local/home/themacge/wichitamoms.org/modules/user.module on line 349.
    * warning: implode(): Bad arguments. in /hsphere/local/home/themacge/wichitamoms.org/modules/user.module on line 349.
    * user warning: You have an error in your SQL syntax. Check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1 query: SELECT DISTINCT(p.perm) FROM role r INNER JOIN permission p ON p.rid = r.rid WHERE r.rid IN () in /hsphere/local/home/themacge/wichitamoms.org/includes/database.mysql.inc on line 120.
dakala’s picture

dakala
he/him
Primary language English
Location Fairford, Glos
commented

This patch seems to work fine. However, why should I see the 'send private message' link in my own profile? Or am I missing something? Cheers.

mindless’s picture

mindless commented

themacgeek, both your comments are problems in the original version that should be fixed with this patched version.
alagba, the module has never checked for this case, afaik.. wouldn't be too hard to add, feel free to submit your own patch :-)

mindless’s picture

mindless commented
Status: Needs review » Fixed

All fixes now in CVS/DRUPAL-4-7!

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)