Red Hat Satellite engineering is moving the tracking of its product development work on Satellite to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "Satellite project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs will be migrated starting at the end of May. If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "Satellite project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/SAT-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1324967 - [RFE] Mark required fields conditionally based on other fields
Summary: [RFE] Mark required fields conditionally based on other fields
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.2.0
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: Unspecified
Assignee: satellite6-bugs
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-04-07 17:12 UTC by Og Maciel
Modified: 2018-07-17 19:19 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-07-17 19:19:32 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
Mark puppet environment as a required field (73.09 KB, image/png)
2016-04-07 17:12 UTC, Og Maciel 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 21678 0 None None None 2017-11-15 14:55:39 UTC

Description Og Maciel 2016-04-07 17:12:05 UTC 
Created attachment 1144819 [details]
Mark puppet environment as a required field

This screenshot shows that "Puppet Environment" is a required field for provisioning new hosts. However, the form does not flag this field as being required. Other required fields have an "*" next to them to denote them as being required.
Comment 4 Ivan Necas 2016-04-15 08:48:52 UTC 
The puppet environment is required field only when a puppet master is selected on the host. http://projects.theforeman.org/issues/7450. Given the current complexity of the host form UI, I don't think the gain for making sure we flag the environment as required only when puppet master is selected is worth the effort (even not sure it would get accepted upstream).

Given the context, I suggest keeping this for sat-future and addressing it as part of bigger UI facelift with some better javascript plumbing around.

Since it's not just a missed flag on obviously required field and there are reasons why it's not flagged by default, I don't agree on the blocker flag here.
Comment 5 Marek Hulan 2017-11-15 14:52:07 UTC 
I think this won't be implemented until we have a way of specifying required fields validations based on other fields selections. That sounds more like a new way of rendering forms. Also it should not be addressed just for this field but rather every field, e.g. subnet has some fields required for ipv4 only.

Converting to a more generic RFE
Comment 6 Marek Hulan 2017-11-15 14:55:36 UTC 
Created redmine issue http://projects.theforeman.org/issues/21678 from this bug
Comment 8 Bryan Kearney 2018-07-17 19:19:32 UTC 
Thank you for your interest in Satellite 6. We have evaluated this request, and we do not expect this to be implemented in product in the forseeable future. We are therefore closing this out as WONTFIX. If you have any concerns about this, please feel free to contact Rich Jerrido or Bryan Kearney. Thank you.
Note You need to log in before you can comment on or make changes to this bug.