progressivetech / net.ourpowerbase.qrcodecheckin

QRCode Checkin allows you to send an email that contains a scanable code to the registered participants for your event.
Other
16 stars 15 forks source link

Install / setup error #1

Closed magnolia61 closed 5 years ago

magnolia61 commented 6 years ago

Hi there, Love the idea of this extension. Tried to install but got an network error: Network Error Unable to reach the server. Please refresh this page in your browser and try again. trying to acces the event infomation (in order to configure)

The drupal watchlog says: Error: Using $this when not in object context in qrcodecheckin_civicrm_buildForm() (line 199 of /var/www/vhosts/xyz/webroot/sites/all/modules/civicrm_extensions/net.ourpowerbase.qrcodecheckin/qrcodecheckin.php).

I'm using CiviCRM 5.6.0 with Drupal 8.5.9 on PHP7.1. Anything you think I could try or help test with?

jmcclelland commented 6 years ago

Thanks for the feedback! I'm not sure how that $this slipped in there.

I just pushed a fix for it, but have not yet published it. Can you please test it and let me know if fixes your problem?

You can test the fix by editing the qrcodecheckin.php file in the top level of the qrcode checkin extension directory.

Go to line 199, which currently looks like this:

if (CRM_Utils_Request::retrieve('snippet', 'String', $this) == 'json') {

And change: $this to $form. It should look like this:

if (CRM_Utils_Request::retrieve('snippet', 'String', $form) == 'json') {
arake commented 5 years ago

Thanks, it worked for me. Version 1.03 fixes it.

jmcclelland commented 5 years ago

Excellent - thanks for the feedback.