Skip to content

Compute notification parsed subject from rich subject when possible #34807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
come-nc merged 8 commits into from
Dec 19, 2022
Merged

Compute notification parsed subject from rich subject when possible #34807

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
323963e
Compute parsed from rich subject and message in Notification
come-nc Oct 25, 2022
db995d4
Same thing for Activity
come-nc Oct 25, 2022
d9151ff
Get rid of all useless calls to setParsedSubject now that setRichSubj…
come-nc Oct 25, 2022
2a3090d
Throw instead of using a bogus value if name field is missing
come-nc Nov 21, 2022
cf8d97c
Revert changes to systemtags application
come-nc Nov 21, 2022
8a04bf5
Apply the same changes to richToParsed in Notification
come-nc Nov 21, 2022
f676076
Throw on missing type of rich subject parameter
come-nc Dec 8, 2022
e553cd1
Fix comments and dav tests with Notifications and Activity changes
come-nc Dec 19, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Revert changes to systemtags application 
Signed-off-by: Côme Chilliet <[email protected]>
  • Loading branch information
@come-nc
come-nc committed Nov 29, 2022
commit cf8d97c9d31fc487f9347c120145ab4d70b4d0ee
133 changes: 101 additions & 32 deletions apps/systemtags/lib/Activity/Provider.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -110,33 +110,53 @@ public function parseShortVersion(IEvent $event) {

if ($event->getSubject() === self::ASSIGN_TAG) {
if ($parsedParameters['actor']['id'] === '') {
$event->setRichSubject($this->l->t('Added system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
$event->setParsedSubject($this->l->t('System tag %1$s added by the system', [
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('Added system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
} elseif ($parsedParameters['actor']['id'] === $this->activityManager->getCurrentUserId()) {
$event->setRichSubject($this->l->t('Added system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
$event->setParsedSubject($this->l->t('Added system tag %1$s', [
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('Added system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
} else {
$event->setRichSubject($this->l->t('{actor} added system tag {systemtag}'), [
'actor' => $parsedParameters['actor'],
'systemtag' => $parsedParameters['systemtag'],
]);
$event->setParsedSubject($this->l->t('%1$s added system tag %2$s', [
$parsedParameters['actor']['name'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('{actor} added system tag {systemtag}'), [
'actor' => $parsedParameters['actor'],
'systemtag' => $parsedParameters['systemtag'],
]);
}
} elseif ($event->getSubject() === self::UNASSIGN_TAG) {
if ($parsedParameters['actor']['id'] === '') {
$event->setRichSubject($this->l->t('Removed system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
$event->setParsedSubject($this->l->t('System tag %1$s removed by the system', [
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('Removed system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
} elseif ($parsedParameters['actor']['id'] === $this->activityManager->getCurrentUserId()) {
$event->setRichSubject($this->l->t('Removed system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
$event->setParsedSubject($this->l->t('Removed system tag %1$s', [
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('Removed system tag {systemtag}'), [
'systemtag' => $parsedParameters['systemtag'],
]);
} else {
$event->setRichSubject($this->l->t('{actor} removed system tag {systemtag}'), [
'actor' => $parsedParameters['actor'],
'systemtag' => $parsedParameters['systemtag'],
]);
$event->setParsedSubject($this->l->t('%1$s removed system tag %2$s', [
$parsedParameters['actor']['name'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('{actor} removed system tag {systemtag}'), [
'actor' => $parsedParameters['actor'],
'systemtag' => $parsedParameters['systemtag'],
]);
}
} else {
throw new \InvalidArgumentException();
Expand All @@ -162,39 +182,88 @@ public function parseLongVersion(IEvent $event) {

if ($event->getSubject() === self::CREATE_TAG) {
if ($parsedParameters['actor']['id'] === $this->activityManager->getCurrentUserId()) {
$event->setRichSubject($this->l->t('You created system tag {systemtag}'), $parsedParameters);
$event->setParsedSubject($this->l->t('You created system tag %1$s', [
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('You created system tag {systemtag}'), $parsedParameters);
} else {
$event->setRichSubject($this->l->t('{actor} created system tag {systemtag}'), $parsedParameters);
$event->setParsedSubject($this->l->t('%1$s created system tag %2$s', [
$parsedParameters['actor']['name'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('{actor} created system tag {systemtag}'), $parsedParameters);
}
} elseif ($event->getSubject() === self::DELETE_TAG) {
if ($parsedParameters['actor']['id'] === $this->activityManager->getCurrentUserId()) {
$event->setRichSubject($this->l->t('You deleted system tag {systemtag}'), $parsedParameters);
$event->setParsedSubject($this->l->t('You deleted system tag %1$s', [
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('You deleted system tag {systemtag}'), $parsedParameters);
} else {
$event->setRichSubject($this->l->t('{actor} deleted system tag {systemtag}'), $parsedParameters);
$event->setParsedSubject($this->l->t('%1$s deleted system tag %2$s', [
$parsedParameters['actor']['name'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('{actor} deleted system tag {systemtag}'), $parsedParameters);
}
} elseif ($event->getSubject() === self::UPDATE_TAG) {
if ($parsedParameters['actor']['id'] === $this->activityManager->getCurrentUserId()) {
$event->setRichSubject($this->l->t('You updated system tag {oldsystemtag} to {newsystemtag}'), $parsedParameters);
$event->setParsedSubject($this->l->t('You updated system tag %2$s to %1$s', [
$this->generatePlainSystemTag($parsedParameters['newsystemtag']),
$this->generatePlainSystemTag($parsedParameters['oldsystemtag']),
]))
->setRichSubject($this->l->t('You updated system tag {oldsystemtag} to {newsystemtag}'), $parsedParameters);
} else {
$event->setRichSubject($this->l->t('{actor} updated system tag {oldsystemtag} to {newsystemtag}'), $parsedParameters);
$event->setParsedSubject($this->l->t('%1$s updated system tag %3$s to %2$s', [
$parsedParameters['actor']['name'],
$this->generatePlainSystemTag($parsedParameters['newsystemtag']),
$this->generatePlainSystemTag($parsedParameters['oldsystemtag']),
]))
->setRichSubject($this->l->t('{actor} updated system tag {oldsystemtag} to {newsystemtag}'), $parsedParameters);
}
} elseif ($event->getSubject() === self::ASSIGN_TAG) {
if ($parsedParameters['actor']['id'] === '') {
unset($parsedParameters['actor']);
$event->setRichSubject($this->l->t('System tag {systemtag} was added to {file} by the system'), $parsedParameters);
$event->setParsedSubject($this->l->t('System tag %2$s was added to %1$s by the system', [
$parsedParameters['file']['path'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('System tag {systemtag} was added to {file} by the system'), $parsedParameters);
} elseif ($parsedParameters['actor']['id'] === $this->activityManager->getCurrentUserId()) {
$event->setRichSubject($this->l->t('You added system tag {systemtag} to {file}'), $parsedParameters);
$event->setParsedSubject($this->l->t('You added system tag %2$s to %1$s', [
$parsedParameters['file']['path'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('You added system tag {systemtag} to {file}'), $parsedParameters);
} else {
$event->setRichSubject($this->l->t('{actor} added system tag {systemtag} to {file}'), $parsedParameters);
$event->setParsedSubject($this->l->t('%1$s added system tag %3$s to %2$s', [
$parsedParameters['actor']['name'],
$parsedParameters['file']['path'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('{actor} added system tag {systemtag} to {file}'), $parsedParameters);
}
} elseif ($event->getSubject() === self::UNASSIGN_TAG) {
if ($parsedParameters['actor']['id'] === '') {
unset($parsedParameters['actor']);
$event->setRichSubject($this->l->t('System tag {systemtag} was removed from {file} by the system'), $parsedParameters);
$event->setParsedSubject($this->l->t('System tag %2$s was removed from %1$s by the system', [
$parsedParameters['file']['path'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('System tag {systemtag} was removed from {file} by the system'), $parsedParameters);
} elseif ($parsedParameters['actor']['id'] === $this->activityManager->getCurrentUserId()) {
$event->setRichSubject($this->l->t('You removed system tag {systemtag} from {file}'), $parsedParameters);
$event->setParsedSubject($this->l->t('You removed system tag %2$s from %1$s', [
$parsedParameters['file']['path'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('You removed system tag {systemtag} from {file}'), $parsedParameters);
} else {
$event->setRichSubject($this->l->t('{actor} removed system tag {systemtag} from {file}'), $parsedParameters);
$event->setParsedSubject($this->l->t('%1$s removed system tag %3$s from %2$s', [
$parsedParameters['actor']['name'],
$parsedParameters['file']['path'],
$this->generatePlainSystemTag($parsedParameters['systemtag']),
]))
->setRichSubject($this->l->t('{actor} removed system tag {systemtag} from {file}'), $parsedParameters);
}
} else {
throw new \InvalidArgumentException();
Expand Down