[PLUGIN][RepeatNote] Do not notify about clone note

Fix redirecion after action
Fix typo in activitypub handler
This commit is contained in:
Diogo Peralta Cordeiro 2021-12-28 17:49:46 +00:00
parent 182c6265a3
commit 4d8e39bf69
Signed by: diogo
GPG Key ID: 18D2D35001FBFAB0
4 changed files with 10 additions and 9 deletions

View File

@ -92,8 +92,7 @@ class Conversation extends Component
$query_string = $request->getQueryString(); $query_string = $request->getQueryString();
// Concatenating get parameter to redirect the user to where he came from // Concatenating get parameter to redirect the user to where he came from
$reply_action_url .= !\is_null($query_string) ? '?from=' : '&from='; $reply_action_url .= '?from=' . urlencode($request->getRequestUri());
$reply_action_url .= mb_substr($query_string, 2);
$reply_action = [ $reply_action = [
'url' => $reply_action_url, 'url' => $reply_action_url,

View File

@ -191,6 +191,7 @@ class Posting extends Component
array $attachments = [], array $attachments = [],
array $processed_attachments = [], array $processed_attachments = [],
array $process_note_content_extra_args = [], array $process_note_content_extra_args = [],
bool $notify = true,
): Note { ): Note {
$scope ??= VisibilityScope::EVERYWHERE; // TODO: If site is private, default to LOCAL $scope ??= VisibilityScope::EVERYWHERE; // TODO: If site is private, default to LOCAL
$rendered = null; $rendered = null;
@ -281,7 +282,10 @@ class Posting extends Component
} }
DB::flush(); DB::flush();
if ($notify) {
Event::handle('NewNotification', [$actor, $activity, ['object' => $mentioned], _m('{nickname} created a note {note_id}.', ['nickname' => $actor->getNickname(), 'note_id' => $activity->getObjectId()])]); Event::handle('NewNotification', [$actor, $activity, ['object' => $mentioned], _m('{nickname} created a note {note_id}.', ['nickname' => $actor->getNickname(), 'note_id' => $activity->getObjectId()])]);
}
return $note; return $note;
} }

View File

@ -109,7 +109,7 @@ class Favourite extends NoteHandlerPlugin
$query_string = $request->getQueryString(); $query_string = $request->getQueryString();
// Concatenating get parameter to redirect the user to where he came from // Concatenating get parameter to redirect the user to where he came from
$favourite_action_url .= !\is_null($query_string) ? '?from=' . mb_substr($query_string, 2) : ''; $favourite_action_url .= '?from=' . urlencode($request->getRequestUri());
$extra_classes = $is_favourite ? 'note-actions-set' : 'note-actions-unset'; $extra_classes = $is_favourite ? 'note-actions-set' : 'note-actions-unset';
$favourite_action = [ $favourite_action = [

View File

@ -65,6 +65,7 @@ class RepeatNote extends NoteHandlerPlugin
language: \is_null($lang_id = $note->getLanguageId()) ? null : Language::getById($lang_id)->getLocale(), language: \is_null($lang_id = $note->getLanguageId()) ? null : Language::getById($lang_id)->getLocale(),
processed_attachments: $note->getAttachmentsWithTitle(), processed_attachments: $note->getAttachmentsWithTitle(),
process_note_content_extra_args: $extra_args, process_note_content_extra_args: $extra_args,
notify: false,
); );
DB::persist(NoteRepeat::create([ DB::persist(NoteRepeat::create([
@ -191,11 +192,8 @@ class RepeatNote extends NoteHandlerPlugin
? Router::url('repeat_remove', $args, $type) ? Router::url('repeat_remove', $args, $type)
: Router::url('repeat_add', $args, $type); : Router::url('repeat_add', $args, $type);
// TODO clean this up
// SECURITY: open redirect?
$query_string = $request->getQueryString();
// Concatenating get parameter to redirect the user to where he came from // Concatenating get parameter to redirect the user to where he came from
$repeat_action_url .= !\is_null($query_string) ? '?from=' . mb_substr($query_string, 2) : ''; $repeat_action_url .= '?from=' . urlencode($request->getRequestUri());
$extra_classes = $is_repeat ? 'note-actions-set' : 'note-actions-unset'; $extra_classes = $is_repeat ? 'note-actions-set' : 'note-actions-unset';
$repeat_action = [ $repeat_action = [
@ -291,7 +289,7 @@ class RepeatNote extends NoteHandlerPlugin
} }
} elseif ($type_object instanceof Note) { } elseif ($type_object instanceof Note) {
$note = $type_object; $note = $type_object;
$note_id = ${$note}->getId(); $note_id = $note->getId();
} else { } else {
return Event::next; return Event::next;
} }