Skip to content
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.

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

Fehler und Warnanzeige bei gelöschtem Account mit Buchungen #1688

Open
printpagestopdf opened this issue Dec 6, 2024 · 0 comments
Open
Labels
bug Something isn't working triage

Comments

@printpagestopdf
Copy link
Contributor

Plugin Version (or master commit hash): 2.9.4 und früher

PHP Version: 7.4

Über die Account Verwaltung können User ihren Account löschen. Die Anzeige im Backend führt bei der Liste der Buchungen in der Spalte Nutzende*r zu einer Fehlerausgabe.
Wird die Buchung angezeigt wird ebenfalls ein Fehler ausgegeben.
Da hier Exceptions geworfen werden kann es bei mehrfachem Aufruf dazu kommen, dass WP die Fehlermail "Ihre Site hat einen schwerwiegenden Fehler ..." sendet.

Steps to reproduce the behavior:

Löschen eines User mit vorhandenen Buchungen => Anzeige der Buchungsliste / Anzeige der Buchung

Lösungshinweise:
src/Wordpress/CustomPostType/Booking.php
Zeile 609 ersetzen gegen z.B:
if($bookingUser === false) echo '<span>**Deleted**</span>'; else echo '<a href="' . get_edit_user_link( $bookingUser->ID ) . '">' . commonsbooking_sanitizeHTML( $bookingUser->user_login ) . '</a>';

src/CB/CB.php
hinzufügen nach Zeile 177:
if($cb_user === false) return '??Deleted??';

Möglicherweise gibt es noch weitere Stellen im Code, die bei gelöschtem Account Probleme verursachen; wurde von mir nicht weiter untersucht.

Screenshots

If applicable, add screenshots to help explain your problem.
grafik

grafik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage
Projects
Development

No branches or pull requests

1 participant