Public bug reported:

I'm looking to improve some Phabricator code for Lugito (Lubuntu's
Python script which acts as a hook for commits such as IRC
notifications), and I can't use diffusion.commit.search because there's
no support for it yet.

>>> phab.diffusion.commit.search(constraints={"query": "a34837de8224"})
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3/dist-packages/phabricator/__init__.py", line 248, in 
__call__
    return self._request(**kwargs)
  File "/usr/lib/python3/dist-packages/phabricator/__init__.py", line 323, in 
_request
    data = self._parse_response(response)
  File "/usr/lib/python3/dist-packages/phabricator/__init__.py", line 333, in 
_parse_response
    raise APIError(parsed['error_code'], parsed['error_info'])
phabricator.APIError: ERR-CONDUIT-CALL: Conduit API method "commit.search" does 
not exist.

However, my Phabricator instance (last updated about a week ago) says
this under diffusions.querycommits:

Frozen Method: This method is frozen and will eventually be deprecated.
New code should use "diffusion.commit.search" instead.

A patch adding support for this is available upstream:
https://github.com/disqus/python-
phabricator/commit/a52c0e1357532f9ef8b26321d3728a0edead6ed4

This needs to be backported, or functionality will be lost.

** Affects: python-phabricator (Ubuntu)
     Importance: High
     Assignee: Simon Quigley (tsimonq2)
         Status: Fix Committed

** Affects: python-phabricator (Ubuntu Bionic)
     Importance: High
     Assignee: Simon Quigley (tsimonq2)
         Status: Confirmed

** Affects: python-phabricator (Ubuntu Cosmic)
     Importance: High
     Assignee: Simon Quigley (tsimonq2)
         Status: Fix Committed

** Affects: python-phabricator (Debian)
     Importance: Unknown
         Status: Unknown

** Bug watch added: Debian Bug tracker #905796
   https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=905796

** Also affects: python-phabricator (Debian) via
   https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=905796
   Importance: Unknown
       Status: Unknown

** Also affects: python-phabricator (Ubuntu Bionic)
   Importance: Undecided
       Status: New

** Also affects: python-phabricator (Ubuntu Cosmic)
   Importance: Undecided
       Status: New

** Changed in: python-phabricator (Ubuntu Bionic)
       Status: New => Confirmed

** Changed in: python-phabricator (Ubuntu Cosmic)
       Status: New => Confirmed

** Changed in: python-phabricator (Ubuntu Bionic)
   Importance: Undecided => High

** Changed in: python-phabricator (Ubuntu Cosmic)
   Importance: Undecided => High

** Changed in: python-phabricator (Ubuntu Bionic)
     Assignee: (unassigned) => Simon Quigley (tsimonq2)

** Changed in: python-phabricator (Ubuntu Cosmic)
     Assignee: (unassigned) => Simon Quigley (tsimonq2)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1786337

Title:
  diffusion.querycommits deprecated in favor of diffusion.commit.search
  which support hasn't been merged for yet

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/python-phabricator/+bug/1786337/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to