ahtlon/deck
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

Remove null board value in activity parameters

Browse Source 
Signed-off-by: Julius Härtl <jus@bitgrid.net>
This commit is contained in:
Julius Härtl
2018-11-23 15:04:04 +01:00
parent d28bf74c73
commit 2c024dedce
2 changed files with 11 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

23
lib/Activity/DeckProvider.php
View File

@@ -79,7 +79,16 @@ class DeckProvider implements IProvider {
/** /**
* Map stored parameter objects to rich string types * Map stored parameter objects to rich string types
*/ */
$board = null;
$author = $event->getAuthor();
$user = $this->userManager->get($author);
$params = [
'user' => [
'type' => 'user',
'id' => $author,
'name' => $user !== null ? $user->getDisplayName() : $author
],
];
if ($event->getObjectType() === ActivityManager::DECK_OBJECT_BOARD) { if ($event->getObjectType() === ActivityManager::DECK_OBJECT_BOARD) {
$board = [ $board = [
'type' => 'highlight', 'type' => 'highlight',
@@ -87,19 +96,9 @@ class DeckProvider implements IProvider {
'name' => $event->getObjectName(), 'name' => $event->getObjectName(),
'link' => $this->deckUrl('/board/' . $event->getObjectId()), 'link' => $this->deckUrl('/board/' . $event->getObjectId()),
]; ];
$params['board'] = $board;
} }
$author = $event->getAuthor();
$user = $this->userManager->get($author);
$params = [
'board' => $board,
'user' => [
'type' => 'user',
'id' => $author,
'name' => $user !== null ? $user->getDisplayName() : $author
],
];
if ($event->getObjectType() === ActivityManager::DECK_OBJECT_CARD) { if ($event->getObjectType() === ActivityManager::DECK_OBJECT_CARD) {
$card = [ $card = [
'type' => 'highlight', 'type' => 'highlight',

3
tests/unit/Activity/DeckProviderTest.php
View File

@@ -180,7 +180,6 @@ class DeckProviderTest extends TestCase {
'name' => 'Board', 'name' => 'Board',
'link' => '#!/board/1', 'link' => '#!/board/1',
], ],
'card' => null,
'user' => [ 'user' => [
'type' => 'user', 'type' => 'user',
'id' => 'admin', 'id' => 'admin',
@@ -216,7 +215,6 @@ class DeckProviderTest extends TestCase {
$this->provider->parse('en_US', $event); $this->provider->parse('en_US', $event);
$data = [ $data = [
'board' => null,
'card' => [ 'card' => [
'type' => 'highlight', 'type' => 'highlight',
'id' => 1, 'id' => 1,
@@ -266,7 +264,6 @@ class DeckProviderTest extends TestCase {
$this->provider->parse('en_US', $event); $this->provider->parse('en_US', $event);
$data = [ $data = [
'board' => null,
'card' => [ 'card' => [
'type' => 'highlight', 'type' => 'highlight',
'id' => 1, 'id' => 1,