Closed (fixed)
Project:
User Points
Version:
5.x-2.14
Component:
Code: userpoints
Priority:
Normal
Category:
Support request
Assigned:
Reporter:
Created:
5 Jul 2007 at 06:23 UTC
Updated:
6 Oct 2007 at 09:11 UTC
The link "My Points" automatically put into the menu isn't working. It points to userpoints/myuserpoints but doesn't redirect to the path with the user id (userpoints/myuserpoints/$user_id) which would be the correct path.
Comments
Comment #1
jredding commentedThe path ( /userpoints/myuserpoints/ ) should default to showing the currently logged on user's points and navigating to /userpoints/myuserpoints/$user->uid shouldn't be necessary. I checked my local installation and it is working this way. However I'll take another look at the logic and submit a patch if necessary.
What is shown on that page for you? anything?
Comment #2
fhelmschrott commentedi get an empty table. this was also reproduced on two other installations so should be a bug indeed. Probably a redirect would to .../$userid would be nice to keep it consistant and alway have the same content on one url only (though this isn't puclicly available)
Comment #3
jredding commentedhhm.. Ok I think I know what the problem is. Check back next week for a patch
Comment #4
fockez commentedi have the same problem
Comment #5
ontoligent commentedThe problem is that the code contains various mysql_* functions which should be replaced with db_* ones. Do a such and replace on the module file for a quick fix.
Comment #6
kbahey commentedFixed in 5.x and HEAD.
If someone can checkout the Drupal 5 version from CVS and test it and find it to be working, I will create an official release.
Comment #7
fockez commentedi uninstall the older one and install the latest dev version
still invalid for my points
Comment #8
(not verified) commentedComment #9
fhelmschrott commentedthis was closed by anonymous? how could it?
It doesn't seem to be fixed according to the last poster. Anyone yet into that? I'm currently not using the lastest version due to some other problems.
Comment #10
debsjenkins commentedThis link isn't working for me either - I checked through the code and it DOES have db_* (not mysql_* as was suggested as the problem) - so maybe it's something else?
Comment #11
debsjenkins commentedThis link isn't working for me either - I checked through the code and it DOES have db_* (not mysql_* as was suggested as the problem) - so maybe it's something else?
Comment #12
C4-1 commentedÄhm i got the same Problem ...
I simply changed the function userpoints_transaction_tools_menu a little bit. (userpoints_transaction_tools.module)
But this is just provisional ...
Comment #13
misterlawrence commentedI have the same issue. Will await upload of updated module tar. Thanks!
Comment #14
funana commentedI don't know exactly why and how, but #12 submitted by C4 seems to change the links to the user profiles in the users by points list too...
Comment #15
funana commentedsorry... posted too quick. It doesnt have to do with the patch, it's a general issue.
I would expect the links to point to the user profiles...
Comment #16
micheHello. Is there a formal solution yet? I just installed it today and am having the same problem.
Comment #17
jredding commentedThis will be fixed in the upcoming version 3.0 which should be released soon.
This issue is not the that URL should go to /userpoints/myuserpoints/UID
userpoints/myuserpoints will display the correct user IF the transaction_tools modules is called AFTER userpoints. Unfortunatley U is after T alphabetically thus transactions_tool is called BEFORE userpoints. The weight of module is shifted to userpoints + 2 (default 2). This should all be handled within the .install file BUT there is an issue with the .install file and the weight isn't being assigned correctly. You can fix this by grabbing the moduleweights modules and manually assigning a heavier weight to transaction_tools (i.e. if userpoints is 0 make transaction_tools 2).
In version 3.0 transaction_tools will be merged into the core of userpoints so this won't be a problem.
Comment #18
jredding commentedissue fixed in cvs version DRUPAL-5--3.. no official release. yet.
Comment #19
(not verified) commented