We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 13825
    • 55 Posts
    It seems the url being generated needs to not contain encoded & signs.

    It produces this: http://website.com/login?webloginmode=actp&wli=17&wlk=earqqAcT

    and I get a javascript alert error (Error while loading user account. Please contact the Site Administrator)

    But if I manuall change the url to: http://website.com/login?webloginmode=actp&wli=17&wlk=earqqAcT

    It works fine.

    How can I fix this permanently please?



      • 8522
      • 145 Posts
      This is still a bug in MODx Evolution 1.0.2, happens with XHTML URLs turned on.

      Attached is a fixed weblogin.processor.inc.php


      // built activation url
      // fix (2010-01-02) for "Another weblogin error"
      // http://modxcms.com/forums/index.php/topic,22250.0.html
      // to prevent & in forget password confirmation e-mail
      $orgxhtmlconfig = $modx->config[’xhtml_urls’];
      $modx->config[’xhtml_urls’] = false; // turn off xhtml urls
      if($_SERVER[’SERVER_PORT’]!=’80’) {
      $url = $modx->config[’server_protocol’].’://’.$_SERVER[’SERVER_NAME’].’:’.$_SERVER[’SERVER_PORT’].$modx->makeURL($modx->documentIdentifier,’’,"webloginmode=actp&wli=".$row[’id’]."&wlk=".$newpwdkey);
      } else {
      $url = $modx->config[’server_protocol’].’://’.$_SERVER[’SERVER_NAME’].$modx->makeURL($modx->documentIdentifier,’’,"webloginmode=actp&wli=".$row[’id’]."&wlk=".$newpwdkey);
      }
      $modx->config[’xhtml_urls’] = $orgxhtmlconfig; // reset to original value
      • Please file a bug report referencing this post please so it will be included in the next release. Thanks!
          Ryan Thrash, MODX Co-Founder
          Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me
          • 8522
          • 145 Posts
          here it is:
          http://svn.modxcms.com/jira/browse/ADDON-98

          sorry i forgot to make a nice diff. I just added the file to replace.
          • Thanks. Moved to MODx project since it’s part of the core Evo distribution.
              Ryan Thrash, MODX Co-Founder
              Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me