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

Display the activity information in activity view page only when there's something to display

    XMLWordPrintable

Details

    • MOODLE_311_STABLE
    • MOODLE_311_STABLE
    • MDL-71342-311-2
    • MDL-71342-master-2
    • Hide
      1. Create four assignments in a course with the following settings:
          Completion tracking Activity dates
        Assign 1 Either auto or manual At least one activity date set
        Assign 2 Either auto or manual No
        Assign 3 No At least one activity date set
        Assign 4 No No
      2. Visit each assignment.
      3. Confirm that assignments 1-3 show the activity information.
      4. Confirm that assignment 4 does not show the activity information and that the horizontal line separator is not being displayed.
      Show
      Create four assignments in a course with the following settings:   Completion tracking Activity dates Assign 1 Either auto or manual At least one activity date set Assign 2 Either auto or manual No Assign 3 No At least one activity date set Assign 4 No No Visit each assignment. Confirm that assignments 1-3 show the activity information. Confirm that assignment 4 does not show the activity information and that the horizontal line separator is not being displayed.

    Description

      Looking into MDL-71337, I realised that there should be no need to display the activity information output component on activities that don't display completion information and activity dates.

      On the current implementation where it's always displayed, this also causes the horizontal line separator for the activity information to be displayed which does not serve much purpose when there's no information to present. So we should conditionally display the activity information output component only when necessary.

      Attachments

        Activity

          People

            jpataleta Jun Pataleta
            jpataleta Jun Pataleta
            Dongsheng Cai Dongsheng Cai
            Jake Dallimore Jake Dallimore
            Anna Carissa Sadia Anna Carissa Sadia
            Sam Marshall, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Raquel Ortega, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              17/May/21

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 3 hours, 10 minutes
                3h 10m