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

          Attachments

            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