Darstellung bei Korrektur und Nachricht nicht verändert

lookshe commented

2025-06-12 17:30:31 +02:00

Owner

Conversations:
nach Klick: https://up.picn.de/files/3a4bf62f-37ab-439b-b0bd-560d0380f731/Screenshot_20250612-172516_Conversations.png
nach Löschen des Inhalts: https://up.picn.de/files/60149a0c-2679-4e2d-b0ad-156e94d22b75/Screenshot_20250612-172532_Conversations.png

PiratX:
nach klick: https://up.picn.de/files/b280ae15-ce6b-4f39-8fb8-c0fd9640a6b4/Screenshot_20250612-172616_PiratX%20Debug.png
nach Löschen des Inhalts: https://up.picn.de/files/17f5eb48-6b1f-46a9-b1bc-22c6c448acd6/Screenshot_20250612-172622_PiratX%20Debug.png

Erwartung: in beiden Fällen das gleiche "X" zum Abbrechen und bei leerer Nachricht auch nur das "X" und keine weiteren Icons.

Conversations: nach Klick: https://up.picn.de/files/3a4bf62f-37ab-439b-b0bd-560d0380f731/Screenshot_20250612-172516_Conversations.png nach Löschen des Inhalts: https://up.picn.de/files/60149a0c-2679-4e2d-b0ad-156e94d22b75/Screenshot_20250612-172532_Conversations.png PiratX: nach klick: https://up.picn.de/files/b280ae15-ce6b-4f39-8fb8-c0fd9640a6b4/Screenshot_20250612-172616_PiratX%20Debug.png nach Löschen des Inhalts: https://up.picn.de/files/17f5eb48-6b1f-46a9-b1bc-22c6c448acd6/Screenshot_20250612-172622_PiratX%20Debug.png Erwartung: in beiden Fällen das gleiche "X" zum Abbrechen und bei leerer Nachricht auch nur das "X" und keine weiteren Icons.

tristan added a new dependency

2025-06-12 18:01:45 +02:00

#59 Beim Editieren einer Nachricht gibt es keine Indikation, dass aktuell editiert wird

lookshe commented

2025-06-12 22:57:21 +02:00

Author

Owner

Es fehlt vermutlich ein Aufruf zu

public void updateChatMsgHint() {

Es fehlt vermutlich ein Aufruf zu https://git.fucktheforce.de/thedevstack/PiratX/src/commit/0a8d000c8a7d0add89fa413bd061764422edd2d3/src/main/java/eu/siacs/conversations/ui/ConversationFragment.java#L1435

lookshe commented

2025-06-13 12:47:20 +02:00

Author

Owner

Conversations:

	`if (c.getCorrectingMessage() != null`
	`&& (empty \|\| text.equals(c.getCorrectingMessage().getBody()))) {`
	`return SendButtonAction.CANCEL;`

Monocles:

	`if (c.getCorrectingMessage() != null`
	`&& (empty \|\| (text.equals(c.getCorrectingMessage().getBody()) && (subject.equals(c.getCorrectingMessage().getSubject())) && (c.getThread() == c.getCorrectingMessage().getThread() \|\| (c.getThread() != null && c.getThread().equals(c.getCorrectingMessage().getThread())))))) {`
	`return SendButtonAction.CANCEL;`

Da es mit leerem Text halbwegs passt, könnte das die betroffene Stelle sein.

Conversations: https://git.fucktheforce.de/mirror/Conversations/src/commit/0e283845bfaef6f8ff22a9bd720037337d7a74ef/src/main/java/eu/siacs/conversations/ui/util/SendButtonTool.java#L58-L60 Monocles: https://git.fucktheforce.de/thedevstack/PiratX/src/commit/6932d16734625a10cf37a0adb571d7b7c3b24ecb/src/main/java/eu/siacs/conversations/ui/util/SendButtonTool.java#L65-L67 Da es mit leerem Text halbwegs passt, könnte das die betroffene Stelle sein.

lookshe commented

2025-06-13 13:06:18 +02:00

Author

Owner

Debugger stimmt der Vermutung zu, Fix dauert aber noch.

Weiterhin offen, warum steht "Nachricht senden" statt "Korrigierte Nachricht senden".

Debugger stimmt der Vermutung zu, Fix dauert aber noch. Weiterhin offen, warum steht "Nachricht senden" statt "Korrigierte Nachricht senden".

lookshe commented

2025-06-17 12:04:26 +02:00

Author

Owner

Weiterhin offen, warum steht "Nachricht senden" statt "Korrigierte Nachricht senden".
Tritt auch bei Conversations auf, bei meinen Tests für die Screenshots muss ich wohl irgendwie updateChatMsgHint() getriggert haben.
Bei beiden Apps ist es nämlich bei kurzzeitigem Abschalten des Telefons korrekt.

Das Problem wird daher in einem anderen Ticket weitergeführt.

> Weiterhin offen, warum steht "Nachricht senden" statt "Korrigierte Nachricht senden". Tritt auch bei Conversations auf, bei meinen Tests für die Screenshots muss ich wohl irgendwie `updateChatMsgHint()` getriggert haben. Bei beiden Apps ist es nämlich bei kurzzeitigem Abschalten des Telefons korrekt. Das Problem wird daher in einem anderen Ticket weitergeführt.