Fixing rendering logic for visibility component

Fixes #4845

Prior to this fix, if we had a new record that either set an embargo or
a lease, we would narrow our rendering of the visibility component to
only a partial related to the embargo or lease.

```gherkin
Given a form that set an embargo (or release date)
And that form had invalid data
When I submitted the form
Then the response form would inidicate there was invalid data
And would render only the portion of the visibility related to
  embargo (or release date if set)
```

With this change, the form renders the entire visibility component,
letting the user change their visibility options (e.g., go from with
Lease to Public or what have you).  However, the error messaging is not
providing insight into the problem; That is to say there's a red flash
error, but the content is empty; and there is no indicator on the lease
field that there existed an error.  This warrants further work, and will
be a commit that builds on this change.

app/views/hyrax/base/_form_visibility_component.html.erb

@@ -1,6 +1,6 @@
-<% if embargo_enforced?(f.object) %>
+<% if embargo_enforced?(f.object) && !f.object.new_record? %>
   <%= render 'form_permission_under_embargo', f: f %>
-<% elsif lease_enforced?(f.object) %>
+<% elsif lease_enforced?(f.object) && !f.object.new_record? %>
   <%= render 'form_permission_under_lease', f: f %>
 <% else %>
     <fieldset>