Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-550

mgnlRemoveParameter does not work if first parameter is replaced

    XMLWordPrintable

Details

    • Bug
    • Resolution: Obsolete
    • Minor
    • None
    • 2.1 Final
    • admininterface
    • None
    • all

    Description

      If the first request parameter has to be replaced the funciton returned appended an ampersand instead of a question mark to the URL.

      patch for revision 1437 of tagged version /magnolia2.1:

      diff -uBbPr originals/magnolia-2.1/src/webapp/admindocroot/js/generic.js patchedForSendingPatches/magnolia-2.1/src/webapp/admindocroot/js/generic.js
      — originals/magnolia-2.1/src/webapp/admindocroot/js/generic.js 2005-08-31 11:30:51.000000000 +0200
      +++ patchedForSendingPatches/magnolia-2.1/src/webapp/admindocroot/js/generic.js 2005-09-17 16:38:42.000000000 +0200
      @@ -193,12 +193,17 @@
      if (tmp[1])
      {
      var paramObj=tmp[1].split("&");
      + var isQuestionMarkAppended = false;

      for (var i=0;i<paramObj.length;i++)
      {
      if (paramObj[i].indexOf(name+"=")!=0)
      {

      • if (i==0) newHref+="?";
        + if (!isQuestionMarkAppended)
        + { + newHref+="?"; + isQuestionMarkAppended = true; + }

        else newHref+="&";
        newHref+=paramObj[i];
        }

      Checklists

        Acceptance criteria

        Attachments

          Activity

            People

              had Jan Haderka
              aemi Michael Aemisegger
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Checklists

                  Bug DoR
                  Task DoD