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

Improve delete confirmation when ajax enabled a bit more

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1
    • Fix Version/s: 2.3
    • Component/s: JavaScript, Usability
    • Labels:
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_21_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE

      Description

      MDL-14790 has improved the deletion message, but we would like it improved a little bit more?

      We have tweaked our 1.9 code so that it appends the following warning to activities (not resources):

      'NOTE: This will delete all student submissions and any feedback/grades you may have given to them'

      We identify the activities by excluding elements with a class 'resource' or 'label'.

      However Moodle 2 now has different class names for each type of resource (folder, url etc). So it is harder to identify them as new ones could be added in future or with plugins.

      Could we therefore have a similar message for all deletions explaining that if there is associated student data, it will also be lost?

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              salvetore Michael de Raadt added a comment -

              That is possible. There are still mechanisms for differentiating modules. Another option could be to create a callback in modules, with a fallback, so that each module could specify its own delete message, if needs be.

              Could you share the change you made in 1.9. As a starting point it could still be useful.

              Show
              salvetore Michael de Raadt added a comment - That is possible. There are still mechanisms for differentiating modules. Another option could be to create a callback in modules, with a fallback, so that each module could specify its own delete message, if needs be. Could you share the change you made in 1.9. As a starting point it could still be useful.
              Hide
              amanda.doughty Amanda Doughty added a comment -

              I have tested this in 2.3 Dev and the message is still the same

              Show
              amanda.doughty Amanda Doughty added a comment - I have tested this in 2.3 Dev and the message is still the same

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    25/Jun/12