1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2025-02-17 01:08:41 +01:00

Update various Asana odds-and-ends for "gid" API changes

Summary:
Ref T13453. Some of the Asana integrations also need API updates.

Depends on D20899.

Test Plan:
  - Viewed "asana.workspace-id" in Config, got a sensible GID list.
  - Created a revision, saw the associated Asana task get assigned.
  - Pasted an Asana link I could view into a revision description, saw it Doorkeeper in the metadata.

Maniphest Tasks: T13453

Differential Revision: https://secure.phabricator.com/D20900
This commit is contained in:
epriestley 2019-11-08 08:57:35 -08:00
parent 2223d6b914
commit cd60a8aa56
3 changed files with 13 additions and 7 deletions

View file

@ -123,8 +123,11 @@ final class DoorkeeperBridgeAsana extends DoorkeeperBridge {
} }
public function fillObjectFromData(DoorkeeperExternalObject $obj, $result) { public function fillObjectFromData(DoorkeeperExternalObject $obj, $result) {
$id = $result['id']; $gid = $result['gid'];
$uri = "https://app.asana.com/0/{$id}/{$id}"; $uri = urisprintf(
'https://app.asana.com/0/%s/%s',
$gid,
$gid);
$obj->setObjectURI($uri); $obj->setObjectURI($uri);
} }

View file

@ -102,7 +102,10 @@ final class PhabricatorAsanaConfigOptions
pht('Workspace Name')); pht('Workspace Name'));
$out[] = '| ------------ | -------------- |'; $out[] = '| ------------ | -------------- |';
foreach ($workspaces as $workspace) { foreach ($workspaces as $workspace) {
$out[] = sprintf('| `%s` | `%s` |', $workspace['id'], $workspace['name']); $out[] = sprintf(
'| `%s` | `%s` |',
$workspace['gid'],
$workspace['name']);
} }
$out = implode("\n", $out); $out = implode("\n", $out);

View file

@ -358,7 +358,7 @@ final class DoorkeeperAsanaFeedWorker extends DoorkeeperFeedWorker {
'POST', 'POST',
$subtask_data + array( $subtask_data + array(
'assignee' => $phid_aid_map[$user_phid], 'assignee' => $phid_aid_map[$user_phid],
'completed' => $is_completed, 'completed' => (int)$is_completed,
'parent' => $parent_ref->getObjectID(), 'parent' => $parent_ref->getObjectID(),
)); ));
@ -393,7 +393,7 @@ final class DoorkeeperAsanaFeedWorker extends DoorkeeperFeedWorker {
'PUT', 'PUT',
$subtask_data + array( $subtask_data + array(
'assignee' => $phid_aid_map[$user_phid], 'assignee' => $phid_aid_map[$user_phid],
'completed' => $is_completed, 'completed' => (int)$is_completed,
)); ));
} }
@ -484,7 +484,7 @@ final class DoorkeeperAsanaFeedWorker extends DoorkeeperFeedWorker {
return array( return array(
'name' => $title, 'name' => $title,
'notes' => $notes, 'notes' => $notes,
'completed' => $is_completed, 'completed' => (int)$is_completed,
); );
} }
@ -632,7 +632,7 @@ final class DoorkeeperAsanaFeedWorker extends DoorkeeperFeedWorker {
->setApplicationType(DoorkeeperBridgeAsana::APPTYPE_ASANA) ->setApplicationType(DoorkeeperBridgeAsana::APPTYPE_ASANA)
->setApplicationDomain(DoorkeeperBridgeAsana::APPDOMAIN_ASANA) ->setApplicationDomain(DoorkeeperBridgeAsana::APPDOMAIN_ASANA)
->setObjectType($type) ->setObjectType($type)
->setObjectID($result['id']) ->setObjectID($result['gid'])
->setIsVisible(true); ->setIsVisible(true);
$xobj = $ref->newExternalObject(); $xobj = $ref->newExternalObject();