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

In hub directory listings, remove "Sites/Courses" counts and replace with "Courses for download: X" and "Courses for enrolling: Y"

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Blocker
    • Resolution: Fixed
    • Affects Version/s: 2.0.4, 2.1.1, 2.2
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Hub
    • Labels:
    • Testing Instructions:
      Hide

      1. Add community finder block
      2. Click on Search link
      3. Check that Mooch does display correct enrollable+downloadable numbers. (Note: if a hub hasn't been updated, the enrollable+downloadable numbers should display 0. All other hubs most likely will display 0)

      Show
      1. Add community finder block 2. Click on Search link 3. Check that Mooch does display correct enrollable+downloadable numbers. (Note: if a hub hasn't been updated, the enrollable+downloadable numbers should display 0. All other hubs most likely will display 0)
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull Master Branch:

      Description

      In hub directory listings, remove "Sites" count and replace with "Courses for download: X" and "Courses for enrolling: Y"

      Also need to be fixed for hub listing in community finder. Test that not updated 2.0 site still work even though hub directory and hub ws code get modified.

        Gliffy Diagrams

          Activity

          Show
          jerome Jérôme Mouneyrac added a comment - Changes on the hub directory: https://github.com/moodlehq/moodle-local_hubdirectory/commit/351046b43193da3f734f129b79fa7176d8ac76e3
          Hide
          jerome Jérôme Mouneyrac added a comment -

          How this is going to be tested then implemented online:
          1- Make change on hub directory => check that hub , and site are still working
          2- Make change on hub => check that sites are still working with the updated hubs
          3- Make change on the sites => check that sites are still working with not updated hubs

          Show
          jerome Jérôme Mouneyrac added a comment - How this is going to be tested then implemented online: 1- Make change on hub directory => check that hub , and site are still working 2- Make change on hub => check that sites are still working with the updated hubs 3- Make change on the sites => check that sites are still working with not updated hubs
          Hide
          jerome Jérôme Mouneyrac added a comment - - edited

          These are all the points to checks:
          Hub directory:

          • cron (get hub info)
          • manage hub => list should now also displayed enrollable courses + downloadable courses (courses and sites number are still displayed for the admin)
          • index.php should now displayed enrollable courses + downloadable courses

          Hub server:

          • register on the hub directory
          • update registration on the hub directory

          Site:

          • community search => hub listing should displayed the number of enrollable courses + downloadable courses for each hubs comming from the hub directory. (private hub information are at this moment incomplete and required more database fields)
          Show
          jerome Jérôme Mouneyrac added a comment - - edited These are all the points to checks: Hub directory: cron (get hub info) manage hub => list should now also displayed enrollable courses + downloadable courses (courses and sites number are still displayed for the admin) index.php should now displayed enrollable courses + downloadable courses Hub server: register on the hub directory update registration on the hub directory Site: community search => hub listing should displayed the number of enrollable courses + downloadable courses for each hubs comming from the hub directory. (private hub information are at this moment incomplete and required more database fields)
          Hide
          jerome Jérôme Mouneyrac added a comment -

          Note: Hub directory and hub server changes have been implemented and push into their respective repo. They work with current 2.0/2.1 STABLE and current HEAD.

          Show
          jerome Jérôme Mouneyrac added a comment - Note: Hub directory and hub server changes have been implemented and push into their respective repo. They work with current 2.0/2.1 STABLE and current HEAD.
          Hide
          dongsheng Dongsheng Cai added a comment -

          Looks good to me

          Show
          dongsheng Dongsheng Cai added a comment - Looks good to me
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          jerome Jérôme Mouneyrac added a comment -

          You can reject if you wish the time I rebase. Cheers.

          Show
          jerome Jérôme Mouneyrac added a comment - You can reject if you wish the time I rebase. Cheers.
          Hide
          nebgor Aparup Banerjee added a comment -

          Hi Jerome,
          just one minor point - the comment could use a little clarification/rephrasing.
          cheers,
          Aparup

          Show
          nebgor Aparup Banerjee added a comment - Hi Jerome, just one minor point - the comment could use a little clarification/rephrasing. cheers, Aparup
          Hide
          nebgor Aparup Banerjee added a comment -

          Hi guys,
          I've just integrated this and also added a quick commit to clarify the comment after chatting to Jerome.

          Show
          nebgor Aparup Banerjee added a comment - Hi guys, I've just integrated this and also added a quick commit to clarify the comment after chatting to Jerome.
          Hide
          jerome Jérôme Mouneyrac added a comment -

          Super, thanks Apu

          Show
          jerome Jérôme Mouneyrac added a comment - Super, thanks Apu
          Hide
          salvetore Michael de Raadt added a comment -

          Test result: Enrolable and downloadable course numbers matched those on Mooch.

          Show
          salvetore Michael de Raadt added a comment - Test result: Enrolable and downloadable course numbers matched those on Mooch.
          Hide
          stronk7 Eloy Lafuente (stronk7) added a comment -

          git & cvs repositories have been populated with this solution. Many thanks for your collaboration, yay!

          Closing, ciao

          Show
          stronk7 Eloy Lafuente (stronk7) added a comment - git & cvs repositories have been populated with this solution. Many thanks for your collaboration, yay! Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Oct/11