Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.5
    • Fix Version/s: 2.5
    • Component/s: Blog
    • Labels:
    • Rank:
      49190

      Description

      Pasted from register an external blog form:

      Did you remember to call setType() for 'url'? Defaulting to PARAM_RAW cleaning.
      
          line 1289 of /lib/formslib.php: call to debugging()
          line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
          line 202 of /lib/formslib.php: call to moodleform->_process_submission()
          line 66 of /blog/external_blog_edit.php: call to moodleform->moodleform()
      
      Did you remember to call setType() for 'name'? Defaulting to PARAM_RAW cleaning.
      
          line 1289 of /lib/formslib.php: call to debugging()
          line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
          line 202 of /lib/formslib.php: call to moodleform->_process_submission()
          line 66 of /blog/external_blog_edit.php: call to moodleform->moodleform()
      
      Did you remember to call setType() for 'filtertags'? Defaulting to PARAM_RAW cleaning.
      
          line 1289 of /lib/formslib.php: call to debugging()
          line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
          line 202 of /lib/formslib.php: call to moodleform->_process_submission()
          line 66 of /blog/external_blog_edit.php: call to moodleform->moodleform()
      
      Did you remember to call setType() for 'autotags'? Defaulting to PARAM_RAW cleaning.
      
          line 1289 of /lib/formslib.php: call to debugging()
          line 281 of /lib/formslib.php: call to moodleform->detectMissingSetType()
          line 202 of /lib/formslib.php: call to moodleform->_process_submission()
          line 66 of /blog/external_blog_edit.php: call to moodleform->moodleform()
      

        Activity

        Hide
        Dan Poltawski added a comment -

        Hi Ankit,

        PARAM_TEXT does not look appropiate for filtertags/autotags - PARAM_TEXT implies multilang support.

        PARAM_TAGLIST seems to fit this.

        Show
        Dan Poltawski added a comment - Hi Ankit, PARAM_TEXT does not look appropiate for filtertags/autotags - PARAM_TEXT implies multilang support. PARAM_TAGLIST seems to fit this.
        Hide
        Ankit Agarwal added a comment -

        Hi Dan,
        thanks for the review. I have made those changes.

        Pushing for integration.
        Thanks

        Show
        Ankit Agarwal added a comment - Hi Dan, thanks for the review. I have made those changes. Pushing for integration. Thanks
        Hide
        Eloy Lafuente (stronk7) added a comment -

        Integrated, thanks!

        Show
        Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
        Hide
        Jason Fowler added a comment -

        Thanks Ankit

        Show
        Jason Fowler added a comment - Thanks Ankit
        Hide
        Dan Poltawski added a comment -

        Blooming Marvelous! It's time for a knees up - your changes are upstream!

        Thanks for making Moodle better!

        Toodle pip

        Show
        Dan Poltawski added a comment - Blooming Marvelous! It's time for a knees up - your changes are upstream! Thanks for making Moodle better! Toodle pip

          People

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

            Dates

            • Created:
              Updated:
              Resolved: