thedevstack/PiratX
3
0
Fork 0
forked from mirror/monocles_chat_clean
Code Issues 17 Pull requests 1 Wiki Activity Actions 1

Anrufbenachrichtigung hat 2x den ablehnen button #34

New issue
Closed
opened 2025-06-02 14:26:50 +02:00 by tristan · 9 comments
tristan commented 2025-06-02 14:26:50 +02:00
Owner
Copy link
No description provided.
Screenshot_20250602-133919~2.png
38 KiB
Screenshot_20250602-133919~2.png
lookshe commented 2025-06-09 21:33:46 +02:00
Owner
Copy link

Line 635 in 80ce833
private void showIncomingCallNotification(

https://git.fucktheforce.de/thedevstack/PiratX/src/commit/80ce833d28501b0f34be35a2f5a2a2927a5f6f0f/src/main/java/eu/siacs/conversations/services/NotificationService.java#L635
lookshe commented 2025-06-09 21:54:31 +02:00
Owner
Copy link

Wenn ich es richtig sehe, dann sind alle Aktionen doppelt drin. Vermutlich ausgelöst durch:

Lines 751 to 758 in 80ce833
NotificationCompat.CallStyle style = NotificationCompat.CallStyle.forIncomingCall(
getPerson(contact),
createCallAction(
id.sessionId,
XmppConnectionService.ACTION_DISMISS_CALL,
102),
createPendingRtpSession(id, RtpSessionActivity.ACTION_ACCEPT_CALL, 103)
);


Lines 796 to 811 in 80ce833
builder.addAction(
new NotificationCompat.Action.Builder(
R.drawable.ic_call_end_24dp,
mXmppConnectionService.getString(R.string.dismiss_call),
createCallAction(
id.sessionId,
XmppConnectionService.ACTION_DISMISS_CALL,
102))
.build());
builder.addAction(
new NotificationCompat.Action.Builder(
R.drawable.ic_call_24dp,
mXmppConnectionService.getString(R.string.answer_call),
createPendingRtpSession(
id, RtpSessionActivity.ACTION_ACCEPT_CALL, 103))
.build());

Wenn ich es richtig sehe, dann sind alle Aktionen doppelt drin. Vermutlich ausgelöst durch: https://git.fucktheforce.de/thedevstack/PiratX/src/commit/80ce833d28501b0f34be35a2f5a2a2927a5f6f0f/src/main/java/eu/siacs/conversations/services/NotificationService.java#L751-L758 https://git.fucktheforce.de/thedevstack/PiratX/src/commit/80ce833d28501b0f34be35a2f5a2a2927a5f6f0f/src/main/java/eu/siacs/conversations/services/NotificationService.java#L796-L811
lookshe commented 2025-06-09 22:05:20 +02:00
Owner
Copy link

Vorschlag nach kurzem Test/Debuggen: das selbst hinzufügen der Actions entfernen.
Muss dann an zwei Stellen entfernt werden in der Klasse.

@tristan Meinung dazu?

Vorschlag nach kurzem Test/Debuggen: das selbst hinzufügen der Actions entfernen. Muss dann an zwei Stellen entfernt werden in der Klasse. @tristan Meinung dazu?
lookshe commented 2025-06-09 22:11:00 +02:00
Owner
Copy link

Conversations macht es ohne die Helferklasse.

Alternativvorschlag daher NotificationCompat.CallStyle.forIncomingCall ausbauen und einmal grob mit Conversations abgleichen, ob der Notificationservice passt.
Auspassen dabei aber, es wurde eine alte Funktion wohl wieder eingebaut: 06e3b183ba

Conversations macht es ohne die Helferklasse. Alternativvorschlag daher `NotificationCompat.CallStyle.forIncomingCall` ausbauen und einmal grob mit Conversations abgleichen, ob der Notificationservice passt. Auspassen dabei aber, es wurde eine alte Funktion wohl wieder eingebaut: https://git.fucktheforce.de/thedevstack/PiratX/commit/06e3b183ba89079129b039eb9223fd1e2b8d2ea9
tristan commented 2025-06-10 09:59:17 +02:00
Author
Owner
Copy link

ich wuerde mit der Helferklasse gehen. warum alles haendisch machen wenn es eigentlich ein Template gibt?

ich wuerde mit der Helferklasse gehen. warum alles haendisch machen wenn es eigentlich ein Template gibt?
lookshe commented 2025-06-10 10:57:45 +02:00
Owner
Copy link

Grundsätzlich bin ich bei dir.
Mir macht nur ein wenig Sorgen, dass es bei Conversations anders gehandhabt wird. Und solange ich die Gründe dafür nicht kenne, nehme ich erstmal an, dass sie in irgendeiner Weise "gut" sind.

Ich konnte anhand der Doku auch noch nicht genau rausfinden, welche Teile automatisch durch den Helper gesetzt werden und welche definitiv noch händisch gesetzt werden müssen.

Ich forsche erstmal noch etwas in der History von Conversations rum, ob ich da etwas zu den Gründen finde.
Ansonsten bleibt ja nur trial&errror um alles unnötige zu entfernen.

Grundsätzlich bin ich bei dir. Mir macht nur ein wenig Sorgen, dass es bei Conversations anders gehandhabt wird. Und solange ich die Gründe dafür nicht kenne, nehme ich erstmal an, dass sie in irgendeiner Weise "gut" sind. Ich konnte anhand der Doku auch noch nicht genau rausfinden, welche Teile automatisch durch den Helper gesetzt werden und welche definitiv noch händisch gesetzt werden müssen. Ich forsche erstmal noch etwas in der History von Conversations rum, ob ich da etwas zu den Gründen finde. Ansonsten bleibt ja nur trial&errror um alles unnötige zu entfernen.
tristan commented 2025-06-10 11:37:00 +02:00
Author
Owner
Copy link

Ohne Helferklasse (App Lock aktiviert):
image

Mit Helferklasse (App Lock deaktiviert):
image

Screenshot mit Xiamo Redmi Note 8 gemacht / Android 11

Ohne Helferklasse (App Lock aktiviert): ![image](/attachments/dc58d442-ab91-4304-bbc6-6437b2596663) Mit Helferklasse (App Lock deaktiviert): ![image](/attachments/2ff08b8a-5bf4-420f-8eb4-00e1d754118a) Screenshot mit Xiamo Redmi Note 8 gemacht / Android 11
image.png
20 KiB
image.png
18 KiB
tristan commented 2025-06-10 13:11:29 +02:00
Author
Owner
Copy link

App Lock aktiviert:
image

App Lock deaktiviert:
image

Google Pixel 7a / Android 15

App Lock aktiviert: ![image](/attachments/b3dd78a6-d726-4fcb-8030-6b8a9f1f4089) App Lock deaktiviert: ![image](/attachments/08490caa-cd0a-4be8-a65c-703462c348ce) Google Pixel 7a / Android 15
image.png
16 KiB
image.png
17 KiB
lookshe commented 2025-06-10 16:59:53 +02:00
Owner
Copy link

Um es auch hier festzuhalten:
Wir bauen auf Helper um und alles andere unnötige fliegt raus.
Sollte daher am besten in einem extra Branch gemacht werden und wenn alles soweit es funktioniert dann via Squash-Commit übernommen werden.

Um es auch hier festzuhalten: Wir bauen auf Helper um und alles andere unnötige fliegt raus. Sollte daher am besten in einem extra Branch gemacht werden und wenn alles soweit es funktioniert dann via Squash-Commit übernommen werden.
lookshe self-assigned this 2025-06-10 20:55:40 +02:00
tristan added a new dependency 2025-06-10 23:47:21 +02:00
#51 App Lock entfernen?
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
No results found.
Labels
Clear labels   
update fork
No labels
update fork
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
lookshe
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Blocks
#51 App Lock entfernen?
thedevstack/PiratX
Reference: thedevstack/PiratX#34
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?