Additional checks when a report is submitted (e.g. no empty description)

git-svn-id: http://www.winterrodeln.org/svn/servermediawiki/trunk/extensions/wrreport@287 7aebc617-e5e2-0310-91dc-80fb5f6d2477

diff --git a/wrreport.body.php b/wrreport.body.php
index d554e6542f1b89118a64a2bd3c36ee5792f6e199..0b554ce27ef4d7db25fb61258d0a4f9971971670 100644 (file)
--- a/wrreport.body.php
+++ b/wrreport.body.php
@@ -267,7 +267,7 @@ class WrReport extends SpecialPage {
 
                // Show error message
                if ($errorMsg) {
-                       $wgOut->addWikiText(utf8_encode('<div class="errorbox">') . $errorMsg . utf8_encode("</div>\n\n"));
+                       $wgOut->addWikiText(utf8_encode('<div class="errorbox">') . $errorMsg . utf8_encode("</div>\n"));
                }
 
                // Action view
@@ -302,8 +302,10 @@ class WrReport extends SpecialPage {
                                // check conditions/permissions
                                $errorMsg = NULL;
                                if (!$page_id) $errorMsg = utf8_encode('Die angegebene Seite wurde nicht gefunden.');
+                               elseif (!$condition && !$description) $errorMsg = utf8_encode('Es sind sowohl die Beschreibung als auch die Bewertung leer/nicht vergeben.');
                                elseif (!$wgUser->isLoggedIn()) {
-                                       if (!(stripos($description, 'http') === FALSE))  $errorMsg = utf8_encode('Der Text "http" ist leider für nicht-angemeldete Benutzer nicht erlaubt, weil wir in der Vergangenheit Fälle hatten, bei denen externe Werbelinks automatisiert eingetragen wurden.');
+                                       if (!$description) $errorMsg = utf8_encode('Bitte bei der Beschreibung eine kurze Begründung für die Bewertung abgeben.');
+                                       elseif (!(stripos($description, 'http') === FALSE)) $errorMsg = utf8_encode('Der Text "http" ist leider für nicht-angemeldete Benutzer nicht erlaubt, weil wir in der Vergangenheit Fälle hatten, bei denen externe Werbelinks automatisiert eingetragen wurden.');
                                }
                                if ($errorMsg) {
                                        $this->execute($par, 'preview', $errorMsg);
@@ -332,7 +334,7 @@ class WrReport extends SpecialPage {
                                // Purge cache
                                $title->invalidateCache();
                                // Show success message
-                               $wgOut->addWikiText(utf8_encode('<div class="successbox">Der Bahnbericht für [[') . $page_title . utf8_encode('#Einträge|') . $page_title . utf8_encode("]] wurde erfolgreich gespeichert.</div>\n\n"));
+                               $wgOut->addWikiText(utf8_encode('<div class="successbox">Der Bahnbericht für [[') . $page_title . utf8_encode('#Einträge|') . $page_title . utf8_encode("]] wurde erfolgreich gespeichert.</div>\n"));
                                // We could redirect to result with the following line but we don't want to.
                                // $wgOut->redirect($title->getFullURL() . '#Eintr.C3.A4ge');
                        }