Project

General

Custom queries

Profile

Actions

Bug #5897

closed

Missing user mail address should not be required for admin updating a user

Added by Dominic Cleal almost 11 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Authentication
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1099988
Description of problem:
When creating a user the --mail option is not required but when updating it is.

Version-Release number of selected component (if applicable):
hammer (0.1.0) * hammer_cli_foreman (0.1.0) * hammer_cli_katello (0.0.3)

How reproducible:
Aways

Steps to Reproduce:
1. # hammer user create --login testuser --password userpasswd --auth-source-id 1
User created
2. # hammer user list --per-page 10000 | grep testuser
48 | testuser | |
3. # hammer user update --id 48 --login usertest
Could not update the user:
Email address can't be blank
4. # hammer user update --id 48 --login usertest --mail
User updated
5. # hammer user list --per-page 10000 | grep "48 "
48 | usertest | |
Actual results:

Expected results:
If the --mail is required on creation then should ensure that it is filled on update. But if it is not required on creation it should be not required when updating.

Additional info:


Related issues 1 (0 open1 closed)

Related to Foreman - Refactor #10946: User mail address should either be always optional or always requiredClosedActions
#1

Updated by Dominic Cleal almost 11 years ago

  • Category set to Authentication
  • Assignee deleted (Dominic Cleal)
#2

Updated by Tom Caspy over 10 years ago

  • Assignee set to Tom Caspy
#3

Updated by The Foreman Bot over 10 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2065 added
  • Pull request deleted ()
#6

Updated by Tom Caspy about 10 years ago

  • Status changed from Ready For Testing to Assigned
  • Pull request added
  • Pull request deleted (https://github.com/theforeman/foreman/pull/2065)
#7

Updated by The Foreman Bot about 10 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2244 added
  • Pull request deleted ()
#8

Updated by Dominic Cleal almost 10 years ago

  • Subject changed from User mail address should either be always optional or always required to Missing user mail address should not be required for admin updating a user
  • Translation missing: en.field_release set to 35
#9

Updated by Dominic Cleal almost 10 years ago

  • Translation missing: en.field_release deleted (35)
#10

Updated by Dominic Cleal almost 10 years ago

  • Related to Refactor #10946: User mail address should either be always optional or always required added
#11

Updated by Dominic Cleal over 9 years ago

  • Translation missing: en.field_release set to 63
#12

Updated by Tom Caspy over 9 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100
Actions

Also available in: Atom PDF