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

Web services uses non-existant CSS class "success" instead of "notifysuccess"

    XMLWordPrintable

Details

    • MOODLE_25_STABLE
    • MOODLE_25_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE
    • Easy
    • Hide
      • In webservices change some service user settings(My home ► Site administration ► Plugins ► Web services ► External services
      • Add a new custom service and save changes
      • Click on 'Authorised users'
      • Add some users
      • Underneath the user selection boxes, choose a user
      • Press ave on the resulting page
      • The confirmation should be shown correctly with the 'notifysuccess' style.
      Show
      In webservices change some service user settings(My home ► Site administration ► Plugins ► Web services ► External services Add a new custom service and save changes Click on 'Authorised users' Add some users Underneath the user selection boxes, choose a user Press ave on the resulting page The confirmation should be shown correctly with the 'notifysuccess' style.

    Description

      admin/webservice/service_user_settings.php, line 71

      $notification = $OUTPUT->notification(get_string('usersettingssaved', 'webservice'), 'success');
      

      In the above line, I assume the class 'success' was intended to be 'notifysuccess' as used in all similar notifications. There's no CSS that matches 'sucess' so it's not actually doing anything.

      Attachments

        Issue Links

          Activity

            People

              apsdehal Amanpreet Singh
              bawjaws David Scotson
              Ankit Agarwal Ankit Agarwal
              Marina Glancy Marina Glancy
              Andrew Lyons Andrew Lyons
              Amy Groshek, David Scotson, David Woloszyn, Huong Nguyen, Jake Dallimore, Michael Hawkins, Stevani Andolo, Juan Leyva, David Woloszyn, Huong Nguyen, Jake Dallimore, Michael Hawkins, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                10/Mar/14