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

Refactoring permission model call

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Implemented
    • Trunk
    • None
    • framework
    • None
    • Community Day 3 - 2016

    Description

      With the subject on mailing list "Permission service not on the same transaction" : http://markmail.org/message/34vxi4ntmbj7vzcr

      I started the improvement but the java code need to be clean and refactoring.

      I share my first draft patch, but need more works. I realize :

      • unified call evalPermission
      • move all related field for permission service on ModelService to ModelPermission
      • Remove deprecated code
      • start labelized error message
      • add new attributes on permission model : required-new-transaction and return-error-on-failure

      Attachments

        1. OFBIZ-7113.patch
          45 kB
          Nicolas Malin
        2. OFBIZ-7113.patch
          37 kB
          Nicolas Malin
        3. OFBIZ-7113.patch
          26 kB
          Nicolas Malin

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            nmalin Nicolas Malin
            nmalin Nicolas Malin
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Agile

                Completed Sprint:
                Community Day 3 - 2016 ended 22/Sep/16
                View on Board

                Slack

                  Issue deployment