Hello, first of all many thanks for a great module!

I created a menu tab with page manager and used context admin and the "Create user" page type. I have found two bugs with this.

First issue:
If I put an image field on the user registration form, using context admin I get these errors when uploading the image:

Notice: Undefined index: context_admin_user_create_menu_render_form in drupal_retrieve_form() (line 765 of /includes/form.inc).
Warning: call_user_func_array() expects parameter 1 to be a valid callback, function 'context_admin_user_create_menu_render_form' not found or invalid function name in drupal_retrieve_form() (line 800 of /includes/form.inc).
Notice: Undefined index: field_profile_image in file_ajax_upload() (line 271 of /modules/file/file.module).
Notice: Undefined index: #suffix in file_ajax_upload() (line 280 of /modules/file/file.module).

This works fine when registering a new user through drupals default "admin/people/create" interface so I'm guessing it's an issue with this module. It might be part of this? #1162890: File Upload AJAX Failure in Local Action Create Node

Second issue:
If I delete the image field and proceed with creating a new user I get these errors:

Notice: Undefined index: administer_users in user_register_submit() (line 3762 of /modules/user/user.module).
Unable to send e-mail. Contact the site administrator if the problem persists.

The user is created but the password I filled in the form is not saved, so the new user can't login. Also, apparently drupal is trying to send an email although the "user notification" setting is not checked, it also happens even if this setting is not included.

Very thankful for some help on these issues, perhaps I'm doing something wrong in my setup?

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

JJSON’s picture

Version: 7.x-1.1 » 7.x-1.x-dev

Perhaps I should have mentioned; I have found the same bugs on the recommended release and the dev version.

Deciphered’s picture

Status: Active » Needs review
FileSize
807 bytes

Patch for the second issue attached.

Haven't run into the 1st issue yet.

drakythe’s picture

New patch for the 2nd issue.

Re-rolled to be against current code base, also just passing TRUE instead of checking Drupal permissions, because we should be using page manager for our access checks.

devenpro’s picture

+1 to #3

Deciphered’s picture

@studfier,

Unfortunately a '+1' has no actual effect, if you have reviewed the patch and manually tested it and it achieves what it set out to achieve without introducing other issues, and you feel it is ready to be committed, please mark the issue status as 'reviewed & tested by the community'.

devenpro’s picture

Status: Needs review » Reviewed & tested by the community

Notice: Undefined index: administer_users in user_register_submit() (line 3795 of /home/----/modules/user/user.module).
Has gone on my live site and also on a fresh installation at http://simplytest.me.

SocialNicheGuru’s picture

this has not worked for me

Deciphered’s picture

@SocialNicheGuru,

If you are certain that this doesn't work for you, change the status this issue and provide notes on how and why it didn't work for you. Just making a statement as per the above is not helpful, especially after a claim by another using stating it works on an existing and a freshly install simplytest.me site, it's possible that what isn't working for you isn't related to this issue, or that there are other factors in play, all of which can't be determined without further information from yourself.

SocialNicheGuru’s picture

this does work. conflict with another module.

EclipseGc’s picture

Which module is it conflicting with?

Eclipse

jeni_dc’s picture

This is already RTBC but I just applied the patch in #3 to 7.x-1.x-dev and it applied cleanly, the error is no longer appearing and the correct email settings were used. I have not seen any other errors or conflicts with other modules.