Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-30448

URL resource with parameters creates incorrect url syntax when URL already has a parameter - Regression

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.2
    • Fix Version/s: 2.0.7, 2.1.4
    • Component/s: Resource
    • Labels:

      Description

      To reproduce in Moodle 2.1.2+ (Build: 20111118):

      It appears a regression, adding a URL resource in the very same way as above in Moodle 2.1.2 (Build: 20111010) the URL syntax is correct: http://anotherdomain.tld/myscript.php?id=31&info=username

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            andreabix Andrea Bicciolo added a comment -

            Assigning to Eloy for triaging

            Show
            andreabix Andrea Bicciolo added a comment - Assigning to Eloy for triaging
            Hide
            skodak Petr Skoda added a comment -

            Thanks a lot for the report and sorry for the trouble!

            Show
            skodak Petr Skoda added a comment - Thanks a lot for the report and sorry for the trouble!
            Hide
            samhemelryk Sam Hemelryk added a comment -

            Thanks Petr - this has been integrated now

            Show
            samhemelryk Sam Hemelryk added a comment - Thanks Petr - this has been integrated now
            Hide
            rwijaya Rossiani Wijaya added a comment -

            This is working great.

            Thanks Petr.

            Test passed.

            Show
            rwijaya Rossiani Wijaya added a comment - This is working great. Thanks Petr. Test passed.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days.

            Thanks for the hard work! Closing, ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - The master fixes corresponding to this issue have been sent upstream. Fixes for other branches (19, 20, 21 stable) will be sent in the very-next days. Thanks for the hard work! Closing, ciao

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  9/Jan/12