summaryrefslogtreecommitdiffstats
path: root/extensions/PhabBugz/lib
AgeCommit message (Collapse)AuthorFilesLines
2018-04-01Merge branch 'master' into unstableDylan William Hardison5-92/+62
2018-03-29Bug 1441063: Fix the unaccepted revision comment (PhabBugz)Piotr Zalewa1-1/+7
* PhabBugz: Fix the unaccepted revision comment Summary: Currently, we're sending the "User removed from revision" comment when the Accept flag has been removed from a revision. This could happen in a number of use cases: 1. The reviewer resigned from being a reviewer. 2. The reviewer has been removed from reviewers list. 3. Someone (author, reviewer) removed acceptance of the revision by changing its status to "Needs Review". This patch is sending a "flag is deactivated" or "reviewer removed from revision" depending on the current status of the reviewer. Test Plan: Accept a revision. Change the revision to "Needs Review". Check the bug comment. Accept the revision. Remove the user from reviewers list. Check the bug comment. Reviewers: dkl Bug #: 1441063 Differential Revision: https://phabricator.services.mozilla.com/D809 * Styling fixed as requested in review. * Perl style fixes
2018-03-27Bug 1440829 - Bugzilla comment for Phabricator commit should include entire ↵dklawren3-7/+9
commit message, not just first line
2018-03-23Bug 1446974 - Revert changes of the subscribers list on secure revisionsdklawren1-3/+5
2018-03-20Bug 1447291 - Remove Apache2::Log from PhabBugs/Push in favor of logging ↵Dylan William Hardison3-81/+41
framework
2018-03-18Bug 1446236 - Allow customizing the name of the "nobody" user ↵Vladimir Panteleev1-1/+1
(nobody@mozilla.org)
2018-03-06Bug 1443537 - Add utility functions for switching database handle to ↵Dylan William Hardison1-12/+7
writeable or read-only mode
2018-03-02Bug 1402494 - BMO Integration User is a full administrative user on Phabricatordklawren3-17/+32
2018-02-28Bug 1440239 - PhabBugz - Assign a secure-revisionPiotr Zalewa4-12/+69
2018-02-28Bug 1437384 - phabbugz_feed.pl in PhabBugz extension should be extended to ↵dklawren2-30/+177
also query for new users in Phab
2018-02-27Bug 1441329 - Fix typos in the PhahBugz User.pm moduledklawren1-3/+3
2018-02-26Bug 1437383: Create User.pm PhabBugz class for loading of a user object from ↵dklawren1-0/+194
phabricator
2018-02-13Bug 1434438 - Refactor Revision.pm to use Moo for cleaner type checkingdklawren2-148/+156
2018-02-13Bug 1434064 - Refactor Project.pm to use Moo for better type checkingdklawren2-157/+152
2018-02-07Bug 1430259 - Update policy code in BMO PhabBugz extension to update custom ↵dklawren3-91/+295
policy if a private bugs groups have changed.
2018-01-11Bug 1429600 - Update Revision.pm type checking to treat bug id as a simple ↵dklawren1-2/+5
string or undefined
2018-01-09Bug 1429110: Update Revision.pm type checking to treat bug id as a simple stringdklawren1-7/+5
2018-01-05Bug 1428156 - BMO will mark a revision as public if it sees a new one ↵dklawren1-2/+12
created without a bug id associated with it
2018-01-03 Bug 1426518 - Revisions can optionally not have a bug id so we need to make ↵dklawren4-25/+42
it optional in the type constraints of Revision.pm
2018-01-03Bug 1427743 - legacy phabbugz API code errors when trying to set an inactive ↵dklawren1-2/+2
review flag
2017-12-20Bug 1426424 - feed daemon complains when trying to set an inactive review flagDavid Lawrence1-2/+2
2017-12-19Bug 1422329 - The phabricator conduit API method feed.query_id return data ↵David Lawrence3-21/+13
format has changed so the phabbugz_feed.pl daemon needs to be updated
2017-12-19Bug 1409957 - Create polling daemon to query Phabricator for recent ↵David Lawrence8-41/+1222
transcations and update bug data according to revision changes
2017-12-16Bug 1403777 - Migrate urlbase from params to localconfigDylan William Hardison1-2/+2
2017-12-05Revert "Bug 1409957 - Create polling daemon to query Phabricator for recent ↵Dylan William Hardison8-1222/+41
transcations and update bug data according to revision changes" This reverts commit 7da8e374e0c96e10077690935e829b0c04fc82f4.
2017-12-05Revert "Bug 1422329 - The phabricator conduit API method feed.query_id ↵Dylan William Hardison3-13/+21
return data format has changed so the phabbugz_feed.pl daemon needs to be updated" This reverts commit 357528ba856fd85aee5967e5d7d8f4426bc8d5b6.
2017-12-01Bug 1422329 - The phabricator conduit API method feed.query_id return data ↵David Lawrence3-21/+13
format has changed so the phabbugz_feed.pl daemon needs to be updated
2017-11-29Bug 1409957 - Create polling daemon to query Phabricator for recent ↵dklawren8-41/+1222
transcations and update bug data according to revision changes
2017-10-11Bug 1407617 - When making a phab revision public, the edit policy also needs ↵dklawren1-0/+4
to be set to 'users' (#261)
2017-10-10Bug 1399593 - Add sync group comments during new revision creation as well ↵David Walsh2-22/+88
as push connector
2017-09-28Bug 1403584: Clear r+ in bug when reviewer is removed from the revision by ↵dklawren1-4/+11
the author via Change Reviewers (#251)
2017-09-14Bug 1399508 - Pass along transaction id to BMO API calls for linking back to ↵dklawren1-0/+2
the exact change in the revision history (#237)
2017-09-12Bug 1396957 - Synchronize CC list on security bugs over to revisions as ↵dklawren1-1/+18
subscribers (#230) * Bug 1396957 - Synchronize CC list on security bugs over to revisions as subscribers * - Updated to only make subscriber changes if no groups have changed. - This cuts down on the amount of custom policy changes in the revision activity. * Bug 1396957 - Synchronize CC list on security bugs over to revisions as subscribers * - Updated to only make subscriber changes if no groups have changed. - This cuts down on the amount of custom policy changes in the revision activity. * Bug 1396042 - Implement logging for the BMO push connector * Bug 1397747 - Move _get_all_group_names to Bugzilla::Config::Common::_get_all_group_names, and remove copy-pasted code
2017-09-11Bug 1397927 - Add comment to bug when a revision is accepted/denied with the ↵dklawren1-7/+31
users real names (#233) Bug 1397927 - Add comment to bug when a revision is accepted/denied with the users real names * - Bug fixes * Wording changes.
2017-08-31Fix 1395685 - Update errant self reference in PhabBugzDavid Walsh1-1/+1
2017-08-31Bug 1395346 - Allow setting obsolete to false for phabricator attachments ↵dklawren1-2/+8
when a user reclaims a revision (#227)
2017-08-30Revert "Bug 1395346 - Allow setting obsolete to false for phabricator ↵David Lawrence1-8/+2
attachments when a user reclaims a revision" This reverts commit c48fede1691feb1056cd21422716f297af9e7bc3.
2017-08-30Bug 1395346 - Allow setting obsolete to false for phabricator attachments ↵David Lawrence1-2/+8
when a user reclaims a revision
2017-08-30Bug 1393023: Approving a revision creates an r+ flag on the revision ↵dklawren2-9/+178
attachment in the associated bug (#219) Approving a revision creates an r+ flag on the revision attachment in the associated bug * - Working version - Splits accepted_users on ':' instead of accepting a list (phab issue)
2017-08-17Bug 1389372 - Add REST API endpoint to validate whether a BMO user ID can ↵David Walsh1-0/+38
see a given BMO bug
2017-08-11Bug 1382225 - Missing code from PhabBugz extension such as http basic auth ↵dklawren3-28/+95
support and other minor improvements
2017-08-01Bug 1385947 - Make revisions fully public (#189)David Walsh1-2/+2
2017-07-31Bug 1383108 - Sync security groups with Phabricator push connectorDavid Walsh2-1/+36
r=dkl
2017-07-20Bug 1380727 - Update BMO Push Connector to Make Revisions Public when Bug ↵dklawren1-2/+2
Made Public
2017-07-19Bug 1380727 - Update BMO Push Connector to Make Revisions Public when Bug ↵David Walsh2-7/+8
Made Public
2017-07-10Bug 1379632 - Error is thrown by update_project_members.pl when Phabricator ↵David Lawrence1-1/+2
returns a null value for a bmo user id search
2017-07-07Bug 1377933 - Remove trailing whitespace from all perl filesDylan William Hardison1-1/+1
2017-07-07Bug 1367604 - BMO extension to apply security policies to Phabricator ↵dklawren2-0/+389
revisions as needed * - Updated based on dylans review - Fixed custom policy to instead allow projects and subscribers and then add BMO roles to the subscriber list - Some other bug fixes * fix lifetime of phabricator_url_re() Instead of passing the value (which depends on runtime configuration) pass in a reference. Also edit extensions/BMO/Extension.pm to allow %autodetect_attach_urls regex option to be a callback instead of just a plain regexp ref. * - Fixed regex in BMO extension to detect phabricator attachments - Use request_cache for useragent handle in Util.pm
2017-06-06Bug 1370378 - Add Phabricator Extension enabled flag to instantly toggle off ↵David Walsh1-0/+5
all Phabricator syncing. r?dkl
2017-06-01Bug 1368066 - When using bmo auth delegation from Phabricator, the ↵David Lawrence1-0/+20
confirmation should be skipped similar to mozreview