Alfresco
  1. Alfresco
  2. ALF-21074

The fix for ACE-2278 committed to r71857 not working

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Unprioritized Unprioritized
    • Resolution: Not a bug
    • Affects Version/s: 5.0
    • Fix Version/s: None
    • Component/s: Forms
    • Security Level: external (External user)
    • Labels:
      None
    • Security Severity:
      None
    • Triage:
      To Do

      Description

      In function generateTransientFieldModel there is an initial test including test if the "set" is set:
      (fieldConfig.getAttributes().get("set") != null && !fieldConfig.getAttributes().get("set").isEmpty())

      This code evaluates to true if the value is set and not empty. You should probably drop the negation of the isEmpty test.

      Since the ACE-2278 is a hidden issue I could not add to that issue. Also, I classified this as "Forms Development Kit", but that is removed in HEAD, maybe rename tag to just "Forms"

        Activity

        Hide
        Gavin Cornwell added a comment -

        Hi Peter,

        Thanks for raising this, the logic has already been changed due to ACE-2278 the fix just hasn't made it's way to HEAD yet.

        I've also changed the component to "Forms" in this project as you suggested.

        Show
        Gavin Cornwell added a comment - Hi Peter, Thanks for raising this, the logic has already been changed due to ACE-2278 the fix just hasn't made it's way to HEAD yet. I've also changed the component to "Forms" in this project as you suggested.

          People

          • Assignee:
            Closed Issues
            Reporter:
            Peter Löfgren
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Date of First Response: