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

More flexibility with pagetype param handling

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.8.2
    • Fix Version/s: 2.7.6, 2.8.4
    • Component/s: Blocks
    • Labels:
    • Testing Instructions:
      1. Log in as an admin
      2. Turn on editing
      3. On several pages (including my moodle and course view page) drag and drop several blocks moving them around to make sure nothing is broken.
    • Affected Branches:
      MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_27_STABLE, MOODLE_28_STABLE
    • Pull Master Branch:
      48968-29
    • Pull Master Diff URL:

      Description

      We hit an issue with pagetype handling for block editing while working on a solution to implement multiple my moodle style pages.
      We want to use the ID as part of the pagetype variable in order to distinguish each of the 1..n customisable pages.
      However the pagetype param is using PARAM_ALPHAEXT.
      I can't see a good reason for it using that instead of PARAM_ALPHANUMEXT and I think the change is potentially one that will come up for Moodle in the near future anyway.
      As such I purpose we make it now.

        Attachments

          Activity

            People

            Assignee:
            samhemelryk Sam Hemelryk
            Reporter:
            samhemelryk Sam Hemelryk
            Peer reviewer:
            Ankit Agarwal
            Integrator:
            Andrew Nicols
            Tester:
            Adrian Greeve
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              9/Mar/15