hook_user changed in HEAD. Keeping up with the new array structure.

CommentFileSizeAuthor
#12 46623-privatemsg-user-profile-shows-p.patch2.29 KBwebchick
#6 privatemsg_2.module25.68 KBbrashquido
privatemsg.module_0.diff2.11 KBdrumm

Comments

Shiva Thomas’s picture

Shiva Thomas commented

Works for me. :-)

pol’s picture

pol
the/that
Location Brussels
commented

Subscribe msg ;)

I'll try at home !

kus’s picture

kus commented
Status: Needs review » Reviewed & tested by the community

works perfect.

ajwwong’s picture

ajwwong commented

Ditto. Patch works perfect for me too. Thx.

ultraBoy’s picture

ultraBoy commented

anybody commit it please

brashquido’s picture

brashquido commented
StatusFileSize
new privatemsg_2.module25.68 KB

Patched module file for those not familiar with patching files, or no access to a *nix OS or Cygwin install for Windows.

hectorplus’s picture

hectorplus commented
Title: Update for user profiles » When no userName is entered in the "To", an error occurs
Status: Reviewed & tested by the community » Needs work

This is the error i get when sending the message when there is no userName entered.
Fatal error: Call to undefined function: array2object() in /............/.../modules/privatemsg/privatemsg.module on line 475

fool2’s picture

fool2 commented

That is part of another error in the code. I really wish someone would commit to CVS so we had a new version to work with.

Also, maybe it's adviseable to use the ajax autocomplete that comes with the forms API now instead of the dropdown select.

MrEricSir’s picture

MrEricSir commented

This patch exposes another bug.

If you follow a link to message someone (i.e. privatemsg/msgto/1) the "To" field is never filled in.

This seems to occur in privatemsg_page(). Comment out the following lines and you're good to go:

//if (isset($edit['messages'])) {
    $msg = array_keys($edit['messages']);
  /*}
  else {
    $msg = array();
  }*/
Carlos Miranda Levy’s picture

Carlos Miranda Levy commented

Is the patch or the patched module listed here v4.7, cvs or 4.6?

MrEricSir’s picture

MrEricSir commented

It's for the CVS version of the module.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
StatusFileSize
new 46623-privatemsg-user-profile-shows-p.patch2.29 KB

Here's a patch for 4.7 (should also apply to CVS) with both drumm and MrEricSir's fixes.

mindless’s picture

mindless commented

I think the original patch attached here is correct, but the 2nd issue mentioned I think should be resolved by this patch rather than comment #9/12 here.

mindless’s picture

mindless commented
Title: When no userName is entered in the "To", an error occurs » User page shows "P" instead of send link
mindless’s picture

mindless commented
Status: Needs work » Reviewed & tested by the community

I have included the original patch in 67612.

mindless’s picture

mindless commented
Status: Reviewed & tested by the community » Fixed
Anonymous’s picture

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