)]}'
{
  "commit": "5b3805a73936deabbafbfdb930df1b16b5b28670",
  "tree": "5a440aa67b0a36c8ceb98c387c5ba73c79dc5353",
  "parents": [
    "59c7671b85ad00d8825d6280816ddd875b323982"
  ],
  "author": {
    "name": "Jacek Centkowski",
    "email": "jcentkowski@collab.net",
    "time": "Mon Apr 20 16:00:00 2020 +0200"
  },
  "committer": {
    "name": "David Pursehouse",
    "email": "dpursehouse@digital.ai",
    "time": "Wed Apr 29 14:14:33 2020 +0900"
  },
  "message": "Fix \u0027visibleto\u0027 predicate for group\n\nIssue:\nWhen one uses a valid group name in \u0027visibleto\u0027 predicate no change\ngets returned even if it should be from the access perspective.\n\nSolution:\nIt boils down to the problem in ChangeIsVisibleToPredicate for\nSingleGroupUser user - it is not an IdentifiedUser hence it gets\nswapped for AnonymousUser and visibility check fails unless changes\nin question are granted to Anonymous Users group. Fix it to let it\nuse the user it has in case it is SingleGroupUser or follow existing\nuser resolution (with fallback to anonymous) as it was before.\n\nBug: Issue 12606\nChange-Id: Ieea8abf4b52d528d4cb2503270c7eff68476fc6c\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "f81ea15947917907eeac6c89725b546d6613149e",
      "old_mode": 33188,
      "old_path": "java/com/google/gerrit/server/query/change/ChangeIsVisibleToPredicate.java",
      "new_id": "b5f3a4fc0c56ec1f824ae87b5f9598652de58e98",
      "new_mode": 33188,
      "new_path": "java/com/google/gerrit/server/query/change/ChangeIsVisibleToPredicate.java"
    },
    {
      "type": "modify",
      "old_id": "bd2b3596d5da0eed13c3381a47af299b0801612a",
      "old_mode": 33188,
      "old_path": "javatests/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java",
      "new_id": "5311b0f4093f8fd26d5b5f6c6d12c2ed31e5de71",
      "new_mode": 33188,
      "new_path": "javatests/com/google/gerrit/server/query/change/AbstractQueryChangesTest.java"
    }
  ]
}
