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

jredding’s picture

Assigned: Unassigned » jredding

The 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?

fhelmschrott’s picture

i 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)

jredding’s picture

hhm.. Ok I think I know what the problem is. Check back next week for a patch

fockez’s picture

i have the same problem

ontoligent’s picture

The 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.

kbahey’s picture

Assigned: jredding » kbahey
Status: Active » Fixed

Fixed 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.

fockez’s picture

i uninstall the older one and install the latest dev version
still invalid for my points

Anonymous’s picture

Status: Fixed » Closed (fixed)
fhelmschrott’s picture

Status: Closed (fixed) » Postponed (maintainer needs more info)

this 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.

debsjenkins’s picture

This 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?

debsjenkins’s picture

This 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?

C4-1’s picture

Ä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 ...



function userpoints_transaction_tools_menu($may_cache) {
  $items = array();
  if (!$may_cache) {
	global $user;
    $items[] = array(
      'path' => 'userpoints/myuserpoints/'.$user->uid.'',
      'title' => t('My') . ' ' . t('!Points', userpoints_translation()),
      'callback' => 'userpoints_transaction_tools_my_userpoints',

....

misterlawrence’s picture

I have the same issue. Will await upload of updated module tar. Thanks!

funana’s picture

I 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...

funana’s picture

sorry... 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...

miche’s picture

Hello. Is there a formal solution yet? I just installed it today and am having the same problem.

jredding’s picture

This 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.

jredding’s picture

Assigned: kbahey » jredding
Status: Postponed (maintainer needs more info) » Fixed

issue fixed in cvs version DRUPAL-5--3.. no official release. yet.

Anonymous’s picture

Status: Fixed » Closed (fixed)