user hook would not display correctly.

Attached patch updates store_user to return the data in the correct format.

CommentFileSizeAuthor
#3 auction.patch848 byteswpd
#2 file_3.patch954 byteswpd
#1 store_8.patch992 byteswpd
store_6.patch981 byteswpd

Comments

wpd’s picture

StatusFileSize
new992 bytes

This patch makes this:

History

Member for
1 day 22 hours
s
s

Look like this:

History
Downloads
View your downloads
Orders
View your order history

Re-rolled patch for head.

wpd’s picture

StatusFileSize
new954 bytes

same fix for file.module

wpd’s picture

StatusFileSize
new848 bytes

And the same fix for auction.module

The same bug exists in stores.module, but I have not messed with that module at all (so I have not patched it).

neclimdul’s picture

duplicate of http://drupal.org/node/51558 with patches for other modules. Going to mark the older one duplicate since this has more fixes. file module patch is exactly the same for both though.

guckie’s picture

Just to report in on what I've tried out of these patches so far, store_8.patch has corrected the problem with the s's on the user account screen. Now the link to order history correctly appears there instead. In about a week of fairly heavy testing since I applied the patch, I haven't turned up any unwanted side-effects, so I think this patch is probably ok for inclusion.

ajk’s picture

buddylist.module seems to have the same problem.

I fixed it by changing this :-

  function buddylist_user($type, &$edit, &$thisuser, $category = NULL)
    global $user;

to

  function buddylist_user($type, &$edit, &$thisuser, $category = NULL)
    global $user; $output = array();

--AjK

wpd’s picture

Status: Needs review » Reviewed & tested by the community

change status based guckie's comments/review

neclimdul’s picture

I agree. Store patch already made it in cvs http://drupal.org/cvs?commit=27219 just the other patches actually need inclusion.

gordon’s picture

Status: Reviewed & tested by the community » Fixed

Commited to cvs, thanks.

Anonymous’s picture

Status: Fixed » Closed (fixed)