Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-2206

Allow long protocols in $mmUtil#isAbsoluteURL

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.3.1
    • Fix Version/s: 3.3.2
    • Component/s: Custom URL Scheme
    • Labels:
    • Testing Instructions:
      Hide
      1. In Moodle web, add a link to a course in the same site (you can use a relative link) using the scheme "moodlemobile://". E.g.: moodlemobile://link=../enrol/index.php?id=2 . This one is already added at the end of Proto 3.2 > MoodleMobile Testing > Links to capture.
      2. In the app, click the link and check that the course is opened.
      3. Check that the SSO works as expected connecting using Android and iOs to the prototype 26 and prototype 27 sites.
      Show
      In Moodle web, add a link to a course in the same site  (you can use a relative link) using the scheme "moodlemobile://". E.g.:  moodlemobile://link=../enrol/index.php?id=2 . This one is already added at the end of Proto 3.2 > MoodleMobile Testing > Links to capture. In the app, click the link and check that the course is opened. Check that the SSO works as expected connecting using Android and iOs to the prototype 26 and prototype 27 sites.
    • Affected Branches:
      MOODLE_33_STABLE
    • Fixed Branches:
      MOODLE_33_STABLE
    • Sprint:
      Moodle Mobile 3.3.2

      Description

      The RegExp of $mmUtil#isAbsoluteURL only allows protocols of 10 characters of less. We should allow longer protocols, otherwise the app won't be able to launch most custom URL schemes (like moodlemobile://).

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            dpalou Dani Palou
            Peer reviewer:
            Pau Ferrer
            Integrator:
            Juan Leyva
            Tester:
            Berta Maroto
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              29/Sep/17