Details

    • Type: Sub-task
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9 Larks
    • Fix Version/s: 2.0 Larks
    • Labels:
      None
    • Sprint:
      1.9 Larks, 2.0 Larks

      Description

      Flexible abstract page should be created to edit any number of typed properties for a particular plugin. Based on available types it should provide different kind of data controls. Please also implement controller & service classes to support that feature. Dao and Model classes are out of scope of this task.

        Attachments

          Activity

          Hide
          ctapobep Stanislav Bashkyrtsev added a comment -

          Doesn't work at all - just a 500 Internal Error.

          Show
          ctapobep Stanislav Bashkyrtsev added a comment - Doesn't work at all - just a 500 Internal Error.
          Show
          nigredo Andrey Ivanov added a comment - Fixed! Implementation notes: http://pronicles.blogspot.com/2009_09_01_archive.html http://www.stevideter.com/2008/12/07/saveorupdate-versus-merge-in-hibernate/
          Hide
          ctapobep Stanislav Bashkyrtsev added a comment -

          Andrey Ivanov, couple of notes:

          • If we use merge(), then our method should return merged values
          • Did you figure out why other objects with the same identity were in the cache? I'd expect that we fix the issue with having only a single copy of object in the session. Current fix looks to be a shortcut.
          Show
          ctapobep Stanislav Bashkyrtsev added a comment - Andrey Ivanov , couple of notes: If we use merge(), then our method should return merged values Did you figure out why other objects with the same identity were in the cache? I'd expect that we fix the issue with having only a single copy of object in the session. Current fix looks to be a shortcut.
          Hide
          nigredo Andrey Ivanov added a comment - - edited

          First of all, Why should i return merged values? We don't use this.

          About current solution, i read a lot of article (more important i added to comment, see above your comment) about NonUniqueObjectException and all them said "Use merge instead of saveOrUpdate".
          Why the same identity is in the cache i think because our system use pouple plugin, i mean load it. But it's only my guess, i didn't investigate it.. If it important i can do this, i mean investigation.

          Show
          nigredo Andrey Ivanov added a comment - - edited First of all, Why should i return merged values? We don't use this. About current solution, i read a lot of article (more important i added to comment, see above your comment) about NonUniqueObjectException and all them said "Use merge instead of saveOrUpdate". Why the same identity is in the cache i think because our system use pouple plugin, i mean load it. But it's only my guess, i didn't investigate it.. If it important i can do this, i mean investigation.
          Hide
          ctapobep Stanislav Bashkyrtsev added a comment -

          Let's discuss this in skype.

          Show
          ctapobep Stanislav Bashkyrtsev added a comment - Let's discuss this in skype.
          Hide
          nigredo Andrey Ivanov added a comment -

          stanislav bashkirtsev please check current solution about replacing merge method

          Show
          nigredo Andrey Ivanov added a comment - stanislav bashkirtsev please check current solution about replacing merge method
          Hide
          ctapobep Stanislav Bashkyrtsev added a comment -

          Looks fine as for me.

          Show
          ctapobep Stanislav Bashkyrtsev added a comment - Looks fine as for me.
          Hide
          ctapobep Stanislav Bashkyrtsev added a comment -

          Everything is going to be fixed in scope of the story, therefore closing.

          Show
          ctapobep Stanislav Bashkyrtsev added a comment - Everything is going to be fixed in scope of the story, therefore closing.

            People

            • Assignee:
              julik Julia Atlygina
              Reporter:
              jk1 Evgeniy Naumenko
            • Votes:
              0 Vote for this issue
              Watchers:
              Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 4h
                4h

                  Structure Helper Panel