Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: High High
    • Resolution: Fixed
    • Affects Version/s: 5.3.12, 2017.12, 2018.06, 5.4.12, 2018.09
    • Fix Version/s: 2018.09, 2017.12.3, 5.4.14
    • Component/s: None
    • Labels:
      None
    • Environment:

      6.12.0 (legacy mode) but the error seems to exists for years.

      Description

      Hi,

      when editing a object in backend you normally get a red border added to the input field when you miss out a required field or if you enter invalid data.

      this error border is now missing. the reason is, that the hasValidationError attribute is ALWAYS false.

      hasValidationError seems to be set properly but then is getting overwritten.

      The cause for this is:

      https://jira.ez.no/browse/EZP-23088
      https://github.com/ezsystems/ezpublish-legacy/commit/99269b546bfa45ae68dd9a8677bf562af219a527#diff-bcd306678394f088cc39c7701591d264

      If you deactivate this line, the border is there.

        Activity

        Robert Röttger created issue -
        Hide
        Patrizio Bekerle added a comment -

        I can confirm this bug and its origin.

        Show
        Patrizio Bekerle added a comment - I can confirm this bug and its origin.
        Hide
        Patrizio Bekerle added a comment -

        After several hours of researching and testing the code I found this solution: https://github.com/ezsystems/ezpublish-legacy/pull/1412

        This pull request solves https://jira.ez.no/browse/EZP-29723 while still fixing https://jira.ez.no/browse/EZP-23088.

        Show
        Patrizio Bekerle added a comment - After several hours of researching and testing the code I found this solution: https://github.com/ezsystems/ezpublish-legacy/pull/1412 This pull request solves https://jira.ez.no/browse/EZP-29723 while still fixing https://jira.ez.no/browse/EZP-23088 .
        Gunnstein Lye made changes -
        Field Original Value New Value
        Priority Blocker [ 1 ] High [ 3 ]
        Gunnstein Lye made changes -
        Summary contentobject attributes not set properly Content object attributes not set properly
        Gunnstein Lye logged work - 11/Jan/19 11:00 AM
        • Time Spent:
          1 hour
           

          .

        Gunnstein Lye made changes -
        Assignee Gunnstein Lye [ gunnstein.lye@ez.no ]
        Gunnstein Lye made changes -
        Status Open [ 1 ] Confirmed [ 10037 ]
        Gunnstein Lye made changes -
        Status Confirmed [ 10037 ] InputQ [ 10001 ]
        Gunnstein Lye made changes -
        Status InputQ [ 10001 ] Development [ 3 ]
        Gunnstein Lye made changes -
        Status Development [ 3 ] Development Review [ 10006 ]
        Show
        Gunnstein Lye added a comment - Merged in master, 2018.09, 2017.12. Thanks Patrizio Bekerle ! https://github.com/ezsystems/ezpublish-legacy/commit/c8ce309e26bac73316c467ec1c7b82909bbf9dc7
        Gunnstein Lye made changes -
        Affects Version/s 5.4.12 [ 14877 ]
        Affects Version/s 2018.06 [ 14890 ]
        Affects Version/s 2017.12 [ 14867 ]
        Affects Version/s 5.3.12 [ 14639 ]
        Affects Version/s 2018.09 [ 15003 ]
        Gunnstein Lye made changes -
        Remaining Estimate 0 minutes [ 0 ]
        Time Spent 1 hour [ 3600 ]
        Worklog Id 69140 [ 69140 ]
        Gunnstein Lye made changes -
        Status Development Review [ 10006 ] Documentation Review done [ 10011 ]
        Fix Version/s 2018.09 [ 15003 ]
        Fix Version/s 2017.12.3 [ 15008 ]
        Fix Version/s 5.4.13 [ 15017 ]
        Assignee Gunnstein Lye [ gunnstein.lye@ez.no ]
        Hide
        Patrizio Bekerle added a comment -

        Great, thank you!

        Show
        Patrizio Bekerle added a comment - Great, thank you!
        Michał Szołtysek made changes -
        Status Documentation Review done [ 10011 ] QA [ 10008 ]
        Assignee Michał Szołtysek [ michal.szoltysek@ez.no ]
        Hide
        Michał Szołtysek added a comment -

        Retest successful on v5.4.14-rc1.

        Show
        Michał Szołtysek added a comment - Retest successful on v5.4.14-rc1.
        Michał Szołtysek made changes -
        Status QA [ 10008 ] QA Done [ 10007 ]
        Fix Version/s 5.4.14 [ 15279 ]
        Fix Version/s 5.4.13 [ 15017 ]
        Assignee Michał Szołtysek [ michal.szoltysek@ez.no ]
        Michał Szołtysek made changes -
        Status QA Done [ 10007 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Confirmed Confirmed
        91d 5h 8m 1 Gunnstein Lye 11/Jan/19 2:16 PM
        Confirmed Confirmed InputQ InputQ
        6s 1 Gunnstein Lye 11/Jan/19 2:16 PM
        InputQ InputQ Development Development
        3s 1 Gunnstein Lye 11/Jan/19 2:16 PM
        Development Development Development Review Development Review
        3s 1 Gunnstein Lye 11/Jan/19 2:16 PM
        Development Review Development Review Documentation Review done Documentation Review done
        11m 55s 1 Gunnstein Lye 11/Jan/19 2:28 PM
        Documentation Review done Documentation Review done QA QA
        118d 22h 22m 1 Michał Szołtysek 10/May/19 1:50 PM
        QA QA QA Done QA Done
        2m 49s 1 Michał Szołtysek 10/May/19 1:53 PM
        QA Done QA Done Closed Closed
        30s 1 Michał Szołtysek 10/May/19 1:53 PM

          People

          • Assignee:
            Unassigned
            Reporter:
            Robert Röttger
          • Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - Not Specified
              Not Specified
              Remaining:
              Remaining Estimate - 0 minutes
              0m
              Logged:
              Time Spent - 1 hour
              1h