diff --git a/classes/TandemManager.class.php b/classes/TandemManager.class.php index 945c68ad684bf4f0b9509ebb21f21bd42f2d2e1f..0507a97b6eb332d388fc368b4f77e34561016091 100644 --- a/classes/TandemManager.class.php +++ b/classes/TandemManager.class.php @@ -215,7 +215,11 @@ class TandemManager $message_text = sprintf( 'Your request for a tandem partnership with %s for the language %s has been accepted!', - $pair->offer->user->getFullName(), + ( + $pair->offer->user + ? $pair->offer->user->getFullName() + : 'unknown' + ), ($pair->request->target_language->name_eng ? $pair->request->target_language->name_eng : $pair->request->target_language->name) @@ -229,7 +233,11 @@ class TandemManager $message_text = sprintf( 'Deine Anfrage zur Tandem-Bildung mit %s für die Sprache %s wurde angenommen!', - $pair->offer->user->getFullName(), + ( + $pair->offer->user + ? $pair->offer->user->getFullName() + : 'unbekannt' + ), ($pair->request->target_language->name_ger ? $pair->request->target_language->name_ger : $pair->request->target_language->name) @@ -262,7 +270,11 @@ class TandemManager $message_text = sprintf( 'You have accepted the request for a tandem partnership with %s for the language %s!', - $pair->request->user->getFullName(), + ( + $pair->request->user + ? $pair->request->user->getFullName() + : 'unknown' + ), ($pair->offer->target_language->name_eng ? $pair->offer->target_language->name_eng : $pair->offer->target_language->name) @@ -275,7 +287,11 @@ class TandemManager $message_text = sprintf( 'Du hast die Anfrage zur Tandem-Bildung mit %s für die Sprache %s angenommen!', - $pair->request->user->getFullName(), + ( + $pair->request->user + ? $pair->request->user->getFullName() + : 'unbekannt' + ), ($pair->offer->target_language->name_ger ? $pair->offer->target_language->name_ger : $pair->offer->target_language->name) @@ -332,8 +348,16 @@ class TandemManager $admin_message_text = sprintf( '%s and %s have created a new tandem for the language combination %s - %s!', - $pair->request->user->getFullName(), - $pair->offer->user->getFullName(), + ( + $pair->request->user + ? $pair->request->user->getFullName() + : 'unknown' + ), + ( + $pair->offer->user + ? $pair->offer->user->getFullName() + : 'unknown' + ), ($pair->request->target_language->name_eng ? $pair->request->target_language->name_eng : $pair->request->target_language->name), @@ -346,8 +370,16 @@ class TandemManager $admin_message_text = sprintf( '%s und %s haben ein neues Tandem für die Sprachkombination %s - %s gebildet!', - $pair->request->user->getFullName(), - $pair->offer->user->getFullName(), + ( + $pair->request->user + ? $pair->request->user->getFullName() + : 'unbekannt' + ), + ( + $pair->offer->user + ? $pair->offer->user->getFullName() + : 'unbekannt' + ), ($pair->request->target_language->name_ger ? $pair->request->target_language->name_ger : $pair->request->target_language->name), @@ -382,8 +414,16 @@ class TandemManager $admin_message_title = 'Ein neues Tandem wurde gebildet!'; $admin_message_text = sprintf( '%s und %s haben ein neues Tandem für die Sprachkombination %s - %s gebildet!', - $pair->request->user->getFullName(), - $pair->offer->user->getFullName(), + ( + $pair->request->user + ? $pair->request->user->getFullName() + : 'unbekannt' + ), + ( + $pair->offer->user + ? $pair->offer->user->getFullName() + : 'unbekannt' + ), ($pair->request->target_language->name_ger ? $pair->request->target_language->name_ger : $pair->request->target_language->name), @@ -407,6 +447,7 @@ class TandemManager /** * This method handles termination of a tandem pair. * + * @return True on success, false on failure. */ public static function terminatePair(TandemPair $pair, User $user, $delete_users_profile = false) @@ -433,7 +474,11 @@ class TandemManager $message_text = sprintf( '%s has terminated the tandem for the language %s!', - $user->getFullName(), + ( + $user + ? $user->getFullName() + : 'unknown' + ), ($target_language->name_eng ? $target_language->name_eng : $target_language->name) @@ -446,7 +491,11 @@ class TandemManager $message_text = sprintf( '%s hat das Tandem für die Sprache %s aufgelöst!', - $user->getFullName(), + ( + $user + ? $user->getFullName() + : 'unbekannt' + ), ($target_language->name_ger ? $target_language->name_ger : $target_language->name) @@ -574,8 +623,16 @@ class TandemManager $admin_message_text = sprintf( '%s und %s haben ihr Tandem für die Sprachkombination %s - %s aufgelöst!', - $request_profile->user->getFullName(), - $offer_profile->user->getFullName(), + ( + $request_profile->user + ? $request_profile->user->getFullName() + : 'unbekannt' + ), + ( + $offer_profile->user + ? $offer_profile->user->getFullName() + : 'unbekannt' + ), ($request_profile->target_language->name_ger ? $request_profile->target_language->name_ger : $request_profile->target_language->name), diff --git a/controllers/admin.php b/controllers/admin.php index aaa748857ec92a47e960c24092941293ad6d315c..d9b719063f2b73ee8c0fed4fe6af5471693d9c1e 100644 --- a/controllers/admin.php +++ b/controllers/admin.php @@ -447,16 +447,18 @@ class AdminController extends PluginController foreach ($this->search_results as $search_result) { $csv_array[] = [ - ($search_result->user - ? $search_result->user->getFullName() - : dgettext('TandemPlugin', 'Unbekannter Nutzer') + ( + $search_result->user + ? $search_result->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') ), $search_result->user->email, $this->mother_language->getLocalName() . ' - ' . - ($search_result->target_language - ? $search_result->target_language->getLocalName() - : dgettext('TandemPlugin', 'Unbekannte Sprache') + ( + $search_result->target_language + ? $search_result->target_language->getLocalName() + : dgettext('TandemPlugin', 'Unbekannte Sprache') ), date('d.m.Y H:i', $search_result->mkdate) ]; diff --git a/controllers/pair.php b/controllers/pair.php index c76feb4c98de66676dc8ea2f501d29dea9c971ff..48f9663628a12b070f64f2be64a1a6c86dc344ec 100644 --- a/controllers/pair.php +++ b/controllers/pair.php @@ -64,8 +64,12 @@ class PairController extends PluginController if($num_matching_requests == 0) { PageLayout::postError( sprintf( - dgettext('TandemPlugin', 'Sie haben keine passende Anfrage für das Angebot von Nutzer %s!'), - $this->offer_profile->user->getFullName() + dgettext('TandemPlugin', 'Sie haben keine passende Anfrage für das Angebot von %s!'), + ( + $this->offer_profile->user + ? $this->offer_profile->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ) ) ); @@ -93,7 +97,11 @@ class PairController extends PluginController sprintf( dgettext('TandemPlugin', '%s ihrer Gesuche passen auf das gewählte Angebot. Bitte wählen Sie das Gesuch aus, welches sie mit dem Angebot von %s verknüpfen wollen.'), $num_matching_requests, - $this->offer_profile->user->getFullName() + ( + $this->offer_profile->user + ? $this->offer_profile->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ) ) ); } @@ -181,8 +189,12 @@ class PairController extends PluginController PageLayout::postError( dgettext('TandemPlugin', 'Tandem kann nicht gebildet werden!') . ' ' . sprintf( - dgettext('TandemPlugin', 'Keine Ihrer Muttersprachen passt auf das Gesuch von Nutzer %s!'), - $this->offer_profile->user->getFullName() + dgettext('TandemPlugin', 'Keine Ihrer Muttersprachen passt auf das Gesuch von %s!'), + ( + $this->offer_profile->user + ? $this->offer_profile->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ) ) ); } @@ -199,8 +211,12 @@ class PairController extends PluginController PageLayout::postError( dgettext('TandemPlugin', 'Tandem kann nicht gebildet werden!') . ' ' . sprintf( - dgettext('TandemPlugin', 'Nutzer %s spricht nicht die Sprache, zu der Sie ein Tandem suchen!'), - $this->offer_profile->user->getFullName() + dgettext('TandemPlugin', '%s spricht nicht die Sprache, zu der Sie ein Tandem suchen!'), + ( + $this->offer_profile->user + ? $this->offer_profile->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ) ) ); } @@ -221,7 +237,11 @@ class PairController extends PluginController PageLayout::postError( sprintf( dgettext('TandemPlugin', 'Für das Angebot von %1$s in der Sprache %2$s wurde bereits eine Anfrage erstellt!'), - $this->offer_profile->user->getFullName(), + ( + $this->offer_profile->user + ? $this->offer_profile->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ), $this->offer_profile->target_language->getLocalName() ) ); @@ -289,14 +309,22 @@ class PairController extends PluginController PageLayout::postSuccess( sprintf( dgettext('TandemPlugin', 'Sie haben %s eine Anfrage zur Tandem-Bildung geschickt!'), - $this->offer_profile->user->getFullName() + ( + $this->offer_profile->user + ? $this->offer_profile->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ) ) ); } else { PageLayout::postError( sprintf( dgettext('TandemPlugin', 'Beim Erstellen der Anfrage zur Tandem-Bildung an %s ist ein Fehler aufgetreten!'), - $this->offer_profile->user->getFullName() + ( + $this->offer_profile->user + ? $this->offer_profile->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ) ) ); } diff --git a/plugin.manifest b/plugin.manifest index d5b6770f4e209531c6965b8a6d7f51ecbde02001..34af76d7fe8f4394e7364bbc064c02314d6c8ade 100644 --- a/plugin.manifest +++ b/plugin.manifest @@ -1,7 +1,7 @@ pluginname=TandemPlugin pluginclassname=TandemPlugin origin=data-quest -version=1.0.6 +version=1.0.7 description=Dieses Plugin ermöglicht es, Sprachtandems innerhalb der Stud.IP Platform zu bilden. studipMinVersion=4.0 studipMaxVersion=4.99.99 diff --git a/templates/start.php b/templates/start.php index 5519a084b74564960d2f65b5ac414f574471cebd..9bdec36a97990ce7d7e129e907dee161daa3c4f4 100644 --- a/templates/start.php +++ b/templates/start.php @@ -15,9 +15,13 @@ <?= sprintf( dgettext('TandemPlugin', 'Bestehendes Tandem mit %s'), ($pairs[$profile->id]->request->user_id == $current_user_id) - ? htmlReady($pairs[$profile->id]->request->user->getFullName()) - : htmlReady($pairs[$profile->id]->offer->user->getFullName()) - ) ?> + ? htmlReady($pairs[$profile->id]->request->user->getFullName()) + : htmlReady( + $pairs[$profile->id]->offer->user + ? $pairs[$profile->id]->offer->user->getFullName() + : dgettext('TandemPlugin', 'unbekannt') + ) + ) ?> <? elseif($pair_requests[$profile->id] > 0): ?> <? if($pair_requests[$profile->id] == 1): ?> <?= dgettext('TandemPlugin', 'Es gibt eine Anfrage zur Tandem-Bildung!') ?> diff --git a/views/_common/_profile.php b/views/_common/_profile.php index bb88acb98390f18931a47e680505e4e48fcbe1c5..5e097ff49169787b1f167cc3971e2355d476a5ae 100644 --- a/views/_common/_profile.php +++ b/views/_common/_profile.php @@ -11,7 +11,7 @@ <strong><?= htmlReady($profile->user->getFullName()) ?></strong> </a> <? else: ?> - <strong><?= dgettext('TandemPlugin', 'Unbekannter Nutzer') ?></strong> + <strong><?= dgettext('TandemPlugin', 'unbekannt') ?></strong> <? endif ?> <? if($established_pair or $admin_view): ?> <? if ($profile->user) : ?> diff --git a/views/admin/search.php b/views/admin/search.php index f98c20f16a094b29dc9b341a902151c4596eb641..0d94fa36da220cd1a350fc61eaca862c8822f96a 100644 --- a/views/admin/search.php +++ b/views/admin/search.php @@ -68,7 +68,7 @@ <td> <?= $result->user ? htmlReady($result->user->getFullName()) - : dgettext('TandemPlugin', 'Unbekannter Nutzer') ?> + : dgettext('TandemPlugin', 'unbekannt') ?> </td> <td> <a href="mailto:<?= htmlReady($result->user->email) ?>">