Ubercart Userpoints Port to Drupal 7

+1 subscribing - I really need this module to move to D7 as well.
Already testing D7 alpha 2 =)

Hey Trupal218 and yaz085.
I won't look into a drupal7 port very soon, because i firstly will rewrite the d6 version.
The d6 version is quite bad code to be honest, and i made the decision to rewrite it firstly before we go on.
Think it will be the best way.

Anyone willing to help with the rewrite of the d6 version is offcoarse always welcomed ;)

thanks betz! i look forward to following your work!

hi betz

i'm available to help with testing when you begin rewriting the module
please let me know when a version becomes available

thanks!

Subscribing

@bsmith451 - yes your post probably was rude since you did you even read the issue first, the maintainer postponed this issue and said they are still working on D6 in #3.

Well, I think/hope that D6 is at a point where most of the bugs are resolved. I am waiting on confirmation on two still that I want to have a path forward/answer to before moving on. I am also thinking D7 will be a complete rewrite to clean up several things and bring in some new features. What I am wondering is if people would rather have a D7 release or keep tweaking and adding features to D6?

I am agree with yaz085, it will take at least a good year before drupal 7 really works with port of modules.

subscribing

+1

Great job mariano.barcia -- will test and see what could be fixed. I was going to start a port of my own this week, but I'll try and work of this one. :)

thx mariano!

Hi,

I got uc_userpoints_product.module working, so I'm sending the attached zip with the files as I have them in my install.

I had to introduce modifications to the way it added or updated a product feature to the DB. I think it inserted new pfids on updates, but I didn't have time to verify the D6 version, I just got the D7 working on my install.

For those who had the previous version, just download this new one and overwrite the previous version. There's no need for an upgrade since the product module didn't work at all when I tried it, so I guess this new version is all goodness :-).

The module uc_userpoints_seller from #25 seems not to be working because conditional actions module was removed from Ubercart. I am now working on the uc_userpoints_seller module to make it up-to-date.

Several new features in uc_userpoints_seller.module:
* You can now choose types of products which authors you want to award with userpoints
* You can now select the category of awarded userpoints
* You can now customize the transaction comment, and tokens can be used in these comments

If you already installed the version from my previous archive, please don't forget to activate module rule in Rules! In this version it is active by default.

I thought everything was updated but there was a newer version of Entity API that fixed this problem. Everything looks like it's working really well! Thanks.

Just installed the last version(#29) and getting the following error:

Everything seems to be functional, but i am getting errors/notices all over the place:

Notice: A non well formed numeric value encountered in uc_currency_format() (line 997 of C:\xampp\htdocs\drupal\sites\all\modules\ubercart\uc_store\uc_store.module).
Notice: A non well formed numeric value encountered in uc_currency_format() (line 997 of C:\xampp\htdocs\drupal\sites\all\modules\ubercart\uc_store\uc_store.module).

subscribing! This is the only thing holding me back from a site move to D7

Hi I installed the latest "userpoints_ubercart-D7-June4-01.zip" and when I proceed to checkout the page just died. It's a almost-blank page with my site name, and a "Checkout" word on the page. After disabling this module, the checkout page just turned back to the original correct one. Any help or fix?

sub

Subscribing!

Thanks for posting - I just installed and will start looking at it.

It looks like there may be a mismatched paren at line 214 in uc_userpoints_award.module which is causing some of the error messages cited above.

I was getting a complaint about USERPOINTS_AWARD_CATEGORY in uc_userpoints_award.module until I added

define('USERPOINTS_AWARD_CATEGORY', 'userpoints_award_category');

at line 10. I just guessed about what would be a good value. Will see if it burns me later.

I'm looking at the "Notice: A non well formed numeric value encountered in uc_currency_format() (line 866" problem - evidently the string "[Points branding name]:[number of points]" is being passed instead of a numeric value. I'll see if I can find out why....

Looks like the problem is in uc_userpoints_payment.module in lines 136-140. $node->content['display_price']['#value'] expects a number but it's being passed a string complete with HTML formatting.

@legendm33066: have you noticed the green "Follow" button in the upper-right corner here? Use that if you only want to "subscribe" (ref. "Stop subscribing, start following": http://drupal.org/node/1306444 )

Oh I know that but I am just used to subscribing :) Will remember that next time

Recently upgraded to

Drupal 7.9
Rules 7.x-2.0
ubercart 7.x-3.0-rc2
userpoints 7.x-1.0

Sad to report that this seems to completely break the D7 patch, which until now has been working well. The order remains pending and no user points get assigned.

I noticed that the Rules module reported problems on its upgrade - and I think this may have something to do with it. The log error messages ( issued by rules ) where

The reaction rule Update order status on full payment (uc_payment_received) fails the integrity check and cannot be executed. Error: Unknown condition uc_payment_condition_order_balance.

The reaction rule Complete non-shippable order after payment received (uc_checkout_complete_paid) fails the integrity check and cannot be executed. Error: Unknown condition uc_order_condition_is_shippable.

The reaction rule Notify customer when a role is renewed (uc_role_notify_renew) was marked dirty, but passes the integrity check now and is active again.

The reaction rule Notify customer when a role is renewed (uc_role_notify_renew) fails the integrity check and cannot be executed. Error: Data selector expiriation for parameter expiration is invalid.

Anyone any idea on what needs to be done to fix this up?

EDIT: Fixed Itself! I think some of the errors listed above come from deprecated D6 stuff that might still be hanging around in the patch, but had no significance. Visiting the rules and resaving them seemed to help. I guess things just got a bit confused in the updates.

I think this module has to get a rewrite from a ground up for D7. I've posted a topic here, so if anyone would like to share the costs to have a developer to develop this for us, feel free to contact me and let me know.