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

Wrong use of "classes" attribute to action_link in activity_navigation class

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.4.6, 3.5.3, 3.6
    • Fix Version/s: 3.4.6, 3.5.3
    • Component/s: Course, Navigation
    • Labels:

      Description

      For the activity_navigation class.
      Into the attributes list for the action_link, the CSS class value is set into the "classes" key. However, the action_link class require it value in "class" key.

      In the method action_link->export_for_template exists the nex line:

      $data->classes = isset($attributes['class']) ? $attributes['class'] : '';

      The "class" key is checked and it is assigned to "classes" attribute.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              cirano David Herney Bernal
              Reporter:
              cirano David Herney Bernal
              Peer reviewer:
              Daniel Neis Araujo
              Integrator:
              David Monllaó
              Tester:
              Janelle Barcega
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Nov/18

                  Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 10 minutes
                  10m