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:

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

        Gliffy Diagrams

          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: