We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 28042 ☆ A M B ☆
    • 24,524 Posts
    I also notice that when the form is returned with that error message, the password verify field and the following phone number field are not filled in; every other field is filled in. That might be a clue... or not rolleyes
      Studying MODX in the desert - http://sottwell.com
      Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
      Join the Slack Community - http://modx.org
      • 28042 ☆ A M B ☆
      • 24,524 Posts
      I think I see something odd... in the source for the form
      <label for="wlpeUserRegisterPasswordConfirm"><span class="required">*</span> Password (confirm)
      				<input id="wlpeUserRegisterPasswordConfirm" type="password" name="password.confirm" value="" />
      				</label>
      
      

      while in the code for processing it,
      			$confirm = $field.'_confirm';
      


      It appears the form doesn’t work at all; a bad email just gets the same error, as does leaving fields empty or a short password.
        Studying MODX in the desert - http://sottwell.com
        Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
        Join the Slack Community - http://modx.org
        • 28042 ☆ A M B ☆
        • 24,524 Posts
        It’s Beta 4. I didn’t notice till I uploaded the files, and I didn’t see the issue with reuploading it for something so trivial.
        Not a good habit to have when coding; it’s the trivial details that will bit you in the tush every time.
          Studying MODX in the desert - http://sottwell.com
          Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
          Join the Slack Community - http://modx.org
          • 28042 ☆ A M B ☆
          • 24,524 Posts
          Got it!

          weblogin.class.php, line 436 is in the wrong place, it should be line at 445, after the email check. You’ll see the mismatched braces
          		if ($_POST['username'] == '' || empty($_POST['username']) || trim($_POST['username']) == '' ) // pixelchutes
          		{			
          			return $this->FormatMessage($this->LanguageArray[0]);
          		}
          		if ($regRequired !== '')
          		if ( strlen($_POST['email']) > 0 ) // pixelchutes
          		{
          			// Validate the email address.
          			$this->ValidateEmail($_POST['email']);
          			if (!empty($this->Report))
          			{
          				return $this->report;
          			}
          		}
          		{
          			$requiredFields = explode(',', str_replace(' ,', ',', $regRequired));
          


          That should be:

          		if ($_POST['username'] == '' || empty($_POST['username']) || trim($_POST['username']) == '' ) // pixelchutes
          		{			
          			return $this->FormatMessage($this->LanguageArray[0]);
          		}
          		if ( strlen($_POST['email']) > 0 ) // pixelchutes
          		{
          			// Validate the email address.
          			$this->ValidateEmail($_POST['email']);
          			if (!empty($this->Report))
          			{
          				return $this->report;
          			}
          		}
          		if ($regRequired !== '')
          		{
          			$requiredFields = explode(',', str_replace(' ,', ',', $regRequired));
          

          I can’t imagine why that wasn’t flagged as a big PHP error!
            Studying MODX in the desert - http://sottwell.com
            Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
            Join the Slack Community - http://modx.org
            • 28033
            • 925 Posts
            I’ll get a Beta 5 released tommorow morning, and properly name it. tongue

            Thanks for finding that issue, btw. wink
              My Snippets
              -> PopUpChunk v1.0
              • 28042 ☆ A M B ☆
              • 24,524 Posts
              I always was a nit-picker; but I always knew where my kid’s socks were wink
                Studying MODX in the desert - http://sottwell.com
                Tips and Tricks from the MODX Forums and Slack Channels - http://modxcookbook.com
                Join the Slack Community - http://modx.org
                • 28033
                • 925 Posts
                Quote from: sottwell at Jul 28, 2008, 10:39 PM

                I always was a nit-picker; but I always knew where my kid’s socks were wink

                :P

                BTW, BETA 5 is now live.
                  My Snippets
                  -> PopUpChunk v1.0
                  • 31471
                  • 206 Posts
                  Maybe it’s not important (I think it can be) but on line 137: [tt]var $DateFormat;[/tt] should’ve been written with small ’d’ like this: [tt]var $dateFormat;[/tt]
                    • 31471
                    • 206 Posts
                    I think I got a bit forward with the "non-latin utf8 characters in username" issue.
                    In the Core Code section I asked the experts if my workaround is acceptable:
                    http://modxcms.com/forums/index.php/topic,27813.0.html

                    So I ask you Shosite, listen to that topic, If that solves it for the next release. Thanks! (I will post here anyway.)
                      • 28033
                      • 925 Posts
                      Quote from: vhollo at Aug 04, 2008, 07:15 AM

                      I think I got a bit forward with the "non-latin utf8 characters in username" issue.
                      In the Core Code section I asked the experts if my workaround is acceptable:
                      http://modxcms.com/forums/index.php/topic,27813.0.html

                      So I ask you Shosite, listen to that topic, If that solves it for the next release. Thanks! (I will post here anyway.)

                      I’ll keep track of it. wink
                        My Snippets
                        -> PopUpChunk v1.0