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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cedewey’s picture

Status: Active » Reviewed & tested by the community

I applied the patch and everything seems to be working fine for me. Thanks for doing this!

dww’s picture

Status: Reviewed & tested by the community » Needs work

Thanks 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

generalconsensus’s picture

Point well noted! Thanks Derek

deanflory’s picture

Adding 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!

Jerenus’s picture

Status: Needs work » Needs review
FileSize
1.19 KB

I think we may need this patch.

generalconsensus’s picture

Agreed Jerenus. My original patch was poorly formatted.

Kebz’s picture

Issue summary: View changes

I'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.

Notice: Undefined index: status in signup_admin_form_after_build() (line 174 of .../sites/all/modules/signup/includes/admin.signup_administration.inc).

Backtrace:
signup_admin_form_after_build(Array, Array) form.inc:1914
form_builder('signup_admin_form', Array, Array) form.inc:885
drupal_process_form('signup_admin_form', Array, Array) form.inc:385
drupal_build_form('signup_admin_form', Array) form.inc:130
drupal_get_form('signup_admin_form') admin.signup_administration.inc:12
signup_admin_page()
call_user_func_array('signup_admin_page', Array) menu.inc:517
menu_execute_active_handler() index.php:21

generalconsensus’s picture

@Kebz -- try applying the patch from #5.

Here is how to apply a patch https://www.drupal.org/patch/apply

Kebz’s picture

Thanks @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 =)

jacob.embree’s picture

Status: Needs review » Reviewed & tested by the community

#5 does the trick.

pietrocap’s picture

#5 works for me too.