Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#15456 closed defect (bug) (fixed)

Warnings in wp-admin/user-new

Reported by: petemall's profile PeteMall Owned by:
Milestone: 3.1 Priority: normal
Severity: normal Version: 3.1
Component: Warnings/Notices Keywords: has-patch
Focuses: Cc:

Description (last modified by ocean90)

  • NOTICE: wp-admin/user-new.php:231 - Undefined variable: new_user_email
  • Notice: wp-admin/user-new.php:20 - Undefined variable: s (PHP 5.3.2 specific)
  • Notice: Undefined property: stdClass::$blogid in wp-includes/ms-functions.php:1390 (add a user without sending mail)

Attachments (3)

user-new-form-updates.diff (4.4 KB) - added by wpdavis 14 years ago.
user-new-changes-updated.diff (4.2 KB) - added by wpdavis 14 years ago.
Removed the existing_user_ variables, as they actually aren't needed
user-new-notices-fix.patch (1.6 KB) - added by wpdavis 14 years ago.
Fixes sprintf to use single quotes, makes sure the user is submitting the multisite form before trying to add a user to the blog to avoid errors.

Download all attachments as: .zip

Change History (11)

#1 @wpdavis
14 years ago

  • Cc wpdavis added

The add user page needs a little bit of work to make it usable — working on a patch right now.

#2 @wpdavis
14 years ago

  • Keywords has-patch added; needs-patch removed

Patch attached.
Summary:
*removed the anchors, as the errors display above them, which is potentially confusing to the user.
*set the vars for adding existing users to blogs to false as default in order to avoid stepping on the other form's toes, filling in the wrong fields and errors when submitting the new user form
*made sure noconfirmation stays checked

@wpdavis
14 years ago

Removed the existing_user_ variables, as they actually aren't needed

#3 @ocean90
14 years ago

  • Component changed from Administration to Warnings/Notices
  • Description modified (diff)
  • Keywords needs-patch added; has-patch removed
  • Summary changed from Warning in wp-admin/user-new to Warnings in wp-admin/user-new

Some other warnings.

#4 @ryan
14 years ago

(In [16518]) Fixes for user-new.php. Props wpdavis. see #15456

@wpdavis
14 years ago

Fixes sprintf to use single quotes, makes sure the user is submitting the multisite form before trying to add a user to the blog to avoid errors.

#5 @wpdavis
14 years ago

  • Keywords has-patch added; needs-patch removed

Above patch should fix both the errors ocean90 was getting.

#6 @ryan
14 years ago

(In [16527]) Notice fixes for user-new.php. Props wpdavis. see #15456

#7 @ryan
14 years ago

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.