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

Database: create ##userpicture## tag to allow the user's profile picture to be displayed

    Details

    • Testing Instructions:
      Hide
      1. Create a database activity.
      2. A a field or two to the activity.
      3. Go to the templates.
      4. In the list template add
      5. Check that the user picture is now available in the "Available tags"
      6. Add the user picture tag to the template.
      7. Add an entry.
      8. Go to the view list and check that the user picture is displayed.
      9. Do a search using the advanced search at the bottom (or the search tab at the top).
      10. No errors are expected to be found.
      11. Delete one entry and make sure no warning/error happens
      12. Create more entries and select all
      13. Click Delete selected
      14. Make sure no error/warning is visible
      15. Click Delete and make sure all selected entries got deleted.
      Show
      Create a database activity. A a field or two to the activity. Go to the templates. In the list template add Check that the user picture is now available in the "Available tags" Add the user picture tag to the template. Add an entry. Go to the view list and check that the user picture is displayed. Do a search using the advanced search at the bottom (or the search tab at the top). No errors are expected to be found. Delete one entry and make sure no warning/error happens Create more entries and select all Click Delete selected Make sure no error/warning is visible Click Delete and make sure all selected entries got deleted.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_19_STABLE, MOODLE_20_STABLE, MOODLE_24_STABLE, MOODLE_29_STABLE
    • Fixed Branches:
      MOODLE_29_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-20053-master

      Description

      Similar to forum, it would be good to display a user's profile picture within the database activity module. We currently have a ##user## tag so ##userpicture## seems obvious. I am attaching a patch which implements this feature. Feel free to test the patch and provide feedback. Peace - Anthony

        Gliffy Diagrams

        1. data_userpicture_large.patch
          3 kB
          Anthony Borrow
        2. data_userpicture.patch
          2 kB
          Anthony Borrow
        3. MDL-20053.patch
          2 kB
          Anthony Borrow
        4. MDL-20053.patch
          2 kB
          Anthony Borrow

          Issue Links

            Activity

            Hide
            wenxin Wenxin Lu added a comment -

            Great! Thanks a lot, Anthony. It works perfect on my Moodle1.9.7. I have been waiting for this photo display for years. Also, the picture links to full profile as well! Just one suggestion: Currently, it shows f2.jpg, the smaller one. Can we have the choice to show f1.jpg, the bigger one?

            Show
            wenxin Wenxin Lu added a comment - Great! Thanks a lot, Anthony. It works perfect on my Moodle1.9.7. I have been waiting for this photo display for years. Also, the picture links to full profile as well! Just one suggestion: Currently, it shows f2.jpg, the smaller one. Can we have the choice to show f1.jpg, the bigger one?
            Hide
            aborrow Anthony Borrow added a comment -

            You can try this patch which introduces ##userpicturelarge## which seemed to be the easiest way to implement. Keep in mind that at first this did not appear to be working because my theme (standard) was forcing the .userpicture to display at 35px despite the right file being selected. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - You can try this patch which introduces ##userpicturelarge## which seemed to be the easiest way to implement. Keep in mind that at first this did not appear to be working because my theme (standard) was forcing the .userpicture to display at 35px despite the right file being selected. Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            Hartmut is requesting a patch for 2.0 (http://moodle.org/mod/forum/discuss.php?d=127757#p756982). Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Hartmut is requesting a patch for 2.0 ( http://moodle.org/mod/forum/discuss.php?d=127757#p756982 ). Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            Hartmut - Give https://github.com/arborrow/moodle/commit/afa1abd1ccb18bc86a78bab5992cf91cf5b258ff a try and see if that helps. I've not tested it but the print_user_picture function is deprecated in 2.0 so the basic approach will be something along these lines. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Hartmut - Give https://github.com/arborrow/moodle/commit/afa1abd1ccb18bc86a78bab5992cf91cf5b258ff a try and see if that helps. I've not tested it but the print_user_picture function is deprecated in 2.0 so the basic approach will be something along these lines. Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            Hartmut - Actually I got to work and tested what I did when I was half awake this morning and of course it did not work so save yourself the trouble. Here is a patch file and the branch can be checked out at https://github.com/arborrow/moodle/tree/MDL-20053. Let me know if you have any questions. There may have been a better way of avoiding getting the whole user record; however, user_picture was throwing an error when I tried to be more minimalistic so I figured I would just use what worked. Peace - Anthony

            Show
            aborrow Anthony Borrow added a comment - Hartmut - Actually I got to work and tested what I did when I was half awake this morning and of course it did not work so save yourself the trouble. Here is a patch file and the branch can be checked out at https://github.com/arborrow/moodle/tree/MDL-20053 . Let me know if you have any questions. There may have been a better way of avoiding getting the whole user record; however, user_picture was throwing an error when I tried to be more minimalistic so I figured I would just use what worked. Peace - Anthony
            Hide
            aborrow Anthony Borrow added a comment -

            updated patch file with courseid parameter

            Show
            aborrow Anthony Borrow added a comment - updated patch file with courseid parameter
            Hide
            tsala Helen Foster added a comment -

            Adrian, I'm just adding you, as database activity component lead, as a watcher, hoping you can take a look at this issue.

            Show
            tsala Helen Foster added a comment - Adrian, I'm just adding you, as database activity component lead, as a watcher, hoping you can take a look at this issue.
            Show
            marina Marina Glancy added a comment - Sending patch from Anthony Borrow for peer review: https://github.com/arborrow/moodle/compare/408612d1118581b36e0de8686d279c7ce740c1be...MDL-20053
            Hide
            cibot CiBoT added a comment -

            Fails against automated checks.

            Error: the repository field is empty. Nothing was checked.

            • Testing instructions are missing.

            More information about this report

            Show
            cibot CiBoT added a comment - Fails against automated checks. Error: the repository field is empty. Nothing was checked. Testing instructions are missing. More information about this report
            Hide
            abgreeve Adrian Greeve added a comment -

            I'm not a fan of the db call for each record to retrieve the user picture information, especially if it isn't needed. This should be done in the main db call that is sent to the data_print_template() function.

            Show
            abgreeve Adrian Greeve added a comment - I'm not a fan of the db call for each record to retrieve the user picture information, especially if it isn't needed. This should be done in the main db call that is sent to the data_print_template() function.
            Hide
            abgreeve Adrian Greeve added a comment -

            I've updated the patch to add user picture information to the general sql queries. It is only three extra columns and should have very little impact.
            Requesting a peer review.

            Show
            abgreeve Adrian Greeve added a comment - I've updated the patch to add user picture information to the general sql queries. It is only three extra columns and should have very little impact. Requesting a peer review.
            Hide
            cibot CiBoT added a comment -

            Code verified against automated checks.

            Checked MDL-20053 using repository: git://github.com/abgreeve/moodle.git

            More information about this report

            Show
            cibot CiBoT added a comment - Code verified against automated checks. Checked MDL-20053 using repository: git://github.com/abgreeve/moodle.git master (0 errors / 0 warnings) [branch: wip-MDL-20053-master | CI Job ] More information about this report
            Hide
            markn Mark Nelson added a comment -

            Hi Adrian, thanks for working on this.

            1. I don't like introducing another commit that fixes issues with a commit that has yet to be integrated. I know they are done by two different authors, but I feel since they are pretty minor they could be squashed. You can choose to ignore this, and wait for an integrator to decide.
            2. I don't believe the following code is necessary in both locations.

              // Remove the id from the string. This already exists in the sql statement.
              $namefields = str_replace('u.id,', '', $namefields);
              

            3. In data_get_advanced_search_sql() $namefields already contains 'u.firstname, u.lastname' in the GROUP BY.
            4. $OUTPUT->user_picture($ruser, array('courseid' => $cm->id)); is wrong, you should be using $data->course, not $cm->id.
            5. The testing instructions do not cover all the places you have edited the code.

            Cheers.

            Show
            markn Mark Nelson added a comment - Hi Adrian, thanks for working on this. I don't like introducing another commit that fixes issues with a commit that has yet to be integrated. I know they are done by two different authors, but I feel since they are pretty minor they could be squashed. You can choose to ignore this, and wait for an integrator to decide. I don't believe the following code is necessary in both locations. // Remove the id from the string. This already exists in the sql statement. $namefields = str_replace('u.id,', '', $namefields); In data_get_advanced_search_sql() $namefields already contains 'u.firstname, u.lastname' in the GROUP BY. $OUTPUT->user_picture($ruser, array('courseid' => $cm->id)); is wrong, you should be using $data->course, not $cm->id. The testing instructions do not cover all the places you have edited the code. Cheers.
            Hide
            abgreeve Adrian Greeve added a comment -

            Hello Mark, Thanks for the peer review.

            1. Okay, I have noted your objection, but I am going to leave this in two commits. I'd prefer to leave owership intact regardless of how small the contribution is.
            2. Yes, this is required in both places. If you don't put in that line you get the following debug message:

              Did you remember to make the first column something unique in your call to get_records? Duplicate value '2' found in column 'id'.
               
                  line 791 of /lib/dml/pgsql_native_moodle_database.php: call to debugging()
                  line 711 of /mod/data/view.php: call to pgsql_native_moodle_database->get_records_sql()
              

            3. This is separate to this patch and I don't know why you have mentioned it.
            4. Yes! This does need to be fixed. Done.
            5. Testing instructions expanded to include doing a search in advanced mode.

            Sending to integration.

            Show
            abgreeve Adrian Greeve added a comment - Hello Mark, Thanks for the peer review. Okay, I have noted your objection, but I am going to leave this in two commits. I'd prefer to leave owership intact regardless of how small the contribution is. Yes, this is required in both places. If you don't put in that line you get the following debug message: Did you remember to make the first column something unique in your call to get_records? Duplicate value '2' found in column 'id'.   line 791 of /lib/dml/pgsql_native_moodle_database.php: call to debugging() line 711 of /mod/data/view.php: call to pgsql_native_moodle_database->get_records_sql() This is separate to this patch and I don't know why you have mentioned it. Yes! This does need to be fixed. Done. Testing instructions expanded to include doing a search in advanced mode. Sending to integration.
            Hide
            markn Mark Nelson added a comment -

            For number 2 I wasn't able to replicate that issue, but will take your word for it. Regarding number 3, you are editing the variable that is used in the function, so I think that area I pointed out if perfectly reasonable to fix.

            Show
            markn Mark Nelson added a comment - For number 2 I wasn't able to replicate that issue, but will take your word for it. Regarding number 3, you are editing the variable that is used in the function, so I think that area I pointed out if perfectly reasonable to fix.
            Hide
            cibot CiBoT added a comment -

            Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!

            Show
            cibot CiBoT added a comment - Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!
            Hide
            cibot CiBoT added a comment -

            Code verified against automated checks.

            Checked MDL-20053 using repository: git://github.com/abgreeve/moodle.git

            More information about this report

            Show
            cibot CiBoT added a comment - Code verified against automated checks. Checked MDL-20053 using repository: git://github.com/abgreeve/moodle.git master (0 errors / 0 warnings) [branch: wip-MDL-20053-master | CI Job ] More information about this report
            Hide
            poltawski Dan Poltawski added a comment -

            Integrated to master, thanks Adrian.

            Show
            poltawski Dan Poltawski added a comment - Integrated to master, thanks Adrian.
            Show
            poltawski Dan Poltawski added a comment - Note, I commited a hacky fix for this issue breaking unit tests: http://git.moodle.org/gw?p=integration.git;a=commitdiff;h=fcb4acbde08d53a39f0913994b7a88f299053e61;hp=0b8a1d93d3ee01015d02ad809cecbde6ef25d5ac
            Hide
            abgreeve Adrian Greeve added a comment -

            Sorry Dan,

            Thanks for fixing that up for me.

            Show
            abgreeve Adrian Greeve added a comment - Sorry Dan, Thanks for fixing that up for me.
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Thanks for fixing this Adrian and Dan,

            User picture tag works as expected.

            Passing...

            Show
            rajeshtaneja Rajesh Taneja added a comment - Thanks for fixing this Adrian and Dan, User picture tag works as expected. Passing...
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Sorry guys this is failing behat...
            There is a fail happening while deleting record

            01. debugging() message/s found:
                Missing picture property in $user object, this is a performance problem that needs to be fixed by a developer. Please use user_picture::fields() to get the full list of required fields.                                                   
                line 201 of /lib/outputcomponents.php: call to debugging()                                                          
                line 2321 of /lib/outputrenderers.php: call to user_picture->__construct()                                          
                line 1277 of /mod/data/lib.php: call to core_renderer->user_picture()
                line 394 of /mod/data/view.php: call to data_print_template()
                In step `And I follow "Delete"'.                           # behat_general::click_link()
                From scenario `Students can add entries to a database'.    # /home/rajesh/moodles/im/moodle/mod/data/tests/behat/add_entries.feature:7
                Of feature `Users can add entries to database activities'. 
            

            Show
            rajeshtaneja Rajesh Taneja added a comment - Sorry guys this is failing behat... There is a fail happening while deleting record 01. debugging() message/s found: Missing picture property in $user object, this is a performance problem that needs to be fixed by a developer. Please use user_picture::fields() to get the full list of required fields. line 201 of /lib/outputcomponents.php: call to debugging() line 2321 of /lib/outputrenderers.php: call to user_picture->__construct() line 1277 of /mod/data/lib.php: call to core_renderer->user_picture() line 394 of /mod/data/view.php: call to data_print_template() In step `And I follow "Delete"'. # behat_general::click_link() From scenario `Students can add entries to a database'. # /home/rajesh/moodles/im/moodle/mod/data/tests/behat/add_entries.feature:7 Of feature `Users can add entries to database activities'.
            Hide
            dobedobedoh Andrew Nicols added a comment -

            I've patched the issue on line 379, but this did not reveal the issues later on (line 419). Adrian Greeve, I understand that you're working on a fix for these - can you ensure that they include behat tests?

            Cheers,

            Andrew

            Show
            dobedobedoh Andrew Nicols added a comment - I've patched the issue on line 379, but this did not reveal the issues later on (line 419). Adrian Greeve , I understand that you're working on a fix for these - can you ensure that they include behat tests? Cheers, Andrew
            Hide
            abgreeve Adrian Greeve added a comment - - edited

            Sorry about that. I have created a new commit that takes into consideration deleting one and multiple entries.

            The branch is
            wip-MDL-20053master-fix <- this is based on the current integration branch.
            repo: git://github.com/abgreeve/moodle.git
            Edit I didn't see your message before Andrew. I'll rebase my branch.
            Adding behat tests probably means that this fix should be reverted.

            Show
            abgreeve Adrian Greeve added a comment - - edited Sorry about that. I have created a new commit that takes into consideration deleting one and multiple entries. The branch is wip- MDL-20053 master-fix < - this is based on the current integration branch. repo: git://github.com/abgreeve/moodle.git Edit I didn't see your message before Andrew. I'll rebase my branch. Adding behat tests probably means that this fix should be reverted.
            Hide
            abgreeve Adrian Greeve added a comment - - edited

            commit for this patch is:
            https://github.com/abgreeve/moodle/commit/efc2acb0aa6d6f05db086fecf7d0b3be20cea4dc
            I'm creating an issue to create behat tests for this fix now. I'll link it to this issue when I have done that.

            Next commit >_>
            https://github.com/abgreeve/moodle/commit/3d326f632608b431d97a37ec26db79e2e721d6ac

            Show
            abgreeve Adrian Greeve added a comment - - edited commit for this patch is: https://github.com/abgreeve/moodle/commit/efc2acb0aa6d6f05db086fecf7d0b3be20cea4dc I'm creating an issue to create behat tests for this fix now. I'll link it to this issue when I have done that. Next commit >_> https://github.com/abgreeve/moodle/commit/3d326f632608b431d97a37ec26db79e2e721d6ac
            Hide
            poltawski Dan Poltawski added a comment -

            Back to testing - thanks

            Show
            poltawski Dan Poltawski added a comment - Back to testing - thanks
            Hide
            rajeshtaneja Rajesh Taneja added a comment -

            Sorry Adrian,

            This is failing again.

            Multi select deletion is not working.
            Steps to reproduce:

            1. Follow testing instructions to create multiple records
            2. Select all and Click delete selected.
            3. Click "Delete" and nothing happens.
            Show
            rajeshtaneja Rajesh Taneja added a comment - Sorry Adrian, This is failing again. Multi select deletion is not working. Steps to reproduce: Follow testing instructions to create multiple records Select all and Click delete selected. Click "Delete" and nothing happens.
            Hide
            poltawski Dan Poltawski added a comment -

            This issue has been reverted.

            These were the commits:

            commit 9b7f427180de97954b40077d0f809305d4555c72
            Author: Adrian Greeve <adrian@moodle.com>
            Date:   Thu Jan 15 16:12:49 2015 +0800
             
                MDL-20053 mod_data: Fix to silly loop mistake when deleting.
            commit 68db2a63caa39432f009fc448ce615b0c8bce07d
            Author: Adrian Greeve <adrian@moodle.com>
            Date:   Thu Jan 15 15:45:01 2015 +0800
             
                MDL-20053 mod_data: Fixed up additional errors when deleting.
             
            commit 3dfd44774efc4fd093b14f9eb6c45a6f02091511
            Author: Andrew Nicols <andrew@nicols.co.uk>
            Date:   Thu Jan 15 15:03:00 2015 +0800
             
                MDL-20053 mod_data: Add missing user_picture fields
                
                These were causing unit test failures.
             
            commit fcb4acbde08d53a39f0913994b7a88f299053e61
            Author: Dan Poltawski <dan@moodle.com>
            Date:   Mon Jan 12 14:27:00 2015 +0000
             
                MDL-20053 mod_data: fix unit test
                
                Add user image fields (bit of a hacky solution)
             
            commit 41d03d61c50424ea61c619f7ba20b04cfe7fb17c
            Merge: b79dca8 bd1ce4a
            Author: Dan Poltawski <dan@moodle.com>
            Date:   Mon Jan 12 12:49:55 2015 +0000
             
                Merge branch 'wip-MDL-20053-master' of git://github.com/abgreeve/moodle
             
            commit bd1ce4a0639d0a4bf645750435dacf5ce9d442d0
            Author: Adrian Greeve <adrian@moodle.com>
            Date:   Mon Jan 5 09:38:17 2015 +0800
             
                MDL-20053 mod_data: Removal of individual DB call.
                
                The main functions for retrieving records have been
                updated to include user picture information.
             
            commit 7b5347337c0dd9ffcfd9661b41fdbb71e9e37b4a
            Author: Anthony Borrow <anthony@moodle.org>
            Date:   Fri Jan 2 13:52:40 2015 +0800
             
                MDL-20053 mod_data: Add userpicture tag to show profile pictures
            

            Show
            poltawski Dan Poltawski added a comment - This issue has been reverted. These were the commits: commit 9b7f427180de97954b40077d0f809305d4555c72 Author: Adrian Greeve <adrian@moodle.com> Date: Thu Jan 15 16:12:49 2015 +0800   MDL-20053 mod_data: Fix to silly loop mistake when deleting. commit 68db2a63caa39432f009fc448ce615b0c8bce07d Author: Adrian Greeve <adrian@moodle.com> Date: Thu Jan 15 15:45:01 2015 +0800   MDL-20053 mod_data: Fixed up additional errors when deleting.   commit 3dfd44774efc4fd093b14f9eb6c45a6f02091511 Author: Andrew Nicols <andrew@nicols.co.uk> Date: Thu Jan 15 15:03:00 2015 +0800   MDL-20053 mod_data: Add missing user_picture fields These were causing unit test failures.   commit fcb4acbde08d53a39f0913994b7a88f299053e61 Author: Dan Poltawski <dan@moodle.com> Date: Mon Jan 12 14:27:00 2015 +0000   MDL-20053 mod_data: fix unit test Add user image fields (bit of a hacky solution)   commit 41d03d61c50424ea61c619f7ba20b04cfe7fb17c Merge: b79dca8 bd1ce4a Author: Dan Poltawski <dan@moodle.com> Date: Mon Jan 12 12:49:55 2015 +0000   Merge branch 'wip-MDL-20053-master' of git://github.com/abgreeve/moodle   commit bd1ce4a0639d0a4bf645750435dacf5ce9d442d0 Author: Adrian Greeve <adrian@moodle.com> Date: Mon Jan 5 09:38:17 2015 +0800   MDL-20053 mod_data: Removal of individual DB call. The main functions for retrieving records have been updated to include user picture information.   commit 7b5347337c0dd9ffcfd9661b41fdbb71e9e37b4a Author: Anthony Borrow <anthony@moodle.org> Date: Fri Jan 2 13:52:40 2015 +0800   MDL-20053 mod_data: Add userpicture tag to show profile pictures
            Hide
            cibot CiBoT added a comment -

            Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.

            Show
            cibot CiBoT added a comment - Moving this reopened issue out from current integration. Please, re-submit it for integration once ready.
            Hide
            abgreeve Adrian Greeve added a comment -

            Hello,

            I have reworked the patch. This now incorporates all of the fixes in the additional commits from before.

            • User picture is displayed when added to the single and list view.
            • Single delete works.
            • bulk delete works.
            • Unit tests have been updated and pass.
            • I have updated a behat test to check for bulk deleting and this also passes.
              Sending back to integration.
            Show
            abgreeve Adrian Greeve added a comment - Hello, I have reworked the patch. This now incorporates all of the fixes in the additional commits from before. User picture is displayed when added to the single and list view. Single delete works. bulk delete works. Unit tests have been updated and pass. I have updated a behat test to check for bulk deleting and this also passes. Sending back to integration.
            Hide
            cibot CiBoT added a comment -

            Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!

            Show
            cibot CiBoT added a comment - Moving this issue to current integration cycle, will be reviewed soon. Thanks for the hard work!
            Hide
            cibot CiBoT added a comment -

            Fails against automated checks.

            Checked MDL-20053 using repository: git://github.com/abgreeve/moodle.git

            More information about this report

            Show
            cibot CiBoT added a comment - Fails against automated checks. Checked MDL-20053 using repository: git://github.com/abgreeve/moodle.git master (2 errors / 1 warnings) [branch: wip-MDL-20053-master | CI Job ] phplint (0/0) , php (2/1) , js (0/0) , css (0/0) , phpdoc (0/0) , commit (0/0) , savepoint (0/0) , thirdparty (0/0) , More information about this report
            Hide
            abgreeve Adrian Greeve added a comment -

            Cibot php errors relate to formatting. This formatting matches the rest of the file, so I am choosing to leave it the way it is.

            Show
            abgreeve Adrian Greeve added a comment - Cibot php errors relate to formatting. This formatting matches the rest of the file, so I am choosing to leave it the way it is.
            Hide
            dmonllao David Monllaó added a comment -

            Thanks all, integrated to master.

            Show
            dmonllao David Monllaó added a comment - Thanks all, integrated to master.
            Hide
            davecooper Dave Cooper added a comment -

            Looks good - marking tests as passed.

            Show
            davecooper Dave Cooper added a comment - Looks good - marking tests as passed.
            Hide
            dobedobedoh Andrew Nicols added a comment -

            Thank you for your contribution. This change is now a part of Moodle and is available from git.moodle.org. It will shortly be available at https://downlaod.moodle.org too!

            A diamond is merely a lump of coal that did well under pressure.
            – Unknown

            Show
            dobedobedoh Andrew Nicols added a comment - Thank you for your contribution. This change is now a part of Moodle and is available from git.moodle.org. It will shortly be available at https://downlaod.moodle.org too! A diamond is merely a lump of coal that did well under pressure. – Unknown
            Hide
            marycooch Mary Cooch added a comment -

            Removing the docs_required label as this improvement is now documented: https://docs.moodle.org/29/en/Database_templates#Tag_usage

            If you notice that the documentation can be improved, please feel free to log in to the wiki and edit the page.

            Help in keeping Moodle documentation accurate and up-to-date is much appreciated

            Show
            marycooch Mary Cooch added a comment - Removing the docs_required label as this improvement is now documented: https://docs.moodle.org/29/en/Database_templates#Tag_usage If you notice that the documentation can be improved, please feel free to log in to the wiki and edit the page. Help in keeping Moodle documentation accurate and up-to-date is much appreciated

              People

              • Votes:
                8 Vote for this issue
                Watchers:
                13 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/May/15