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

shared_data_provider should have shared_userlist

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Minor Minor
    • 3.4.6, 3.5.3
    • 3.4.5, 3.5.2, 3.6
    • Privacy
    • MOODLE_34_STABLE, MOODLE_35_STABLE, MOODLE_36_STABLE
    • MOODLE_34_STABLE, MOODLE_35_STABLE
    • MDL-63924-master
    • Hide
      1. Login as admin
      2. Navigate to Site administration->Users -> Privacy & Policies -> Plugin registry
      3. Choose "Expand all"
      4. Search for "Userlist provider missing"
        1. Confirm that you don't see any results
          NOTE: There will be one result for the core_favourites,core_backup and block_recentlyaccesseditems subsystems for MASTER only because it has not yet been written. This is expected and does not constitute a fail
      5. Apply the attached patch to the branch: 

        git am 0001-MDL-63924-privacy-Add-test-to-ensure-shared_data_pro.patch

      6. Initialise phpunit 

        php admin/tool/phpunit/cli/init.php

      7. Run phpunit 

        ./vendor/bin/phpunit privacy/tests/provider_test.php

        1. Confirm passes
          NOTE: There is a known failure for the favourites subsystem for MASTER only because it has not yet been written. This is expected
      Show
      Login as admin Navigate to Site administration->Users -> Privacy & Policies -> Plugin registry Choose "Expand all" Search for " Userlist provider missing " Confirm that you don't see any results NOTE: There will be one result for the core_favourites,core_backup and block_recentlyaccesseditems subsystems for MASTER only because it has not yet been written. This is expected and does not constitute a fail Apply the attached patch to the branch: git am 0001-MDL-63924-privacy-Add-test-to-ensure-shared_data_pro.patch Initialise phpunit php admin/tool/phpunit/cli/init.php Run phpunit ./vendor/bin/phpunit privacy/tests/provider_test.php Confirm passes NOTE: There is a known failure for the favourites subsystem for MASTER only because it has not yet been written. This is expected
    • GDPR Followup Sprint 1

      I realised today that we are missing this.

      It's far too late to add this to stables, and many of the areas have actually been done already (they're just missing the interface to say so).

      We have a core_user_data_provider. Where this is implemented we are requiring that a core_userlist_provider also be implemented.

      Equally we have a shared_data_provider which is used by subsystems, and subplugins which do not store data but provide data to others. This needs to have a shared_userlist_provider too.

      I'd propose that we just add the empty interfaces now so that they can be used consistently in the future. Later on we can add the tests for them.
      This is largely an accounting thing and will only be used in unit tests, and the plugin registry.

        1. 0001-MDL-63924-privacy-Add-test-to-ensure-shared_data_pro.patch
          2 kB
        2. MDL-63924.PNG
          MDL-63924.PNG
          166 kB

            dobedobedoh Andrew Lyons
            dobedobedoh Andrew Lyons
            Mihail Geshoski Mihail Geshoski
            Jun Pataleta Jun Pataleta
            Anna Carissa Sadia Anna Carissa Sadia
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved:

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 1 hour
                1h

                  Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.