Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-7833

Redundant options at select filter on find screens

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: ALL COMPONENTS
    • Labels:
      None

      Description

      On some of the find screens, where we have select drop down as a search filter, there are two options : Blank (No Value) and Any. Functional meaning of these two options are same. We can remove blank option and use 'Any' option wherever applicable to avoid redundancy and for consistency throughout the application.

      For more clarification please refer the attached screenshot.

      1. OFBIZ-7833.patch
        9 kB
        Renuka Srishti
      2. RedundantOptions.png
        297 kB
        Renuka Srishti
      3. RedundantOptions.png
        259 kB
        Deepak Nigam

        Activity

        Hide
        pfm.smits Pierre Smits added a comment -

        I believe the 'blank' variant is more common, even beyond the OFBiz world. As it is perceived as no filtering data is provided, thus all are applicable.

        Show
        pfm.smits Pierre Smits added a comment - I believe the 'blank' variant is more common, even beyond the OFBiz world. As it is perceived as no filtering data is provided, thus all are applicable.
        Hide
        Renuka_Srishti Renuka Srishti added a comment - - edited

        There are redundant options because in those Find Screens, in dropdown allow-empty is set to true and put one option value having description "Any". So remove "Any" to make all screens consistent also set allow-empty=true where "Any" option was added.

        We can initiate new discussion to show empty or "Any" keyword if allow-empty is set to true in new ticket.

        Show
        Renuka_Srishti Renuka Srishti added a comment - - edited There are redundant options because in those Find Screens, in dropdown allow-empty is set to true and put one option value having description "Any". So remove "Any" to make all screens consistent also set allow-empty=true where "Any" option was added. We can initiate new discussion to show empty or "Any" keyword if allow-empty is set to true in new ticket.
        Hide
        deepak.dixit Deepak Dixit added a comment -

        Thanks Deepak N and Renuka for your contribution.
        This has been committed at r#1758032

        Show
        deepak.dixit Deepak Dixit added a comment - Thanks Deepak N and Renuka for your contribution. This has been committed at r#1758032

          People

          • Assignee:
            deepak.dixit Deepak Dixit
            Reporter:
            deepak.nigam Deepak Nigam
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development