Uploaded image for project: 'Poulpe'
  1. Poulpe
  2. POULPE-528

Poulpe creates erroneous records in DB if you open Personal permissions after clean install.

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1 Swallows
    • Fix Version/s: backlog
    • Labels:
      None
    • Environment:

      poulpe-2.1.343.e62d084, Win XP, Google Chrome

      Description

      Steps to reproduce:
      1. Deploy JC and configure it to use empty database.
      2. Start JC (you needn't log)
      3. Deploy Poulpe and configure it to use the same database.
      4. Login as admin/admin.
      5. Open Personal permissions mode.
      Actual result: Record with sid=user:0 appears in acl_sid table after step 3. Several records are also created in acl_object_identity.
      Expected result: sid field always corresponds to existing entity id, not 0.

        Attachments

        1. group_ownership.png
          group_ownership.png
          10 kB
        2. jcommune.txt
          75 kB
        3. jtalks.acl_object_identity.jpg
          jtalks.acl_object_identity.jpg
          25 kB
        4. jtalks.acl_sid.jpg
          jtalks.acl_sid.jpg
          14 kB
        5. localhost.txt
          44 kB
        6. sql.log
          278 kB
        7. user_0.png
          user_0.png
          6 kB

          Issue Links

            Activity

            Hide
            ctapobep Stanislav Bashkyrtsev added a comment -

            General impression is very good. JavaDocs & commit message are impressive.
            But next time it's better to send a PR - easier to see what changed comparing to `master`. Also, see minor comments to the commit.

            Show
            ctapobep Stanislav Bashkyrtsev added a comment - General impression is very good. JavaDocs & commit message are impressive. But next time it's better to send a PR - easier to see what changed comparing to `master`. Also, see minor comments to the commit.
            Hide
            vladimir Vladimir Kamenksiy added a comment -

            Thank you for review.
            I'll check your remarks, correct code and create a PR.

            Show
            vladimir Vladimir Kamenksiy added a comment - Thank you for review. I'll check your remarks, correct code and create a PR.
            Hide
            jenkins Jenkins Bot added a comment -

            SUCCESS: Integrated in JC-UnitTests #2930
            Poulpe-528 Issue with 'User #0 not found' error (v.kamenskyi: 6766f81185a4664e75b3e47c97d8110c75a790b5)

            Show
            jenkins Jenkins Bot added a comment - SUCCESS: Integrated in JC-UnitTests #2930 Poulpe-528 Issue with 'User #0 not found' error (v.kamenskyi: 6766f81185a4664e75b3e47c97d8110c75a790b5 )
            Hide
            ctapobep Stanislav Bashkyrtsev added a comment -

            Artem R, you can proceed with testing locally.

            Show
            ctapobep Stanislav Bashkyrtsev added a comment - Artem R , you can proceed with testing locally.
            Hide
            varro Artem R added a comment -

            Test Environment
            Poulpe 2.1.357.331e400

            Test Scenario
            Steps
            1. Deploy JC and configure it to use empty database.
            2. Start JC (you needn't log)
            3. Deploy Poulpe and configure it to use the same database.
            4. Login as admin/admin.
            5. Open Personal permissions mode.

            Actual result = Expected result
            There is no user with id=0 in DB and logs

            Test results:
            Issue can be closed
            But POULPE-530 was found

            Show
            varro Artem R added a comment - Test Environment Poulpe 2.1.357.331e400 Test Scenario Steps 1. Deploy JC and configure it to use empty database. 2. Start JC (you needn't log) 3. Deploy Poulpe and configure it to use the same database. 4. Login as admin/admin. 5. Open Personal permissions mode. Actual result = Expected result There is no user with id=0 in DB and logs Test results: Issue can be closed But POULPE-530 was found

              People

              • Assignee:
                varro Artem R
                Reporter:
                begemosha Anton Barkov
              • Votes:
                0 Vote for this issue
                Watchers:
                Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Structure Helper Panel