Hi,
I have two different builds showing the same bug (and this is happening on android and iOS): when a user either creates a new account or logs in to the app with an existing account, and then clicks one of the buttons to open a webview on the associated drupal website, I get errors for any drupalgap modules I've installed on the build. So if a build has loggintobbogan installed, the user sees this error in the webview:
Warning: Parameter 1 to drupalgap_logintoboggan_drupalgap_site_settings() expected to be a reference, value given in module_invoke_all() (line 895 of home/my_username/public_html/includes/module.inc).

If multiple modules are enabled, I get multiple error messages of the same kind.

Worth noting: if I have one of the new supporting drupalgap modules enabled on the website (like drupalgap rate) but not on the app, it will also throw this error.

Edited to add: if the user logs out and then logs in again without rebuilding the app in the emulator, the error disappears.

Thanks for any advice you can lend.

Comments

eudaemonia’s picture

eudaemonia commented
Issue summary: View changes
tyler.frankenstein’s picture

tyler.frankenstein commented
Status: Active » Postponed (maintainer needs more info)

To clarify, you have a button in your app that opens up an external browser window for a page on your Drupal website? When that window opens the page on your Drupal site, you are seeing Drupal error messages?

Either way, I think this has to do with what PHP version is on your host.

For example, in the DrupalGap Login Toboggan module, try changing this line:

function drupalgap_logintoboggan_drupalgap_site_settings(&$names) {

To this:

function drupalgap_logintoboggan_drupalgap_site_settings($names) {

I think later versions of PHP (above the recommended version for D7), changed to pass by reference, so the '&' might not be needed on some sites. I'm not entirely sure that is accurate though. But please try that on your site to see if it prevents the error from displaying. If it does, see if Login Toboggan works inside the app (e.g. login with an e-mail address).

eudaemonia’s picture

eudaemonia commented

Bingo! Your clarification above was correct, and removing the & from the module solved the problem. Thanks again!

tyler.frankenstein’s picture

tyler.frankenstein commented
Status: Postponed (maintainer needs more info) » Closed (works as designed)