Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Notice: Undefined index: status in signup_admin_form_after_build() error at /node/add/project-issue/signup. I have included my first Drupal patch, so be nice :). Decided to remove the variables and set the full values along with the indices within foreach loop.
Comment | File | Size | Author |
---|---|---|---|
#5 | signup-signup_administration_undefined_index-1465930-5.patch | 1.19 KB | Jerenus |
admin.signup_administration.inc_.patch | 927 bytes | generalconsensus |
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 CreditAttribution: generalconsensus commentedPoint well noted! Thanks Derek
Comment #4
deanflory CreditAttribution: 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 CreditAttribution: Jerenus commentedI think we may need this patch.
Comment #6
generalconsensus CreditAttribution: generalconsensus commentedAgreed Jerenus. My original patch was poorly formatted.
Comment #7
Kebz CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: jacob.embree at St. Louis Integration commented#5 does the trick.
Comment #11
pietrocap CreditAttribution: pietrocap commented#5 works for me too.