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

      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.)

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Petr Skoda 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 Skoda 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 Skoda added a comment -

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

            Show
            Petr Skoda 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 Skoda added a comment -

            thanks for the fix, closing as fixed

            Petr Skoda

            Show
            Petr Skoda 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: