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

Add a look-up for UserLogin Id in Find Party Screen

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: party
    • Labels:
      None
    • Sprint:
      Community Day 2 - 2016

      Description

      Currently, In Find Party Screen, there is UserLogin Id is a text field, we should have a lookup for this these field.

      1. OFBIZ-7303.patch
        0.7 kB
        Pawan Verma

        Activity

        Hide
        pawan.verma Pawan Verma added a comment -

        Providing patch for the improvement. Added UserLoginId look-up for Find Party screen.

        Show
        pawan.verma Pawan Verma added a comment - Providing patch for the improvement. Added UserLoginId look-up for Find Party screen.
        Hide
        pfm.smits Pierre Smits added a comment -

        Add a lookup for a userLoginId? Does that even make sense nowadays? It doesn't seem appropriate, in the same line as asking for a password and a pin code....

        If a person can't be found while searching for first and last name (and probably some additionals like organisation, department, address, etc) then there is something seriously wrong with how the solutions does a search and find, but searching for a userId is not kosher.

        Show
        pfm.smits Pierre Smits added a comment - Add a lookup for a userLoginId? Does that even make sense nowadays? It doesn't seem appropriate, in the same line as asking for a password and a pin code.... If a person can't be found while searching for first and last name (and probably some additionals like organisation, department, address, etc) then there is something seriously wrong with how the solutions does a search and find, but searching for a userId is not kosher.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        For a reason, you may only know the username, it's then convenient. I see no security issue there.

        Show
        jacques.le.roux Jacques Le Roux added a comment - For a reason, you may only know the username, it's then convenient. I see no security issue there.
        Hide
        deepak.dixit Deepak Dixit added a comment -

        This has been fixed at r#1748942

        Thanks Pawan for your contribution

        Show
        deepak.dixit Deepak Dixit added a comment - This has been fixed at r#1748942 Thanks Pawan for your contribution

          People

          • Assignee:
            deepak.dixit Deepak Dixit
            Reporter:
            pawan.verma Pawan Verma
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development

                Agile