Uploaded image for project: 'Blesta Core'
  1. Blesta Core
  2. CORE-1236

Hide labels for hidden service fields

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 3.2.0-b3
    • Fix Version/s: 3.2.1
    • Component/s: Modules
    • Labels:
      None

      Description

      On order forms, labels for hidden fields are displayed. The page should appear as if there is no hidden field, by not rendering the label or adding additional space.

        Issue Links

          Activity

          admin Paul Phillips created issue -
          Hide
          admin Paul Phillips added a comment -

          On the order form the "Configurable Options" h4 heading also appears if there are only hidden fields set for the Universal Module product. It would be nice if this heading didn't appear if there are no visible fields.

          Show
          admin Paul Phillips added a comment - On the order form the "Configurable Options" h4 heading also appears if there are only hidden fields set for the Universal Module product. It would be nice if this heading didn't appear if there are no visible fields.
          tyson Tyson Phillips (Inactive) made changes -
          Field Original Value New Value
          Summary Universal Module: Do not display labels for hidden fields Hide labels for hidden service fields
          tyson Tyson Phillips (Inactive) made changes -
          Assignee Cody Phillips [ cody ] Tyson Phillips [ tyson ]
          tyson Tyson Phillips (Inactive) made changes -
          Issue Type Improvement [ 4 ] Bug [ 1 ]
          Hide
          tyson Tyson Phillips (Inactive) added a comment -

          Applies to:
          Add/edit service (admin)
          Add/edit package (admin)
          Add addon (admin/client)

          Show
          tyson Tyson Phillips (Inactive) added a comment - Applies to: Add/edit service (admin) Add/edit package (admin) Add addon (admin/client)
          tyson Tyson Phillips (Inactive) made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          tyson Tyson Phillips (Inactive) made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          tyson Tyson Phillips (Inactive) made changes -
          Link This issue relates to CORE-1266 [ CORE-1266 ]
          Hide
          cody Cody Phillips (Inactive) added a comment -

          Needs to extract hidden fields and render them first, then skip all hidden fields when rendering all other fields.

          Or, perhaps much better solution would be to automatically set the order of the fields when invoking ModuleFields::getFields(), so the views don't need to be updated?

          Show
          cody Cody Phillips (Inactive) added a comment - Needs to extract hidden fields and render them first, then skip all hidden fields when rendering all other fields. Or, perhaps much better solution would be to automatically set the order of the fields when invoking ModuleFields::getFields(), so the views don't need to be updated?

            People

            • Assignee:
              tyson Tyson Phillips (Inactive)
              Reporter:
              admin Paul Phillips
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                25/Jun/14