Fixes for XHTML validity

diff --git a/application/ship-simu/config.php b/application/ship-simu/config.php
index 2976a8bc4ef3f579f61234686cc4ddd59db54b8e..d3d98f7cdb9ecf3d3491325c3a321bc0dcf95020 100644 (file)
--- 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 a134e303b1b7f93e567dd3c62c9de7cba4216b52..0e044f0b54e70c2ec1f9febd692c31008f99da5d 100644 (file)
--- 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 @@
 <?php
 // Get a helper instance without a form tag
 $captchaHelper = WebFormHelper::createWebFormHelper($this, 'captcha_code', false, false);
-$captchaHelper->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]
 ?>
-<div id="captcha_image">
-       <img src="$config[base_url]/index.php?app=ship-simu&amp;page=captcha_image&amp;encrypt={?encrypted_code?}&amp;response=image" alt="CAPTCHA-Bild" title="CAPTCHA-Bild" id="captcha_img" />
+<div class="captcha_image">
+       <img src="$config[base_url]/index.php?app=ship-simu&amp;page=captcha_image&amp;encrypt={?encrypted_code?}&amp;response=image" alt="CAPTCHA-Bild" title="CAPTCHA-Bild" class="captcha_img" />
 </div>
-<div id="captcha_code">
+<div class="captcha_code">
        {?captcha_code?}
 </div>

diff --git a/application/ship-simu/templates/de/code/login_form.ctp b/application/ship-simu/templates/de/code/login_form.ctp
index 4bdb82dd573116dcb34817ce1a2e0ff4bca2caa6..bf6c803127e33f916aa64bd036cbb10acf32c097 100644 (file)
--- 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: <span id=\"disabled_reason\">".$this->getConfigInstance()->readConfig('login_disabled_reason')."</span>");
+       $helper->addFormNote('form_deactivated', "Einloggen in&#39;s Spiel ist derzeit administrativ deaktiviert worden. Bitte komme sp&auml;ter noch mal wieder.");
+       $helper->addFormNote('admin_notice', "Nachricht vom Admin: <span id=\"disabled_reason\">".$this->getConfigInstance()->readConfig('login_disabled_reason')."</span>");
 }
 
 // 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&uuml;tzt. Bitte wiederhole den angezeigten Code, damit du dich einloggen kannst.");
+               $helper->addFormGroup('captcha_guest', "Unser Gast-Login ist durch ein CAPTCHA gesch&uuml;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&auml;nkt. Mehr dazu unter &quot;Gastlogin&quot;.");
+       $helper->addFormGroup('buttons_guest', "Gastlogins sind in der Funkionsweise eingeschr&auml;nkt. Mehr dazu unter &quot;Gastlogin&quot;.");
        $helper->addInputSubmitButton("Als Gast einloggen");
        $helper->flushContent();
 }
@@ -85,5 +85,5 @@ if ($helper->ifGuestLoginAllowed()) {
 </div>
 
 <div id="content_footer">
-       Noch kein Spieleaccount? [<a href="$config[base_url]/index.php?app=ship-simu&page=register">Dann hier kostenlos anmelden!</a>]
+       Noch kein Spieleaccount? [<a href="$config[base_url]/index.php?app=ship-simu&amp;page=register">Dann hier kostenlos anmelden!</a>]
 </div>

diff --git a/application/ship-simu/templates/de/code/login_profile.ctp b/application/ship-simu/templates/de/code/login_profile.ctp
index 4fa5ababee76fbc94751e275839a79f5d2b3e102..92b2dec241b6d97bedee8859e2f798a8c4b56e8d 100644 (file)
--- 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: <span id=\"displayed_field\">".$helper->getField('email')."</span>");
+$helper->addFormNote('current_email', "Derzeitige Email-Adresse: <span id=\"displayed_field\">".$helper->getField('email')."</span>");
 
 // 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&auml;tigungslink an deine neue Email-Adresse gesendet. Bitte klicke diesen dann m&ouml;glichst schnell an.");
+               $helper->addFormNote('confirm_link', "Es wird ein Best&auml;tigungslink an deine neue Email-Adresse gesendet. Bitte klicke diesen dann m&ouml;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&uuml;ltigen Datenschutzgesetzten gespeichert und werden nicht an Dritte weitergegeben. Weiteres dazu siehe Link &quot;Datenschutz&quot;.");
+$helper->addFormNote('data_protection', "Deine Daten werden nach den g&uuml;ltigen Datenschutzgesetzten gespeichert und werden nicht an Dritte weitergegeben. Weiteres dazu siehe Link &quot;Datenschutz&quot;.");
 $helper->flushContent();
 ?>
 <div id="content_header">

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 7d3c940a4c6dde8ccc4c8da79d83832153f64944..200d9b83e9d44845b9d7841a69fccc60087640de 100644 (file)
--- 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:
                <span id=\"lock_reason\">".$helper->getField('lock_reason')."</span>
                Bitte melde dich beim Support, damit dieser dir weiterhelfen kann."
        );
 } elseif ($helper->ifUserAccountGuest()) {
        // Account is guest account
-       $helper->addFormNote("G&auml;steaccounts sind in der Funktionalit&auml;t
+       $helper->addFormNote('status_guest', "G&auml;steaccounts sind in der Funktionalit&auml;t
                leicht eingeschr&auml;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 ceddb92a3a5af1d7656488eee218244b65ea3772..77602295984f5ecbcdf7f1f95829c7e9e6309ad2 100644 (file)
--- 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();
 ?>
 <div id="content_header">

diff --git a/inc/classes/main/helper/web/class_WebFormHelper.php b/inc/classes/main/helper/web/class_WebFormHelper.php
index 1ec70bca8d5e36c56b75e3030bb2153f6997a80b..9a668d123d193ac82efb4c1ba485dca028fec194 100644 (file)
--- 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 = "    </div>\n</div><!-- Sub group- CLOSE //-->";
 
@@ -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("       <div id=\"form_note\">
+               $inputContent = sprintf("       <div id=\"form_note_%s\">
                %s
        </div>",
+                       $noteId,
                        $formNotes
                );