Reviewed & tested by the community
Project:
Signup
Version:
7.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
3 Mar 2012 at 04:49 UTC
Updated:
16 Feb 2017 at 17:40 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
cedeweyI applied the patch and everything seems to be working fine for me. Thanks for doing this!
Comment #2
dwwThanks for contributing a patch to fix a problem! Since it's your first, I'll give you some feedback to improve for other patches you post. Instead of leaving the dead code commented out, the patch should just remove those lines entirely.
Thanks!
-Derek
Comment #3
generalconsensus commentedPoint well noted! Thanks Derek
Comment #4
deanflory commentedAdding first part of error to make it easier for others to find via search:
Notice: Undefined index: limit in signup_admin_form_after_build() (line 173 of
Notice: Undefined index: status in signup_admin_form_after_build() (line 174 of
The patch seemed to work for me, at least the errors went away. Thanks!
Comment #5
jerenus commentedI think we may need this patch.
Comment #6
generalconsensus commentedAgreed Jerenus. My original patch was poorly formatted.
Comment #7
kebz commentedI'm getting this notice and all I did was click on the "sign up" tab. I have another site with this module and I'm not getting any notices at all on that site.
Comment #8
generalconsensus commented@Kebz -- try applying the patch from #5.
Here is how to apply a patch https://www.drupal.org/patch/apply
Comment #9
kebz commentedThanks @generalconsensus
So far so good. I'm not seeing error messages when I click on the tab
btw, I manually apply my patches... for me, it's easier and I'm able to see what's being removed and/or added.
I of course duplicate the original file first before applying any patch, this way, it's easier to revert back.
Thanks again =)
Comment #10
jacob.embree commented#5 does the trick.
Comment #11
pietrocap commented#5 works for me too.