Project

General

Profile

Bug #6693

UI shouldn't switch to other tab while creating OS parameters with same name or with blank name/values

Added by Ori Rabin over 4 years ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Web Interface
Target version:
Difficulty:
Triaged:
Bugzilla link:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1120663
Description of problem:
I added OS parameter with blank name/value and UI raises error on submit but as soon as I submit the form, UI switched to main(Operating System) tab. It should stay on same 'Parameters' tab since error is raised under it.

Version-Release number of selected component (if applicable):
sat6 GA snap0

How reproducible:
always

Steps to Reproduce:
1. create OS with parameters name/value as blank
2. submit it
3.

Actual results:
when you submit the form with blank name/value for OS parameters, neither the form was submitted nor you will see any error. Actually error raised but it displays only under 'Parameters' tab. Also, on submit the form, UI quickly switches from 'parameters' tab to 'operating system' tab

Expected results:
Since UI raised error so, It should stay on same page/tab where error was raised.

Additional info:


Related issues

Related to Foreman - Bug #6695: Adding two OS parameters with same name doesn't raise any error when values are differentClosed2014-07-20

Associated revisions

Revision 2b5f9c11 (diff)
Added by Ori Rabin over 4 years ago

Fixes #6693 - set parameters tab to error state and focus when validation fails

History

#1 Updated by Dominic Cleal over 4 years ago

  • Category set to Web Interface

#2 Updated by Dominic Cleal over 4 years ago

  • Related to Bug #6695: Adding two OS parameters with same name doesn't raise any error when values are different added

#3 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Target version set to 1.8.0
  • Pull request https://github.com/theforeman/foreman/pull/1611 added
  • Pull request deleted ()

#4 Updated by Dmitri Dolguikh over 4 years ago

  • Target version changed from 1.8.0 to 1.7.5

#5 Updated by Dominic Cleal over 4 years ago

  • Legacy Backlogs Release (now unused) set to 10

#6 Updated by Ori Rabin over 4 years ago

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

Also available in: Atom PDF