Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: General
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Rank:
      33974

      Description

      PARAM_ALPHAEXT - no "/" anymore
      PARAM_SAFEPATH - replaces PARAM_ALPHAEXT when "/" needed
      PARAM_ALPHANUMEXT

      PARAM_TAG - block "<>"and control chars and allow "\"

      PARAM_PLAINTEXT like PARAM_TEXT but without multilang support (wiki page names, etc.)

        Issue Links

          Activity

          Hide
          Petr Škoda added a comment -

          not sure about PARAM_PLAINTEXT anymore - PARAM_NOTAGS seems like a good candidate instead.

          Anyway there is a lot to improve in this area

          Show
          Petr Škoda added a comment - not sure about PARAM_PLAINTEXT anymore - PARAM_NOTAGS seems like a good candidate instead. Anyway there is a lot to improve in this area
          Hide
          Petr Škoda added a comment -

          also adding is_number() for integers including string form; aliasing PRAM_NUMBER to PARAM_FLOAT and deprecating

          Show
          Petr Škoda added a comment - also adding is_number() for integers including string form; aliasing PRAM_NUMBER to PARAM_FLOAT and deprecating
          Hide
          Martin Dougiamas added a comment -

          I tidied up PARAM_XXXX defines to use strings instead of octal numbers, sort alphabetically and segregate the aliases and deprecated definitions.

          Also added new definitions for PARAM_AUTH, PARAM_LANG and PARAM_THEME

          Show
          Martin Dougiamas added a comment - I tidied up PARAM_XXXX defines to use strings instead of octal numbers, sort alphabetically and segregate the aliases and deprecated definitions. Also added new definitions for PARAM_AUTH, PARAM_LANG and PARAM_THEME
          Hide
          Ashley Holman added a comment -

          I had to update the admin_setting_configtext validation check because it was assuming these constants were integers and treating all strings are regex patterns. The check now requires the regex patterns to be surrounded by slashes, otherwise will treat it as a PARAM constant.

          Show
          Ashley Holman added a comment - I had to update the admin_setting_configtext validation check because it was assuming these constants were integers and treating all strings are regex patterns. The check now requires the regex patterns to be surrounded by slashes, otherwise will treat it as a PARAM constant.
          Hide
          Petr Škoda added a comment -

          thanks for the fix, closing as fixed

          Petr Skoda

          Show
          Petr Škoda added a comment - thanks for the fix, closing as fixed Petr Skoda

            People

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

              Dates

              • Created:
                Updated:
                Resolved: