Closed (fixed)
Project:
Mobile Codes
Version:
7.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
8 Jun 2011 at 15:40 UTC
Updated:
28 Aug 2012 at 02:31 UTC
Jump to comment: Most recent file
Comments
Comment #1
Tony Stratton commentedI'm getting the same error in 7.x-2.0
Comment #2
roam2345 commentedchange line 9 of mobile_codes.admin.inc to
that will only get you the first setting the rest of the form is still missing.
Comment #3
rocbrook commentedan update to jucallme's reply
Edit "sites/all/modules/mobile_codes/includes/mobile_codes.admin.inc"
Just remove the "&" inside
function mobile_codes_settings_form(&$form_state) {on Line 9As of this date there is only one option there > "URL settings"
Comment #4
rplescia commented#3 worked for me
Comment #5
eMPee584 commentedThat function doesn't even need NOR GET the form_state.
And there was another of these call-by-reference issues found when enabling the module, patch attached, please review. :)
Comment #6
rellimevad commentedpatch in #5 worked for me
Comment #7
mccune commentedPatch in #5 also worked for me.
Hope this module gets updated.
I use this module for my website so I can attach QR-Codes instead of regular attachments.
Comment #8
socialnicheguru commentedlooks like this was already committed to dev
Comment #9
paulhudson commentedhummm, I couldn't see it in the current dev release.
Drupal 7 doesn't support passing by reference anymore as it was depreciated in php 5.3 - Perhaps preaching to the converted there but that's the history to it anyway. :-)
Comment #10
socialnicheguru commentedThe new dev from 6/25 does not have the patch in #5 in it.
Comment #11
decipheredFixed and committed.