Skip to content
Snippets Groups Projects
Commit 6eb10c5a authored by Moritz Strohm's avatar Moritz Strohm
Browse files

added more checks in plugin code in case one of the tandem users have been... 

added more checks in plugin code in case one of the tandem users have been deleted, hopefully really fixes #12
parent 744e9731
No related branches found
No related tags found
No related merge requests found
Show whitespace changes
Inline Side-by-side
classes/TandemManager.class.php
+ 71
− 14
View file @ 6eb10c5a
...@@ -215,7 +215,11 @@ class TandemManager ...@@ -215,7 +215,11 @@ class TandemManager
$message_text = sprintf( $message_text = sprintf(
'Your request for a tandem partnership with %s for the language %s has been accepted!', '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_eng ? $pair->request->target_language->name_eng
: $pair->request->target_language->name) : $pair->request->target_language->name)
...@@ -229,7 +233,11 @@ class TandemManager ...@@ -229,7 +233,11 @@ class TandemManager
$message_text = sprintf( $message_text = sprintf(
'Deine Anfrage zur Tandem-Bildung mit %s für die Sprache %s wurde angenommen!', '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_ger ? $pair->request->target_language->name_ger
: $pair->request->target_language->name) : $pair->request->target_language->name)
...@@ -262,7 +270,11 @@ class TandemManager ...@@ -262,7 +270,11 @@ class TandemManager
$message_text = sprintf( $message_text = sprintf(
'You have accepted the request for a tandem partnership with %s for the language %s!', '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_eng ? $pair->offer->target_language->name_eng
: $pair->offer->target_language->name) : $pair->offer->target_language->name)
...@@ -275,7 +287,11 @@ class TandemManager ...@@ -275,7 +287,11 @@ class TandemManager
$message_text = sprintf( $message_text = sprintf(
'Du hast die Anfrage zur Tandem-Bildung mit %s für die Sprache %s angenommen!', '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_ger ? $pair->offer->target_language->name_ger
: $pair->offer->target_language->name) : $pair->offer->target_language->name)
...@@ -332,8 +348,16 @@ class TandemManager ...@@ -332,8 +348,16 @@ class TandemManager
$admin_message_text = sprintf( $admin_message_text = sprintf(
'%s and %s have created a new tandem for the language combination %s - %s!', '%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_eng ? $pair->request->target_language->name_eng
: $pair->request->target_language->name), : $pair->request->target_language->name),
...@@ -346,8 +370,16 @@ class TandemManager ...@@ -346,8 +370,16 @@ class TandemManager
$admin_message_text = sprintf( $admin_message_text = sprintf(
'%s und %s haben ein neues Tandem für die Sprachkombination %s - %s gebildet!', '%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_ger ? $pair->request->target_language->name_ger
: $pair->request->target_language->name), : $pair->request->target_language->name),
...@@ -382,8 +414,16 @@ class TandemManager ...@@ -382,8 +414,16 @@ class TandemManager
$admin_message_title = 'Ein neues Tandem wurde gebildet!'; $admin_message_title = 'Ein neues Tandem wurde gebildet!';
$admin_message_text = sprintf( $admin_message_text = sprintf(
'%s und %s haben ein neues Tandem für die Sprachkombination %s - %s gebildet!', '%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_ger ? $pair->request->target_language->name_ger
: $pair->request->target_language->name), : $pair->request->target_language->name),
...@@ -407,6 +447,7 @@ class TandemManager ...@@ -407,6 +447,7 @@ class TandemManager
/** /**
* This method handles termination of a tandem pair. * This method handles termination of a tandem pair.
* *
* @return True on success, false on failure. * @return True on success, false on failure.
*/ */
public static function terminatePair(TandemPair $pair, User $user, $delete_users_profile = false) public static function terminatePair(TandemPair $pair, User $user, $delete_users_profile = false)
...@@ -433,7 +474,11 @@ class TandemManager ...@@ -433,7 +474,11 @@ class TandemManager
$message_text = sprintf( $message_text = sprintf(
'%s has terminated the tandem for the language %s!', '%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_eng ? $target_language->name_eng
: $target_language->name) : $target_language->name)
...@@ -446,7 +491,11 @@ class TandemManager ...@@ -446,7 +491,11 @@ class TandemManager
$message_text = sprintf( $message_text = sprintf(
'%s hat das Tandem für die Sprache %s aufgelöst!', '%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_ger ? $target_language->name_ger
: $target_language->name) : $target_language->name)
...@@ -574,8 +623,16 @@ class TandemManager ...@@ -574,8 +623,16 @@ class TandemManager
$admin_message_text = sprintf( $admin_message_text = sprintf(
'%s und %s haben ihr Tandem für die Sprachkombination %s - %s aufgelöst!', '%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_ger ? $request_profile->target_language->name_ger
: $request_profile->target_language->name), : $request_profile->target_language->name),
......
controllers/admin.php
+ 8
− 6
View file @ 6eb10c5a
...@@ -447,14 +447,16 @@ class AdminController extends PluginController ...@@ -447,14 +447,16 @@ class AdminController extends PluginController
foreach ($this->search_results as $search_result) { foreach ($this->search_results as $search_result) {
$csv_array[] = [ $csv_array[] = [
($search_result->user (
$search_result->user
? $search_result->user->getFullName() ? $search_result->user->getFullName()
: dgettext('TandemPlugin', 'Unbekannter Nutzer') : dgettext('TandemPlugin', 'unbekannt')
), ),
$search_result->user->email, $search_result->user->email,
$this->mother_language->getLocalName() . $this->mother_language->getLocalName() .
' - ' . ' - ' .
($search_result->target_language (
$search_result->target_language
? $search_result->target_language->getLocalName() ? $search_result->target_language->getLocalName()
: dgettext('TandemPlugin', 'Unbekannte Sprache') : dgettext('TandemPlugin', 'Unbekannte Sprache')
), ),
......
controllers/pair.php
+ 38
− 10
View file @ 6eb10c5a
...@@ -64,8 +64,12 @@ class PairController extends PluginController ...@@ -64,8 +64,12 @@ class PairController extends PluginController
if($num_matching_requests == 0) { if($num_matching_requests == 0) {
PageLayout::postError( PageLayout::postError(
sprintf( sprintf(
dgettext('TandemPlugin', 'Sie haben keine passende Anfrage für das Angebot von Nutzer %s!'), dgettext('TandemPlugin', 'Sie haben keine passende Anfrage für das Angebot von %s!'),
$this->offer_profile->user->getFullName() (
$this->offer_profile->user
? $this->offer_profile->user->getFullName()
: dgettext('TandemPlugin', 'unbekannt')
)
) )
); );
...@@ -93,7 +97,11 @@ class PairController extends PluginController ...@@ -93,7 +97,11 @@ class PairController extends PluginController
sprintf( 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.'), 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, $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 ...@@ -181,8 +189,12 @@ class PairController extends PluginController
PageLayout::postError( PageLayout::postError(
dgettext('TandemPlugin', 'Tandem kann nicht gebildet werden!') . ' ' . dgettext('TandemPlugin', 'Tandem kann nicht gebildet werden!') . ' ' .
sprintf( sprintf(
dgettext('TandemPlugin', 'Keine Ihrer Muttersprachen passt auf das Gesuch von Nutzer %s!'), dgettext('TandemPlugin', 'Keine Ihrer Muttersprachen passt auf das Gesuch von %s!'),
$this->offer_profile->user->getFullName() (
$this->offer_profile->user
? $this->offer_profile->user->getFullName()
: dgettext('TandemPlugin', 'unbekannt')
)
) )
); );
} }
...@@ -199,8 +211,12 @@ class PairController extends PluginController ...@@ -199,8 +211,12 @@ class PairController extends PluginController
PageLayout::postError( PageLayout::postError(
dgettext('TandemPlugin', 'Tandem kann nicht gebildet werden!') . ' ' . dgettext('TandemPlugin', 'Tandem kann nicht gebildet werden!') . ' ' .
sprintf( sprintf(
dgettext('TandemPlugin', 'Nutzer %s spricht nicht die Sprache, zu der Sie ein Tandem suchen!'), dgettext('TandemPlugin', '%s spricht nicht die Sprache, zu der Sie ein Tandem suchen!'),
$this->offer_profile->user->getFullName() (
$this->offer_profile->user
? $this->offer_profile->user->getFullName()
: dgettext('TandemPlugin', 'unbekannt')
)
) )
); );
} }
...@@ -221,7 +237,11 @@ class PairController extends PluginController ...@@ -221,7 +237,11 @@ class PairController extends PluginController
PageLayout::postError( PageLayout::postError(
sprintf( sprintf(
dgettext('TandemPlugin', 'Für das Angebot von %1$s in der Sprache %2$s wurde bereits eine Anfrage erstellt!'), 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() $this->offer_profile->target_language->getLocalName()
) )
); );
...@@ -289,14 +309,22 @@ class PairController extends PluginController ...@@ -289,14 +309,22 @@ class PairController extends PluginController
PageLayout::postSuccess( PageLayout::postSuccess(
sprintf( sprintf(
dgettext('TandemPlugin', 'Sie haben %s eine Anfrage zur Tandem-Bildung geschickt!'), 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 { } else {
PageLayout::postError( PageLayout::postError(
sprintf( sprintf(
dgettext('TandemPlugin', 'Beim Erstellen der Anfrage zur Tandem-Bildung an %s ist ein Fehler aufgetreten!'), 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')
)
) )
); );
} }
......
plugin.manifest
+ 1
− 1
View file @ 6eb10c5a
pluginname=TandemPlugin pluginname=TandemPlugin
pluginclassname=TandemPlugin pluginclassname=TandemPlugin
origin=data-quest 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. description=Dieses Plugin ermöglicht es, Sprachtandems innerhalb der Stud.IP Platform zu bilden.
studipMinVersion=4.0 studipMinVersion=4.0
studipMaxVersion=4.99.99 studipMaxVersion=4.99.99
templates/start.php
+ 7
− 3
View file @ 6eb10c5a
...@@ -16,7 +16,11 @@ ...@@ -16,7 +16,11 @@
dgettext('TandemPlugin', 'Bestehendes Tandem mit %s'), dgettext('TandemPlugin', 'Bestehendes Tandem mit %s'),
($pairs[$profile->id]->request->user_id == $current_user_id) ($pairs[$profile->id]->request->user_id == $current_user_id)
? htmlReady($pairs[$profile->id]->request->user->getFullName()) ? htmlReady($pairs[$profile->id]->request->user->getFullName())
: htmlReady($pairs[$profile->id]->offer->user->getFullName()) : htmlReady(
$pairs[$profile->id]->offer->user
? $pairs[$profile->id]->offer->user->getFullName()
: dgettext('TandemPlugin', 'unbekannt')
)
) ?> ) ?>
<? elseif($pair_requests[$profile->id] > 0): ?> <? elseif($pair_requests[$profile->id] > 0): ?>
<? if($pair_requests[$profile->id] == 1): ?> <? if($pair_requests[$profile->id] == 1): ?>
......
views/_common/_profile.php
+ 1
− 1
View file @ 6eb10c5a
...@@ -11,7 +11,7 @@ ...@@ -11,7 +11,7 @@
<strong><?= htmlReady($profile->user->getFullName()) ?></strong> <strong><?= htmlReady($profile->user->getFullName()) ?></strong>
</a> </a>
<? else: ?> <? else: ?>
<strong><?= dgettext('TandemPlugin', 'Unbekannter Nutzer') ?></strong> <strong><?= dgettext('TandemPlugin', 'unbekannt') ?></strong>
<? endif ?> <? endif ?>
<? if($established_pair or $admin_view): ?> <? if($established_pair or $admin_view): ?>
<? if ($profile->user) : ?> <? if ($profile->user) : ?>
......
views/admin/search.php
+ 1
− 1
View file @ 6eb10c5a
...@@ -68,7 +68,7 @@ ...@@ -68,7 +68,7 @@
<td> <td>
<?= $result->user <?= $result->user
? htmlReady($result->user->getFullName()) ? htmlReady($result->user->getFullName())
: dgettext('TandemPlugin', 'Unbekannter Nutzer') ?> : dgettext('TandemPlugin', 'unbekannt') ?>
</td> </td>
<td> <td>
<a href="mailto:<?= htmlReady($result->user->email) ?>"> <a href="mailto:<?= htmlReady($result->user->email) ?>">
......
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment