Development Branch: Loading Unclean URL

[nyanginator]

Hi, I wasn't sure where to post to reference the Development Branch, so sorry if I am posting in the wrong place.

EDIT: Added additional test case of /moodle/course/index.php

In the Master Branch, I could go to an unclean URL and have them resolve as follows:

• http://192.168.1.10/moodle/course/view.php?id=3 →
  http://192.168.1.10/moodle/course/My+Course+Shortname
  
  
• http://192.168.1.10/moodle/course/index.php →
  http://192.168.1.10/moodle/course/
  
  
• http://192.168.1.10/moodle/mod/page/?id=3 →
  http://192.168.1.10/moodle/course/My+Course+Shortname/page
  
  
• http://192.168.1.10/moodle/mod/page/index.php?id=3 →
  http://192.168.1.10/moodle/course/My+Course+Shortname/page
  
  

However, in the Development Branch, I get:

• http://192.168.1.10/moodle/course/view.php?id=3 →
  http://192.168.1.10/moodle/course/view.php?id=3 (page loads, but URL is still unclean)
  
  
• http://192.168.1.10/moodle/course/index.php →
  http://192.168.1.10/moodle/moodle/course/ (page loads and index.php is gone, but URL is still unclean and has the Moodle directory appearing twice)
  
  
• http://192.168.1.10/moodle/mod/page/?id=3 →
  http://192.168.1.10/moodle/mod/page/?id=3 (page loads, but URL is still unclean)
  
  
• http://192.168.1.10/moodle/mod/page/index.php?id=3 →
  http://192.168.1.10//moodle/moodle/mod/page/?id=3 (page loads, but URL is still unclean and has the Moodle directory appearing twice)
  
  

Actually, I didn't even know about the last 2 test cases until I looked at the cleaner code. Where does this link (to display all mods of a certain course) usually appear?

I know this is an experimental side project, so any help or insight is very much appreciated.

Thanks again!

[nyanginator]

Narrowed it down to classes/url_rewriter.php. In the function html_head_setup_info(), when a new moodle_url is created with $ME, the Moodle directory (in my case, "'/moodle") is prepended to the path automatically. Doing a check for the Moodle directory at the start of the string, and removing it if present, seems to fix the issues I mentioned above.

private static function html_head_setup_info() {
    global $CFG, $ME, $PAGE;

    // Remove Moodle directory at beginning of URL, if present
    $wwwroot = new moodle_url($CFG->wwwroot);
    $wwwpath = $wwwroot->get_path();
    if (substr($ME, 0, strlen($wwwpath)) === $wwwpath) {
        $ME = substr($ME, strlen($wwwpath));
    }

    // Continue with the rest of the code as usual ...

    $me = new moodle_url($ME);
     ...
}

[brendanheywood]

thanks @nyanginator

I'm on long term leave from Catalyst so can't really spend much time on this. If you can put together a pull request including unit tests for this then someone will probably merge it in for you.

[nyanginator]

Thanks for the reply. I haven't done unit tests before, so it may take me awhile to read up on it and set everything up. From my initial tries, it looks like the Moodle setup for PHPUnit doesn't play nice with XAMPP, or at least how XAMPP is on my system. I'll be working on reinstalling my LAMP stack using tasksel (Ubuntu-based distro).

Do you or anybody else happen to have a link handy to a previous pull request with accompanying unit tests that I can use as a reference?

[brendanheywood]

In theory this should be everything you need:

https://docs.moodle.org/dev/PHPUnit

[nyanginator]

Thanks, I think I got it. Pull request #129. Included additional test cases:

1. Course view URL
   (e.g. http://www.example.com/moodle/course/view.php?id=5)
2. Course users URL
   (e.g. http://www.example.com/moodle/user/?id=5)
3. Course users URL with "index.php"
   (e.g. http://www.example.com/moodle/user/index.php?id=5)
4. Course index URL
   (e.g. http://www.example.com/moodle/course/?categoryid=2)
5. Course index URL with "index.php"
   (e.g. http://www.example.com/moodle/course/index.php?categoryid=2)
6. Module URL
   (e.g. http://www.example.com/moodle/mod/page/?id=5)
7. Module URL with "index.php"
   (e.g. http://www.example.com/moodle/mod/page/index.php?id=5)
8. Module view URL
   (e.g. http://www.example.com/moodle/mod/page/view.php?id=9)
9. User profile URL
   (e.g. http://www.example.com/moodle/user/profile.php?id=11)