From 2ab27850ef2171954c7e04e79b4978154468251c Mon Sep 17 00:00:00 2001 From: =?utf8?q?Roland=20H=C3=A4der?= Date: Mon, 23 Jun 2008 21:39:48 +0000 Subject: [PATCH] Fixes for XHTML validity --- application/ship-simu/config.php | 4 ++-- .../templates/de/code/captch_graphic_code.ctp | 11 +++++------ .../ship-simu/templates/de/code/login_form.ctp | 18 +++++++++--------- .../templates/de/code/login_profile.ctp | 10 +++++----- .../templates/de/code/login_status_problem.ctp | 4 ++-- .../templates/de/code/register_form.ctp | 4 ++-- .../main/helper/web/class_WebFormHelper.php | 12 ++++++++++-- 7 files changed, 35 insertions(+), 28 deletions(-) diff --git a/application/ship-simu/config.php b/application/ship-simu/config.php index 2976a8b..d3d98f7 100644 --- a/application/ship-simu/config.php +++ b/application/ship-simu/config.php @@ -149,10 +149,10 @@ $cfg->setConfigEntry('email_change_confirmation', "Y"); $cfg->setConfigEntry('guest_login_allowed', "Y"); // CFG: GUEST-LOGIN-USERNAME -$cfg->setConfigEntry('guest_login_username', "guest"); +$cfg->setConfigEntry('guest_login_user', "guest"); // CFG: GUEST-LOGIN-PASS -$cfg->setConfigEntry('guest_login_pass', "guest"); +$cfg->setConfigEntry('guest_login_passwd', "guest"); // CFG: LOGIN-WELCOME-ACTION-CLASS $cfg->setConfigEntry('login_welcome_action_class', "ShipSimuLoginAction"); diff --git a/application/ship-simu/templates/de/code/captch_graphic_code.ctp b/application/ship-simu/templates/de/code/captch_graphic_code.ctp index a134e30..0e044f0 100644 --- a/application/ship-simu/templates/de/code/captch_graphic_code.ctp +++ b/application/ship-simu/templates/de/code/captch_graphic_code.ctp @@ -1,17 +1,16 @@ addFormGroup('code', ""); -$captchaHelper->addFieldText('code', "Bitte wiederhole den Code:"); -$captchaHelper->addInputTextField('code'); +$captchaHelper->addFieldText('captcha_code', "Bitte wiederhole den Code:"); +$captchaHelper->addInputTextField('captcha_code'); $captchaHelper->addInputHiddenField('hash', $this->readVariable('captcha_hash')); $captchaHelper->flushContent(); // [EOF] ?> -
- CAPTCHA-Bild +
+ CAPTCHA-Bild
-
+
{?captcha_code?}
diff --git a/application/ship-simu/templates/de/code/login_form.ctp b/application/ship-simu/templates/de/code/login_form.ctp index 4bdb82d..bf6c803 100644 --- a/application/ship-simu/templates/de/code/login_form.ctp +++ b/application/ship-simu/templates/de/code/login_form.ctp @@ -33,18 +33,18 @@ if ($helper->ifLoginIsEnabled()) { // CAPTCHA enabled? if ($helper->ifFormSecuredWithCaptcha()) { - $helper->addFormGroup('captcha', "Unser Benuzter-Login ist durch ein CAPTCHA geschützt. Bitte wiederhole den angezeigten Code, damit du dich einloggen kannst."); + $helper->addFormGroup('captcha_user', "Unser Benuzter-Login ist durch ein CAPTCHA geschützt. Bitte wiederhole den angezeigten Code, damit du dich einloggen kannst."); $helper->addCaptcha(); } // END - if // Submit buttons - $helper->addFormGroup('buttons', "Alles richtig eingegeben?"); + $helper->addFormGroup('buttons_user', "Alles richtig eingegeben?"); $helper->addInputResetButton("Formular leeren"); $helper->addInputSubmitButton("Zum Spiel einloggen"); } else { // Formular deaktiviert - $helper->addFormNote("Einloggen in's Spiel ist derzeit administrativ deaktiviert worden. Bitte komme später noch mal wieder."); - $helper->addFormNote("Nachricht vom Admin: ".$this->getConfigInstance()->readConfig('login_disabled_reason').""); + $helper->addFormNote('form_deactivated', "Einloggen in's Spiel ist derzeit administrativ deaktiviert worden. Bitte komme später noch mal wieder."); + $helper->addFormNote('admin_notice', "Nachricht vom Admin: ".$this->getConfigInstance()->readConfig('login_disabled_reason').""); } // Formular schliessen @@ -54,17 +54,17 @@ $helper->flushContent(); if ($helper->ifGuestLoginAllowed()) { // Neue Helper-Instanz holen $helper = WebFormHelper::createWebFormHelper($this, 'shipsimu_guest_login'); - $helper->addInputHiddenConfiguredField('username', 'guest_login'); - $helper->addInputHiddenConfiguredField('pass', 'guest_login'); + $helper->addInputHiddenConfiguredField('user', 'guest_login'); + $helper->addInputHiddenConfiguredField('passwd', 'guest_login'); // CAPTCHA enbaled? if ($helper->ifFormSecuredWithCaptcha()) { - $helper->addFormGroup('captcha', "Unser Gast-Login ist durch ein CAPTCHA geschützt. Bitte wiederhole den angezeigten Code, damit du dich einloggen kannst."); + $helper->addFormGroup('captcha_guest', "Unser Gast-Login ist durch ein CAPTCHA geschützt. Bitte wiederhole den angezeigten Code, damit du dich einloggen kannst."); $helper->addCaptcha(); } // END - if // Submit button - $helper->addFormGroup('buttons', "Gastlogins sind in der Funkionsweise eingeschränkt. Mehr dazu unter "Gastlogin"."); + $helper->addFormGroup('buttons_guest', "Gastlogins sind in der Funkionsweise eingeschränkt. Mehr dazu unter "Gastlogin"."); $helper->addInputSubmitButton("Als Gast einloggen"); $helper->flushContent(); } @@ -85,5 +85,5 @@ if ($helper->ifGuestLoginAllowed()) {
- Noch kein Spieleaccount? [Dann hier kostenlos anmelden!] + Noch kein Spieleaccount? [Dann hier kostenlos anmelden!]
diff --git a/application/ship-simu/templates/de/code/login_profile.ctp b/application/ship-simu/templates/de/code/login_profile.ctp index 4fa5aba..92b2dec 100644 --- a/application/ship-simu/templates/de/code/login_profile.ctp +++ b/application/ship-simu/templates/de/code/login_profile.ctp @@ -16,7 +16,7 @@ $helper->addFieldText('pass2', "Neues Passwort, Wiederholung:"); $helper->addInputPasswordField('pass2'); // Display current email -$helper->addFormNote("Derzeitige Email-Adresse: ".$helper->getField('email').""); +$helper->addFormNote('current_email', "Derzeitige Email-Adresse: ".$helper->getField('email').""); // Only for changing email address if ($helper->ifEmailChangeAllowed()) { @@ -28,11 +28,11 @@ if ($helper->ifEmailChangeAllowed()) { $helper->addInputTextField('email2'); if ($helper->ifEmailMustBeUnique()) { - $helper->addFormNote("Die von dir eingegebene Email-Adresse darf nur einmal im Spiel verwendet worden sein."); + $helper->addFormNote('email', "Die von dir eingegebene Email-Adresse darf nur einmal im Spiel verwendet worden sein."); } // END - Unique email addresses if ($helper->ifEmailChangeRequireConfirmation()) { - $helper->addFormNote("Es wird ein Bestätigungslink an deine neue Email-Adresse gesendet. Bitte klicke diesen dann möglichst schnell an."); + $helper->addFormNote('confirm_link', "Es wird ein Bestätigungslink an deine neue Email-Adresse gesendet. Bitte klicke diesen dann möglichst schnell an."); } // END - Change requires confirmation } // END - email change allowed @@ -92,7 +92,7 @@ if ($helper->ifRulesHaveChanged()) { $helper->addFieldText('rules', "Ja, ich akzeptiere die aktuellen Spielregeln:"); $helper->addInputCheckboxField('rules', false); } else { - $helper->addFormNote("Du hast die aktuellen Spielregeln akzeptiert. Vielen Dank!"); + $helper->addFormNote('rules_accepted', "Du hast die aktuellen Spielregeln akzeptiert. Vielen Dank!"); $helper->addInputHiddenField('rules', "1"); } @@ -105,7 +105,7 @@ $helper->addInputPasswordField('pass_old'); $helper->addFormGroup('buttons', "Sind alle Daten nun korrekt eingegeben? Dann sende sie mit einem Klick einfach ab!"); $helper->addInputResetButton("Alles nochmal eingeben"); $helper->addInputSubmitButton("Accountdaten aktualisieren"); -$helper->addFormNote("Deine Daten werden nach den gültigen Datenschutzgesetzten gespeichert und werden nicht an Dritte weitergegeben. Weiteres dazu siehe Link "Datenschutz"."); +$helper->addFormNote('data_protection', "Deine Daten werden nach den gültigen Datenschutzgesetzten gespeichert und werden nicht an Dritte weitergegeben. Weiteres dazu siehe Link "Datenschutz"."); $helper->flushContent(); ?>
diff --git a/application/ship-simu/templates/de/code/login_status_problem.ctp b/application/ship-simu/templates/de/code/login_status_problem.ctp index 7d3c940..200d9b8 100644 --- a/application/ship-simu/templates/de/code/login_status_problem.ctp +++ b/application/ship-simu/templates/de/code/login_status_problem.ctp @@ -14,13 +14,13 @@ if ($helper->ifUserAccountUnconfirmed()) { $helper->addInputSubmitButton("Bestätigungslink erneut aussenden"); } elseif ($helper->ifUserAccountLocked()) { // Account is locked - $helper->addFormNote("Dein Account wurde gesperrt! Grund der Sperre: + $helper->addFormNote('status_locked', "Dein Account wurde gesperrt! Grund der Sperre: ".$helper->getField('lock_reason')." Bitte melde dich beim Support, damit dieser dir weiterhelfen kann." ); } elseif ($helper->ifUserAccountGuest()) { // Account is guest account - $helper->addFormNote("Gästeaccounts sind in der Funktionalität + $helper->addFormNote('status_guest', "Gästeaccounts sind in der Funktionalität leicht eingeschränkt. Bitte melde dich an, damit du ein vollwertiges Account bekommst." ); diff --git a/application/ship-simu/templates/de/code/register_form.ctp b/application/ship-simu/templates/de/code/register_form.ctp index ceddb92..7760229 100644 --- a/application/ship-simu/templates/de/code/register_form.ctp +++ b/application/ship-simu/templates/de/code/register_form.ctp @@ -23,7 +23,7 @@ if ($helper->ifRegisterRequiresEmailVerification()) { $helper->addInputTextField('email2'); if ($helper->ifEmailMustBeUnique()) { - $helper->addFormNote("Die von dir eingegebene Email-Adresse darf nur einmal im Spiel verwendet worden sein."); + $helper->addFormNote('email', "Die von dir eingegebene Email-Adresse darf nur einmal im Spiel verwendet worden sein."); } // END - Unique email addresses } // END - email verification @@ -103,7 +103,7 @@ $helper->addFormGroup('buttons', "Wenn du alle benötigten Felder korrekt au $helper->addInputResetButton("Alles nochmal eingeben"); $helper->addInputSubmitButton("Anmeldung zum Spiel abschliessen"); -$helper->addFormNote("Deine Daten werden nach den gültigen Datenschutzgesetzten gespeichert und werden nicht an Dritte weitergegeben. Weiteres dazu siehe Link "Datenschutz"."); +$helper->addFormNote('data_protection', "Deine Daten werden nach den gültigen Datenschutzgesetzten gespeichert und werden nicht an Dritte weitergegeben. Weiteres dazu siehe Link "Datenschutz"."); $helper->flushContent(); ?>
diff --git a/inc/classes/main/helper/web/class_WebFormHelper.php b/inc/classes/main/helper/web/class_WebFormHelper.php index 1ec70bc..9a668d1 100644 --- a/inc/classes/main/helper/web/class_WebFormHelper.php +++ b/inc/classes/main/helper/web/class_WebFormHelper.php @@ -509,6 +509,12 @@ class WebFormHelper extends BaseHelper implements HelpableTemplate { $subGroupName = $this->subGroupName; } + // Same sub group to open? + if (($this->subGroupOpened === false) && ($subGroupName == $this->subGroupName)) { + // Abort here silently + return false; + } // END - if + // Initialize content with closing div by default $content = "
\n
"; @@ -586,11 +592,12 @@ class WebFormHelper extends BaseHelper implements HelpableTemplate { * Add text (notes) surrounded by a div block. Still opened groups or sub * groups will be automatically closed. * + * @param $noteId Id for this note * @param $formNotes The form notes we shell addd * @return void * @throws FormClosedException If the form is not yet opened */ - public function addFormNote ($formNotes) { + public function addFormNote ($noteId, $formNotes) { // Is the form opened? if (($this->formOpened === false) && ($this->formEnabled === true)) { // Throw an exception @@ -604,9 +611,10 @@ class WebFormHelper extends BaseHelper implements HelpableTemplate { } // END - if // Generate the content - $inputContent = sprintf("
+ $inputContent = sprintf("
%s
", + $noteId, $formNotes ); -- 2.30.2