Comments

brightbold’s picture

Title: Port to Drupal 7? » Port to Registration Codes to Drupal 7?

I am also wondering if you have plans to port to D7. (And I changed the issue title to distinguish it from other "port to D7" issues on people's dashboards.)

davycw’s picture

subscribe

sw3b’s picture

subscribe

scthomps312’s picture

subscribe

aidanlis’s picture

I will be porting this to D7 but not in the next few weeks unfortunately.

rpsu’s picture

subcribe

verynic’s picture

+1

Anonymous’s picture

subscribe

aidanlis’s picture

The regcode core module is now D7 enabled, though I've only tested the generation and viewing of codes.

chertzog’s picture

I created a new code, and tried to register with that code, but i get this error:

Fatal error: Call to undefined function dvm() in /sites/all/modules/regcode/regcode.module on line 312

After looking at api.drupal.org, it seems that dvm was exclided from drupal 7.

http://api.drupal.org/api/devel/devel.module/function/dvm/6

I will be glad to help with the process of porting and testing, but i have no idea what to do here.

aidanlis’s picture

Just delete the line with dvm on it, it shouldn't be there ... also making empty promises to help port when you don't know what dvm is is not helpful.

chertzog’s picture

im just learning php, and drupal. i have helped port a couple things already. most of what i can do is help test. which is what i said.

znerol’s picture

Attached is a serie of patches fixing validation on user registration form as well as on the regcode_load function.

znerol’s picture

Fix property assignement in regcode_load.

aidanlis’s picture

znerol you are a star. I will review and apply these patches the second I get back (in two days).

modoq’s picture

subscribing

sw3b’s picture

any news ?

aidanlis’s picture

Patches have been applied. znerol gets a gold star - thank you for the contribution.

There's still a few critical pieces missing before I can release a beta:
#1146132: Registration code field doesn't show up on User Profile Edit page or on a tab for the profile
#1146150: Call to function that doesn't exist: db_fetch_object()
#1145230: Database error when trying to enable "Code mailer" option

I don't have time to work on this module and don't expect to have any time in the next six months, so community contributions are welcome.

znerol’s picture

Thanks for applying the patches. Could you please rollback your last changeset? You introduced syntax errors (missing semicolons) and SelectQuery::join does not work like one might expect: #1147950: SelectQuery->addJoin method is not chainable.

aidanlis’s picture

Jesus I'm not sure how that escaped testing - apologies for the late night refactoring.

znerol’s picture

@aidanlis: Use git am for applying patches to the git repository. Like this you don't have to copy-paste commit messages etc.

znerol’s picture

Another series of patches fixing the voucher tab on user profiles. It would probably be wise to refactor the code-field into an element because i've just duplicated the validation code from the signup form element... Might do that later.

znerol’s picture

Another one for regcode_dynamic. Touched up the UI for drupal 7, use machine_name instead of rule_id and use only one variable to store settings in it. I've not worked out yet if the code-validation really works.

aidanlis’s picture

Status: Active » Fixed

Thanks largely to znerol, regcode has been ported to Drupal 7. I've just released a 1.0-beta1!

If there's any bugs please open them as separate issues.

Status: Fixed » Closed (fixed)

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