Skip to content
General Revolution Evolution Add-ons International
Login | Register | MODX.com
MODX Open Source Content Management System, Framework, Platform and More.
Find a Partner | Hosts + SaaS | Jobs | Donate
  • RegisterSign Up with the MODX Community
  • LoginUse Your MODX.com Account
  • MODX Community Forums
  • Evolution
  • Legacy Release Archive
  • 0.9.6.1
  •  
  • Another weblogin error#

  • 13825
    55

    jdelight Reply #1, 4 years, 3 months ago

    Reply
    • Link to this post#1
    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

    eerne Reply #2, 2 years, 4 months ago

    Reply
    • Link to this post#2
    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


  • 25663
    12,071


    Ryan Thrash

    MODX Co-Founder & Leader of Awesomeness




    MODX Revolution

    Your Content, Your Way.


    Issues | Documentation | Git the Source


    Need help? Help us help you.


    rthrash Reply #3, 2 years, 4 months ago

    Reply
    • Link to this post#3
    Please file a bug report referencing this post please so it will be included in the next release. Thanks!


  • 8522
    145

    eerne Reply #4, 2 years, 4 months ago

    Reply
    • Link to this post#4
    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.


  • 25663
    12,071


    Ryan Thrash

    MODX Co-Founder & Leader of Awesomeness




    MODX Revolution

    Your Content, Your Way.


    Issues | Documentation | Git the Source


    Need help? Help us help you.


    rthrash Reply #5, 2 years, 4 months ago

    Reply
    • Link to this post#5
    Thanks. Moved to MODx project since it's part of the core Evo distribution.





Actions

Login to Post

Other Support Options

To file a bug or make a feature request visit our issue tracker, or you can also purchase commercial support.

Love MODX?

If you build sites for a living with MODX or just love using it, why not give back?

Information

Posted in this thread:
eerne, jdelight, rethrash

 
Back to Top

MODX Global HQ

1333 N Stemmons Fwy, Ste 110
Dallas, TX 75207
United States

+1 (469) 777-MODX (6639)

The MODX Company

  • Contact
  • Media Center
  • Careers at MODX
  • Wall of Fame
  • The MODX Blog

Sponsors

SoftLayer Firehost: Secure Cloud Hosting

Stay Connected

Read our previous email newsletters.

Twitter Facebook Google+ LinkedIn github Feeds

Privacy Policy | Terms of Service | Pixels by AKTA Web Studio© 2005-2012 MODX. All rights reserved. Trademark Policy