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

core_contentbank\external\rename_content is not using validated parameters

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 3.9.4, 3.10.1, 3.11, 4.0
    • 3.9.5, 3.10.2
    • Content bank
    • MOODLE_310_STABLE, MOODLE_311_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE
    • MOODLE_310_STABLE, MOODLE_39_STABLE
    • MDL-70910-311
    • MDL-70910-master
    • Easy
    • Hide
      1. Go to Site Administration/Server/Tasks/Scheduled tasks
      2. Find the task "Download available H5P content types from h5p.org" and click on "Run now"
      3. Click "Run now" on the Confirm message
      4. When Process is done click on the "Content bank" icon on the left navigation drawer.
      5. Click on "Add" action button and select for example "Summary"
      6. Set the Title as "Cool bananas", and click Save.
      7. Click on the cog icon and select "Rename"
      8. Set Content name to "Frozen oranges" and click "Rename".
      9. You should see the message "The content has been renamed." and see that the name has been changed to "Frozen oranges".
      Show
      Go to Site Administration/Server/Tasks/Scheduled tasks Find the task "Download available H5P content types from h5p.org" and click on "Run now" Click "Run now" on the Confirm message When Process is done click on the "Content bank" icon on the left navigation drawer. Click on "Add" action button and select for example "Summary" Set the Title as "Cool bananas", and click Save. Click on the cog icon and select "Rename" Set Content name to "Frozen oranges" and click "Rename". You should see the message "The content has been renamed." and see that the name has been changed to "Frozen oranges".

    Description

      core_contentbank\external\rename_content execute method is using $contentid instead $params['contentid'] in some cases. For instance, here.

      Attachments

        Issue Links

          Activity

            People

              davidmatamoros David Matamoros
              sarjona Sara Arjona (@sarjona)
              Paul Holden Paul Holden
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Anna Carissa Sadia Anna Carissa Sadia
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                8/Mar/21

                Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 hour, 5 minutes
                  1h 5m