Even though hook_userpoints() appears to be deprecated (although the nodes and comments integration module is using it) and even though hook_userpoints_info() hasn't been implemented, it appears that Userpoints integration is working. To be future-proof, however, we should look into the API changes in Userpoints 7.x.

CommentFileSizeAuthor
#1 userpoints-integration-uses-deprecated-api-1685442-1.patch953 bytesmathankumarc

Comments

mathankumarc’s picture

mathankumarc commented
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new userpoints-integration-uses-deprecated-api-1685442-1.patch953 bytes

Other than hook_userpoints_info(), I think there is no new change in userpoints. This information is used in "Add a transaction"

Here is the patch.
TODO: Improve the desribtions

Even though hook_userpoints() appears to be deprecated

I don't think so.

icecreamyou’s picture

icecreamyou commented
Status: Needs review » Needs work

Okay cool.

+++ b/submodules/fbss_userpoints/fbss_userpoints.module
@@ -6,6 +6,26 @@
+      'description' => t('User posted a new statuses.'),

"User posted a new status update."

+++ b/submodules/fbss_userpoints/fbss_userpoints.module
@@ -6,6 +6,26 @@
+    'statuses add other' => array(
+      'description' => t('User wrote a message to another recipient.'),
+    ),
+    'statuses add other' => array(
+      'description' => t('User wrote a message at another recipient.'),
+    ),

Duplicate?

+++ b/submodules/fbss_userpoints/fbss_userpoints.module
@@ -6,6 +6,26 @@
+      'description' => t('User pposted a new status comment.'),

"User posted a new status comment." (You had two P's in "pposted")

This is pretty trivial, so RTBC from me as soon as these descriptions are fixed. Thanks.

mathankumarc’s picture

mathankumarc commented
Status: Needs work » Fixed

Will review the patch before uploading it :)

Committed a fix dev.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.