GoogleGit

commit0f243ef2a783bedf56c0df3603c9cec400c6154f[log][tgz]
authorDave Borowitz <dborowitz@google.com>Tue May 05 10:13:09 2015 -0700
committerDave Borowitz <dborowitz@google.com>Tue May 05 10:42:03 2015 -0700
treedb28fe67b4ce610fca867d97229979663104884d
parent74b919a6c96e0becadcef0137b685c32ed336065[diff]
PostReview: Don't try to publish other users' drafts

We were calling the wrong PatchLineCommentsUtil method by accident,
getting all drafts on a change rather than just the current user's
drafts. Fortunately, there is a sanity check in ChangeDraftUpdate that
was triggering and turning these into 500s rather than actually
publishing other users' drafts.

Fix this bug, and add some more draft changes to the test case that
are explicitly excluded.

Change-Id: Ifae5b0a61328b33170664a81a39f02fa7cfd4e4e
2 files changed
tree: db28fe67b4ce610fca867d97229979663104884d
  1. .buckconfig
  2. .buckversion
  3. .editorconfig
  4. .gitignore
  5. .gitmodules
  6. .mailmap
  7. .pydevproject
  8. .settings/
  9. .watchmanconfig
  10. BUCK
  11. COPYING
  12. Documentation/
  13. INSTALL
  14. ReleaseNotes/
  15. SUBMITTING_PATCHES
  16. VERSION
  17. bucklets/
  18. contrib/
  19. gerrit-acceptance-tests/
  20. gerrit-antlr/
  21. gerrit-cache-h2/
  22. gerrit-common/
  23. gerrit-extension-api/
  24. gerrit-gwtdebug/
  25. gerrit-gwtexpui/
  26. gerrit-gwtui-common/
  27. gerrit-gwtui/
  28. gerrit-httpd/
  29. gerrit-launcher/
  30. gerrit-lucene/
  31. gerrit-main/
  32. gerrit-oauth/
  33. gerrit-openid/
  34. gerrit-patch-commonsnet/
  35. gerrit-patch-jgit/
  36. gerrit-pgm/
  37. gerrit-plugin-api/
  38. gerrit-plugin-archetype/
  39. gerrit-plugin-gwt-archetype/
  40. gerrit-plugin-gwtui/
  41. gerrit-plugin-js-archetype/
  42. gerrit-prettify/
  43. gerrit-reviewdb/
  44. gerrit-server/
  45. gerrit-solr/
  46. gerrit-sshd/
  47. gerrit-util-cli/
  48. gerrit-util-http/
  49. gerrit-util-ssl/
  50. gerrit-war/
  51. lib/
  52. plugins/
  53. tools/
  54. website/