Details

    • Technical task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.10.0, 1.9.4
    • core
    • None

    Description

      Seems like a trivial change, in an area almost no one touches anyways

      Attachments

        1. OAK-7527.patch
          0.6 kB
          Alex Deparvu

        Issue Links

          Activity

            stillalex Alex Deparvu added a comment -

            anchela, mduerig any ideas of what the impact on changing the node type definition might be? hopefully none?

            stillalex Alex Deparvu added a comment - anchela , mduerig any ideas of what the impact on changing the node type definition might be? hopefully none?

            stillalex, i would be really surprised if removing a 'mandatory' would have any impact. while i don't know all the details of the nt-code-base in oak, i can assure you that i am not aware of any regression/ticket/complain associated with dropping the 'mandatory' from the "rep:password" property quite some years ago. so +1 for the proposed fix. if you don't mind, i will also link it to the corresponding issue that made us aware of the problem.

            angela Angela Schreiber added a comment - stillalex , i would be really surprised if removing a 'mandatory' would have any impact. while i don't know all the details of the nt-code-base in oak, i can assure you that i am not aware of any regression/ticket/complain associated with dropping the 'mandatory' from the "rep:password" property quite some years ago. so +1 for the proposed fix. if you don't mind, i will also link it to the corresponding issue that made us aware of the problem.
            stillalex Alex Deparvu added a comment -

            thanks a lot Angela, I moved the issue under OAK-7498, you are right, it looks like a better 'parent issue'.

            stillalex Alex Deparvu added a comment - thanks a lot Angela, I moved the issue under OAK-7498 , you are right, it looks like a better 'parent issue'.
            stillalex Alex Deparvu added a comment - fixed with http://svn.apache.org/viewvc?rev=1832984&view=rev

            Bulk close 1.9.4

            edivad Davide Giannella added a comment - Bulk close 1.9.4

            Adding 1.10 version before 1.10.0 release

            edivad Davide Giannella added a comment - Adding 1.10 version before 1.10.0 release

            People

              stillalex Alex Deparvu
              stillalex Alex Deparvu
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: