Moodle
  1. Moodle
  2. MDL-18083

META: Gradebook improvements - Stage 1

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.4
    • Fix Version/s: 1.9.5
    • Component/s: Gradebook, Usability
    • Labels:
      None
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_19_STABLE
    • Rank:
      36536
    1. gradebook_1.9.5.patch
      267 kB
      Nicolas Connault
    2. index.php
      12 kB
      Matt Gibson
    3. styles.php
      14 kB
      Matt Gibson
    1. gradebook.JPG
      57 kB

      Issue Links

      Progress
      Resolved Sub-Tasks

      Sub-Tasks

      1.
      Gradebook: Assign weights to all categories and items on one screen Sub-task Closed Nicolas Connault
       
      2.
      Combine category grade item and category edit forms Sub-task Closed Nicolas Connault
       
      3.
      Tabs navigation for gradebook Sub-task Closed Nicolas Connault
       
      4.
      Grader report breadcrumbs are too long with editing on Sub-task Closed Nicolas Connault
       
      5.
      Adding a help button next to the "Synchronise legacy grades" in gradebook 1.9 Sub-task Closed Nicolas Connault
       
      6.
      Add horizontal scrollbar to the grader report Sub-task Closed Nicolas Connault
       
      7.
      Change gradereport/overview:view default setting to 'Not set' for the role of teacher and non-editing teacher in new installs Sub-task Closed Nicolas Connault
       
      8.
      Outcomes report is available even when outcomes are disabled Sub-task Closed Nicolas Connault
       
      9.
      In practice, grades can go over 100% Sub-task Closed Nicolas Connault
       
      10.
      [PATCH] New visual organization for gradebook user report (including category nesting, ranges) Sub-task Closed Nicolas Connault
       
      11.
      regression: sum icon deleted in grade/lib.php Sub-task Closed Petr Škoda
       
      12.
      regressions: performance get_parent_category() used instead of load_parent_category() Sub-task Closed Petr Škoda
       
      13.
      regression: execute_sql in gradelib.php prints feedback Sub-task Closed Petr Škoda
       
      14.
      regression: double colon - Move selected items to: : Sub-task Closed Petr Škoda
       
      15.
      regression: the dropdown section names are selectable Sub-task Closed Petr Škoda
       
      16.
      regression: PHP5isms function()->function() Sub-task Closed Petr Škoda
       
      17.
      regressions: no exceptions in PHP4! Sub-task Closed Petr Škoda
       
      18.
      regression: Must not assign default value to variable passed by reference Sub-task Closed Petr Škoda
       
      19.
      Only display the 'Synchronise legacy grades' button when necessary Sub-task Closed Petr Škoda
       
      20.
      group support in user gradebook report Sub-task Closed Petr Škoda
       
      21.
      group support in overview gradebook report Sub-task Closed Petr Škoda
       
      22.
      regression: user report does not validate if more users printed Sub-task Closed Petr Škoda
       
      23.
      rework group support on grader report Sub-task Closed Petr Škoda
       
      24.
      Need to be able to add extra credit items when simple mean aggreagation used Sub-task Closed Petr Škoda
       
      25.
      upgrade sets aggregation to weighted mean with 0 weight Sub-task Closed Petr Škoda
       
      26.
      Tooltips over grade cells in grader report Sub-task Closed Nicolas Connault
       
      27.
      add group accesss control to gradebook csv improt Sub-task Closed Petr Škoda
       
      28.
      implement group access control in gradebook export plugins Sub-task Closed Petr Škoda
       
      29.
      new grader report - formslib error when editting item on item edit form Sub-task Closed Petr Škoda
       
      30.
      Gradebook miscalculates aggregates using scales (custom or site-wide) Sub-task Closed Petr Škoda
       
      31.
      proper locking of grade type and min/max when sum agg selected Sub-task Closed Petr Škoda
       
      32.
      sum max does not consider drop low and high Sub-task Closed Petr Škoda
       
      33.
      extra credis should not be dropped when drop low or keep high specified Sub-task Closed Petr Škoda
       
      34.
      multiplicator and offset are used only for activity grades - must be disabled elsewhere Sub-task Closed Petr Škoda
       
      35.
      Make full use of title attributes in edit categories and items page Sub-task Closed Nicolas Connault
       
      36.
      remove category item editing option from item edit form Sub-task Closed Petr Škoda
       
      37.
      new category edit form does not validate Sub-task Closed Petr Škoda
       
      38.
      Restrict teachers to their own groups when accessing User Reports Sub-task Closed Petr Škoda
       
      39.
      Showing all groups in gradebook when user restricted to one Sub-task Closed Petr Škoda
       
      40.
      Typo in grade/lib.php causes incorrect sorting Sub-task Closed Petr Škoda
       
      41.
      Preview and actual export inconsistent for XML grade export Sub-task Closed Petr Škoda
       
      42.
      Hide/Show groups link in grader report should be hidden when groups are not activated in course Sub-task Closed Petr Škoda
       
      43.
      gradebook reports and plugins do not validate Sub-task Closed Petr Škoda
       
      44.
      Gradebook export - "Pragma: no-cache" header prevents download over SSL in IE Sub-task Closed Petr Škoda
       

        Activity

        Hide
        Petr Škoda added a comment -

        Please see linked bug.

        Show
        Petr Škoda added a comment - Please see linked bug.
        Hide
        Nicolas Connault added a comment -

        This is a very large and important patch, so I am waiting for all issues to be resolved before I submit the code to CVS.

        Show
        Nicolas Connault added a comment - This is a very large and important patch, so I am waiting for all issues to be resolved before I submit the code to CVS.
        Hide
        Nicolas Connault added a comment -

        Attached big patch

        Show
        Nicolas Connault added a comment - Attached big patch
        Hide
        Nicolas Connault added a comment -

        Just updated the patch with a few bug fixes.

        Show
        Nicolas Connault added a comment - Just updated the patch with a few bug fixes.
        Hide
        Nicolas Connault added a comment -

        Patch updated again

        Show
        Nicolas Connault added a comment - Patch updated again
        Hide
        Nicolas Connault added a comment -

        Updated patch again

        Show
        Nicolas Connault added a comment - Updated patch again
        Hide
        Petr Škoda added a comment - - edited

        security and coding problems:
        1/ CSRFs in the new tree/index.php code - we MUST test sesskey before changing anything in db
        2/ wordwrap(addslashes_js is not correct, the addslashes_js should be the last step - grader report
        3/ text going into overlib should be processed with addslashes_js() too - s($intersection_info) in grader report

        Show
        Petr Škoda added a comment - - edited security and coding problems: 1/ CSRFs in the new tree/index.php code - we MUST test sesskey before changing anything in db 2/ wordwrap(addslashes_js is not correct, the addslashes_js should be the last step - grader report 3/ text going into overlib should be processed with addslashes_js() too - s($intersection_info) in grader report
        Hide
        Gary Anderson added a comment -

        I have applied this patch to a local copy of Moodle 1.9 stable for testing.

        I received two errors:

        \moodle\lang\en_utf8\gradereport_grader.php (Cannot apply hunk @@ 7 )
        \moodle\lib\simpletest\testmoodlelib.php (Cannot apply hunk @@ 4 )

        Since both seemed to be mostly harmless, I proceed with testing, initially trying to recreate the Moodle Gradebook Tutorial I write for for docs.moodle.org.

        I did not see the new tabs in the grader page. There is also no way to get to Categories and items that I can see (the menu item is missing).

        I will start testing this and then try it with my school's many courses once this is ready.

        I was happy to see that turning off the unlimited grading feature did not change the scores already entered (only the summaries).

        --Gary

        Show
        Gary Anderson added a comment - I have applied this patch to a local copy of Moodle 1.9 stable for testing. I received two errors: \moodle\lang\en_utf8\gradereport_grader.php (Cannot apply hunk @@ 7 ) \moodle\lib\simpletest\testmoodlelib.php (Cannot apply hunk @@ 4 ) Since both seemed to be mostly harmless, I proceed with testing, initially trying to recreate the Moodle Gradebook Tutorial I write for for docs.moodle.org. I did not see the new tabs in the grader page. There is also no way to get to Categories and items that I can see (the menu item is missing). I will start testing this and then try it with my school's many courses once this is ready. I was happy to see that turning off the unlimited grading feature did not change the scores already entered (only the summaries). --Gary
        Hide
        Nicolas Connault added a comment -

        Gary,

        The new patch includes 4 new files, which you seem to be missing. Depending on how you are applying the patch and your file and directory permissions, these files may not be correctly created in your source tree. Please make sure you have the following:

        grade/edit/tree/lib.php
        grade/edit/tree/functions.js
        grade/edit/tree/tree.css
        grade/report/styles.css

        Show
        Nicolas Connault added a comment - Gary, The new patch includes 4 new files, which you seem to be missing. Depending on how you are applying the patch and your file and directory permissions, these files may not be correctly created in your source tree. Please make sure you have the following: grade/edit/tree/lib.php grade/edit/tree/functions.js grade/edit/tree/tree.css grade/report/styles.css
        Hide
        Gary Anderson added a comment -

        Note that after chatting with Nicolas, it turns out these items are there. But since the Category and items entry has become a heading and changed its location in the menu structure, it was not where I was expecting it and I missed it.

        It is now near the top, where it belongs.

        Also, tabs were turned off by default which is not what I expected.

        I will add these to my notes as others will probably have similar problems. A fix would probably be to have tabs and the menu on by default.

        I will continue testing over the next few days as I have time.

        Show
        Gary Anderson added a comment - Note that after chatting with Nicolas, it turns out these items are there. But since the Category and items entry has become a heading and changed its location in the menu structure, it was not where I was expecting it and I missed it. It is now near the top, where it belongs. Also, tabs were turned off by default which is not what I expected. I will add these to my notes as others will probably have similar problems. A fix would probably be to have tabs and the menu on by default. I will continue testing over the next few days as I have time.
        Hide
        Nicolas Connault added a comment -

        Actually the tabs must be turned off by default, this is what most people will be expecting, since it keeps the previous behaviour. This is documented on http://docs.moodle.org/en/Development:Gradebook_improvements_in_Moodle_1.9.5#Tabs_navigation

        I hope other users won't give up looking for "Categories and items" as quickly as you did, though

        Show
        Nicolas Connault added a comment - Actually the tabs must be turned off by default, this is what most people will be expecting, since it keeps the previous behaviour. This is documented on http://docs.moodle.org/en/Development:Gradebook_improvements_in_Moodle_1.9.5#Tabs_navigation I hope other users won't give up looking for "Categories and items" as quickly as you did, though
        Hide
        Elena Ivanova added a comment -

        Should we post our testing notes here too?
        We have applied the patch to our test environment with the course data, and everything looks great so far! Most previous calculations seem to be in place.
        Several of our faculty tested it as well, and liked it a lot. I am going to post a summary in the forum later.
        The only thing that we have got for some reason is the missing indents in the dropdown menu. Here is a screenshot: http://umn.edu/moodle/samples/dropdown.png

        Show
        Elena Ivanova added a comment - Should we post our testing notes here too? We have applied the patch to our test environment with the course data, and everything looks great so far! Most previous calculations seem to be in place. Several of our faculty tested it as well, and liked it a lot. I am going to post a summary in the forum later. The only thing that we have got for some reason is the missing indents in the dropdown menu. Here is a screenshot: http://umn.edu/moodle/samples/dropdown.png
        Hide
        Petr Škoda added a comment -

        indents: Looks like a CSS issue, could you please try purging your browser cache and using standard theme?

        Show
        Petr Škoda added a comment - indents: Looks like a CSS issue, could you please try purging your browser cache and using standard theme?
        Hide
        Nicolas Connault added a comment -

        I confirm Petr's comments. I had the same issue until I reset my browser's cache (ctrl+F5).

        Show
        Nicolas Connault added a comment - I confirm Petr's comments. I had the same issue until I reset my browser's cache (ctrl+F5).
        Hide
        Elena Ivanova added a comment -

        Hi, yes, I completely forgot that our css was not updated there!

        I think I found more bugs. Well, some of them are things that need interface improvement. Here are my notes (Please let me know if I should post them somewhere else or create separate tickets).

        1. Non-graded Assignment and Quizzes are getting displayed in the gradebook, and they look like being included into aggregation in the Cat&Items interface (Other activities behave definitely. E.g. If Forum is not graded, it does not appear in the gradebook as a column. Which for me is an expected behavior)

        • Add an Assignment. Set it to No Grade. Column will still appear in the gradebook. Now go to the Cat&Items interface. It will display Max Grade for that column to be 100 points. (Though it will not be actually included into the calculation)
        • Add an Assignment. Set it to be graded on a scale (scales are NOT included in aggregation for us). Maximum Grade in the Cat&Items interface will display some number (e.g. 3, if there are 3 items in a scale). It is confusing, since one would now expect that 3 to be calculated toward course/category total.
        • Add a quiz, add some questions, and make the quiz to be worth 0 points. Maximum Grade in the Cat&Items interface will still be 10 points (or 100). Interestingly enough, I can manually delete a gradebook column for this kind of quiz.

        I think it MDL-14017 is related

        2. Once you add a category and set Aggregation method to Sum, it defaults to be worth 100 points (even though this category can have nothing inside). Those 100 points are included in the calculation of the course total, so students are getting penalized for nothing.
        Is it possible to default Max grade to 0 points in such case?

        3. Weird "extra credit "check mark, if the Course Total is set to Sum aggregation
        When I look on the Cat&Items interface, it looks fine (no extra credits marked anywhere). Then, I click Edit icon near the name of any category, and it has the Extra Credit checkbox marked.

        Show
        Elena Ivanova added a comment - Hi, yes, I completely forgot that our css was not updated there! I think I found more bugs. Well, some of them are things that need interface improvement. Here are my notes (Please let me know if I should post them somewhere else or create separate tickets). 1. Non-graded Assignment and Quizzes are getting displayed in the gradebook, and they look like being included into aggregation in the Cat&Items interface (Other activities behave definitely. E.g. If Forum is not graded, it does not appear in the gradebook as a column. Which for me is an expected behavior) Add an Assignment. Set it to No Grade. Column will still appear in the gradebook. Now go to the Cat&Items interface. It will display Max Grade for that column to be 100 points. (Though it will not be actually included into the calculation) Add an Assignment. Set it to be graded on a scale (scales are NOT included in aggregation for us). Maximum Grade in the Cat&Items interface will display some number (e.g. 3, if there are 3 items in a scale). It is confusing, since one would now expect that 3 to be calculated toward course/category total. Add a quiz, add some questions, and make the quiz to be worth 0 points. Maximum Grade in the Cat&Items interface will still be 10 points (or 100). Interestingly enough, I can manually delete a gradebook column for this kind of quiz. I think it MDL-14017 is related 2. Once you add a category and set Aggregation method to Sum, it defaults to be worth 100 points (even though this category can have nothing inside). Those 100 points are included in the calculation of the course total, so students are getting penalized for nothing. Is it possible to default Max grade to 0 points in such case? 3. Weird "extra credit "check mark, if the Course Total is set to Sum aggregation When I look on the Cat&Items interface, it looks fine (no extra credits marked anywhere). Then, I click Edit icon near the name of any category, and it has the Extra Credit checkbox marked.
        Hide
        Nicolas Connault added a comment -

        Helena, your items 1 and 2 are valid but not related to this tracker issue. No doubt we will still find small bugs in the gradebook 6 months from now, but we don't need to delay the current patch for them. If you could create a separate tracker issue for each of these, it would be very appreciated.

        Item 3 has just been fixed, I'm updating the patch now.

        Show
        Nicolas Connault added a comment - Helena, your items 1 and 2 are valid but not related to this tracker issue. No doubt we will still find small bugs in the gradebook 6 months from now, but we don't need to delay the current patch for them. If you could create a separate tracker issue for each of these, it would be very appreciated. Item 3 has just been fixed, I'm updating the patch now.
        Hide
        Matt Gibson added a comment -

        Just looking at the test site, I noticed the immensely useful tooltips for each cell. Nice one!

        However, I also noticed from the markup that they are using overlib and that each one is defined individually. I think it would be far better to use YUI panel and attach one event listener to the gradebook table so that all of these panels are defined in one place. Is overlib in the process of being phased out in favour of YUI, or is it being kept in parallell?

        Apart from MDL-13433 (sometimes, long tooltips go off the bottom of the page), my major reason for saying this is that it would save effor later in the AJAXifying of the gradebook, when I'm sure we will want to be able to edit cells using YUI dialogues (which are extensions of the panel module) as well as perhaps adding more information to the cells or altering the styles so that the tooltips are sticky or have links etc.

        If the data was placed in the cell in standard module format http://developer.yahoo.com/yui/examples/container/module.html
        ready to be picked up by the mouseover event handler, then it could be easily extended dynamically for later display, i.e. adding/altering a div after a cell edit, which will be difficult at the moment, using hardcoded function calls.

        Show
        Matt Gibson added a comment - Just looking at the test site, I noticed the immensely useful tooltips for each cell. Nice one! However, I also noticed from the markup that they are using overlib and that each one is defined individually. I think it would be far better to use YUI panel and attach one event listener to the gradebook table so that all of these panels are defined in one place. Is overlib in the process of being phased out in favour of YUI, or is it being kept in parallell? Apart from MDL-13433 (sometimes, long tooltips go off the bottom of the page), my major reason for saying this is that it would save effor later in the AJAXifying of the gradebook, when I'm sure we will want to be able to edit cells using YUI dialogues (which are extensions of the panel module) as well as perhaps adding more information to the cells or altering the styles so that the tooltips are sticky or have links etc. If the data was placed in the cell in standard module format http://developer.yahoo.com/yui/examples/container/module.html ready to be picked up by the mouseover event handler, then it could be easily extended dynamically for later display, i.e. adding/altering a div after a cell edit, which will be difficult at the moment, using hardcoded function calls.
        Hide
        Elena Ivanova added a comment -

        Thank you, Nicolas.
        I have created MDL-18770 and MDL-18772 (I could not choose 1.9.5 as "affected"version though).
        Sorry if I will add more non-related stuff here. It is hard to distinguish which bugs should be considered for this release, and which for the next ones.
        p.s. I am still struggling with the css stuff in the gradebook (items are non-indented even in the standard theme, plus my editing controls for columns/items are 2 columns off from where they are supposed to be).

        Show
        Elena Ivanova added a comment - Thank you, Nicolas. I have created MDL-18770 and MDL-18772 (I could not choose 1.9.5 as "affected"version though). Sorry if I will add more non-related stuff here. It is hard to distinguish which bugs should be considered for this release, and which for the next ones. p.s. I am still struggling with the css stuff in the gradebook (items are non-indented even in the standard theme, plus my editing controls for columns/items are 2 columns off from where they are supposed to be).
        Hide
        Nicolas Connault added a comment -

        Thanks for your idea, Matt. I've followed your advice and replaced overlib with the YUI Tooltip widget. I've improved the look of the tooltip, and fixed the issue of category totals not showing. I've also updated the patch attached to this issue.

        Show
        Nicolas Connault added a comment - Thanks for your idea, Matt. I've followed your advice and replaced overlib with the YUI Tooltip widget. I've improved the look of the tooltip, and fixed the issue of category totals not showing. I've also updated the patch attached to this issue.
        Hide
        Matt Gibson added a comment -

        Linking to the issue I commented about.

        Thanks for the changes Nicolas - it's much better and fixes this issue (almost). I've added huge feedback comments at the start and end of the lowest row of the gradebook in the 1.9 test site gradebook test course. A bit of an issue still with the tooltips vanishing when they are hovered over.

        Show
        Matt Gibson added a comment - Linking to the issue I commented about. Thanks for the changes Nicolas - it's much better and fixes this issue (almost). I've added huge feedback comments at the start and end of the lowest row of the gradebook in the 1.9 test site gradebook test course. A bit of an issue still with the tooltips vanishing when they are hovered over.
        Hide
        Matt Gibson added a comment -

        I think the tooltips would be complete with a fix for MDL-13902 as the quiz essay questions are the only place where human inputted feedback will be missing from both the grader report and user report. I have stopped using quizzes for timed essays because of this, as the students don't even realise there are any comments there at all a lot of the time.

        I suggest:

        <Name>
        <Quiz name>

        "General feedback"
        <General feedback>

        "Manually enetered feedback on essay question answers follows (automatic feedback on your other answers available in the quiz itself)"

        "Question 1 "
        <teacher feedback>

        "Question 2"
        <teacher feedback>

        etc

        Show
        Matt Gibson added a comment - I think the tooltips would be complete with a fix for MDL-13902 as the quiz essay questions are the only place where human inputted feedback will be missing from both the grader report and user report. I have stopped using quizzes for timed essays because of this, as the students don't even realise there are any comments there at all a lot of the time. I suggest: <Name> <Quiz name> "General feedback" <General feedback> "Manually enetered feedback on essay question answers follows (automatic feedback on your other answers available in the quiz itself)" "Question 1 " <teacher feedback> "Question 2" <teacher feedback> etc
        Hide
        Jesus Martin added a comment -

        Hi.

        When I apply the last patch (03/Apr/09) and I try to configure the categories, there is a fatal error:

        Fatal error: Cannot redeclare class grade_edit_tree_column_category in /var/www/moodle/grade/edit/tree/lib.php on line 1454

        Thanks

        Show
        Jesus Martin added a comment - Hi. When I apply the last patch (03/Apr/09) and I try to configure the categories, there is a fatal error: Fatal error: Cannot redeclare class grade_edit_tree_column_category in /var/www/moodle/grade/edit/tree/lib.php on line 1454 Thanks
        Hide
        Elena Ivanova added a comment -

        Students can see hidden grades and categories in the gradebook if the new patch is applied (even if you set viewhidden capability to Prevent as an admin).
        Just in case I have tested regular 1.9.4 and it was OK (students could not see hidden grades).

        Show
        Elena Ivanova added a comment - Students can see hidden grades and categories in the gradebook if the new patch is applied (even if you set viewhidden capability to Prevent as an admin). Just in case I have tested regular 1.9.4 and it was OK (students could not see hidden grades).
        Hide
        Nicolas Connault added a comment -

        Thanks for the bug report, Elena, I fixed it and updated the patch file.

        Show
        Nicolas Connault added a comment - Thanks for the bug report, Elena, I fixed it and updated the patch file.
        Hide
        Elena Ivanova added a comment -

        I wold love to test it, but getting an error at Cat and Items page: "Fatal error: Cannot redeclare class grade_edit_tree_column_category in /var/www/html/test/1.9/grade/edit/tree/lib.php on line 1454"
        (May be you are working on it at the moment).

        Show
        Elena Ivanova added a comment - I wold love to test it, but getting an error at Cat and Items page: "Fatal error: Cannot redeclare class grade_edit_tree_column_category in /var/www/html/test/1.9/grade/edit/tree/lib.php on line 1454" (May be you are working on it at the moment).
        Hide
        Nicolas Connault added a comment -

        The patch got applied in a funny way, I've fixed it now.

        Show
        Nicolas Connault added a comment - The patch got applied in a funny way, I've fixed it now.
        Hide
        Elena Ivanova added a comment -

        If you login as a real student, User report is getting misaligned.
        Studnet view: http://umn.edu/moodle/samples/student1.jpg
        Instructor view: http://umn.edu/moodle/samples/instr1.jpg

        Show
        Elena Ivanova added a comment - If you login as a real student, User report is getting misaligned. Studnet view: http://umn.edu/moodle/samples/student1.jpg Instructor view: http://umn.edu/moodle/samples/instr1.jpg
        Hide
        Gary Anderson added a comment -

        In the new Category and items view, whether the weight and extra credit columns appear depends on whether certain aggregation types are used someplace in the grading method. This can be confusing, as neither is true in the default case (Simple weighted mean of grades), so it make it difficult to know the options are even available.

        I would suggest showing those columns in all cases, even if there is no checkbox of fields for methods where these features are not yet implemented. This would mean that the columns are always consistent, and users would be able to discover which methods have the feature implemented.

        Either or both of these columns could be visible in only the "full view" if desired (or keep both in the simple view).

        Removing the conditional check at line 62 and 65 of \grade\edit\tree solves this issue.

        Show
        Gary Anderson added a comment - In the new Category and items view, whether the weight and extra credit columns appear depends on whether certain aggregation types are used someplace in the grading method. This can be confusing, as neither is true in the default case (Simple weighted mean of grades), so it make it difficult to know the options are even available. I would suggest showing those columns in all cases, even if there is no checkbox of fields for methods where these features are not yet implemented. This would mean that the columns are always consistent, and users would be able to discover which methods have the feature implemented. Either or both of these columns could be visible in only the "full view" if desired (or keep both in the simple view). Removing the conditional check at line 62 and 65 of \grade\edit\tree solves this issue.
        Hide
        Petr Škoda added a comment -

        hmm, I personally think we do not need to show those weight and extra credit columns because not everybody needs/wants them

        Show
        Petr Škoda added a comment - hmm, I personally think we do not need to show those weight and extra credit columns because not everybody needs/wants them
        Hide
        Gary Anderson added a comment -

        Petr:

        I don't think the criteria for including items on the Category and item page is one's view on whether everyone will want our use them. For example, I am fairly certain that more people will use Extra Credit and Weight than will use multiplicator and offset. In the full view, at least, this page should show any setting that may affect computation.

        The more subtle point in my last suggestion is whether a column should appear or be hidden depending on if an aggregation method someplace within a course is able to use the feature. It is a balance between three usability qualities: minimalism, discoverability, and completeness.

        Having tested the feature, I am of the view that columns should not be changing around as one tries different settings of aggregation methods. The reason is that, if one does want extra credit or weight (or any other feature), having the column always there means that one knows they are on the right page to determine if a feature is enabled or not. And I think that is more important than the screen space saved by not displaying the column. Since the cells will be empty when not used, I don't think it adds too much to the complexity of the page.

        In both cases I think it is fine to put these columns in the full view, which gives a very clean display for the most simple (and the default) gradebook which will be fine and less intimidating for first time teachers.

        Show
        Gary Anderson added a comment - Petr: I don't think the criteria for including items on the Category and item page is one's view on whether everyone will want our use them. For example, I am fairly certain that more people will use Extra Credit and Weight than will use multiplicator and offset. In the full view, at least, this page should show any setting that may affect computation. The more subtle point in my last suggestion is whether a column should appear or be hidden depending on if an aggregation method someplace within a course is able to use the feature. It is a balance between three usability qualities: minimalism, discoverability, and completeness. Having tested the feature, I am of the view that columns should not be changing around as one tries different settings of aggregation methods. The reason is that, if one does want extra credit or weight (or any other feature), having the column always there means that one knows they are on the right page to determine if a feature is enabled or not. And I think that is more important than the screen space saved by not displaying the column. Since the cells will be empty when not used, I don't think it adds too much to the complexity of the page. In both cases I think it is fine to put these columns in the full view, which gives a very clean display for the most simple (and the default) gradebook which will be fine and less intimidating for first time teachers.
        Hide
        Nicolas Connault added a comment -

        I disagree with Gary on this one. Something that is not there (weight/extra credit) cannot be confusing. Besides, seeing a column with nothing in it is very likely to confuse people, especially if the column is called "Weight" and the aggregation method is "Simple weighted mean of grades".

        I'm happy with the conditional columns as they are.

        Show
        Nicolas Connault added a comment - I disagree with Gary on this one. Something that is not there (weight/extra credit) cannot be confusing. Besides, seeing a column with nothing in it is very likely to confuse people, especially if the column is called "Weight" and the aggregation method is "Simple weighted mean of grades". I'm happy with the conditional columns as they are.
        Hide
        Elena Ivanova added a comment -

        I agree with Gary.
        It is indeed confusing to have those columns to appear/disappear. Empty unused cells (if the aggregation type does not support such calculation method) are much less confusing.

        I also agree that more people will use Extra Credit and Weight rather than Multiplicator and Offset (I honestly do not get at the moment how they are supposed to work. Are they supposed to act like extra credits? Since I was actually expecting them to act like calculation mechanisms and increase the individual grade item AND category total).

        Show
        Elena Ivanova added a comment - I agree with Gary. It is indeed confusing to have those columns to appear/disappear. Empty unused cells (if the aggregation type does not support such calculation method) are much less confusing. I also agree that more people will use Extra Credit and Weight rather than Multiplicator and Offset (I honestly do not get at the moment how they are supposed to work. Are they supposed to act like extra credits? Since I was actually expecting them to act like calculation mechanisms and increase the individual grade item AND category total).
        Hide
        Petr Škoda added a comment -

        Please do not speak for the whole world again, some countries may not use extra credit and weights at all

        Show
        Petr Škoda added a comment - Please do not speak for the whole world again, some countries may not use extra credit and weights at all
        Hide
        Elena Ivanova added a comment -

        Petr, I was simply wondering how Multiplicator and Offset are supposed to work, plus I was expressing my own opinion on what people would want to use.
        And, honestly, I do not understand the whole deal with "you, in canada/us".
        Yes, there can be different needs depending on different ways of Moodle use, but they do not depend on what country we are from.

        Show
        Elena Ivanova added a comment - Petr, I was simply wondering how Multiplicator and Offset are supposed to work, plus I was expressing my own opinion on what people would want to use. And, honestly, I do not understand the whole deal with "you, in canada/us". Yes, there can be different needs depending on different ways of Moodle use, but they do not depend on what country we are from.
        Hide
        Petr Škoda added a comment -

        Multiplicator and offset were in the old 1.8.x gradebook because somebody implemented it that way. I hope they will be removed from 2.0 after we complete the transition of grading implementation from activities into gradebook plugins.

        All I say is I know many people here that do not use extra credit or weight at all. I understand that it may be needed in some other parts of the world. In this case I think it is a good compromise to show those columns only when appropriate - imagine admin disables all aggregation types with weights or extra credit, why should the column be there at all?

        To Elena: Do you know how is grading done in Europe? It sounds like previous gradebook was "perfect" for US/Canada, but for teachers in my country it was totally unusable. During the gradebook development we talked a lot how grading is done in several countries, unfortunately there was nobody from the US/Canada in the development team - all the feedback came much later after the database and UI freeze (very big thanks to Gary Several people from different parts of the world discussed what to do with those weights and extra credits and we all agreed that the autohiding of those columns is a good compromise - indeed it may not be the best solution for everybody (in this case for people that like to use extra credit and weights).

        In any case thanks for the all the feedback, it is really welcome. I personally hope we will all learn something new and in the end 1.9.5 or 1.9.6 will be much more suitable/usable for everybody (especially US/Canada)

        Show
        Petr Škoda added a comment - Multiplicator and offset were in the old 1.8.x gradebook because somebody implemented it that way. I hope they will be removed from 2.0 after we complete the transition of grading implementation from activities into gradebook plugins. All I say is I know many people here that do not use extra credit or weight at all. I understand that it may be needed in some other parts of the world. In this case I think it is a good compromise to show those columns only when appropriate - imagine admin disables all aggregation types with weights or extra credit, why should the column be there at all? To Elena: Do you know how is grading done in Europe? It sounds like previous gradebook was "perfect" for US/Canada, but for teachers in my country it was totally unusable. During the gradebook development we talked a lot how grading is done in several countries, unfortunately there was nobody from the US/Canada in the development team - all the feedback came much later after the database and UI freeze (very big thanks to Gary Several people from different parts of the world discussed what to do with those weights and extra credits and we all agreed that the autohiding of those columns is a good compromise - indeed it may not be the best solution for everybody (in this case for people that like to use extra credit and weights). In any case thanks for the all the feedback, it is really welcome. I personally hope we will all learn something new and in the end 1.9.5 or 1.9.6 will be much more suitable/usable for everybody (especially US/Canada)
        Hide
        Elena Ivanova added a comment -

        Thank you, Petr.

        And I am from Russia (does it considered to be Europe?) , where we have both concepts of extra credit and weights. Even though they are called and managed differently, they do exist.

        Show
        Elena Ivanova added a comment - Thank you, Petr. And I am from Russia (does it considered to be Europe?) , where we have both concepts of extra credit and weights. Even though they are called and managed differently, they do exist.
        Hide
        Petr Škoda added a comment -

        sure, it is also in Europe - that makes your votes more important, we did not get much feedback from Russia

        Show
        Petr Škoda added a comment - sure, it is also in Europe - that makes your votes more important, we did not get much feedback from Russia
        Hide
        Elena Ivanova added a comment -

        Nicolas already fixed the issue when students could see hidden grades/items and categories in the gradebook when "Show hidden items" is set to No. (Though there is a misalignment issue)

        I have just noticed some other things.

        1. Go to Settings > User Report > Show hidden items > Set to Yes.
        Students will now see the hidden grade item AND the grades. I believe they shouldn't see the actual grade, just an item.

        2. Go to Settings > User Report > Show hidden items > Set to Hidden until
        Hidden grade items will now be hidden completely from students, but items that are set to be Hidden until", will be displayed to a student grayed out with a grade assigned.

        3. Even though the item is hidden from the student view, it is still included into the aggregation, and such total is displayed on the User Report for a student.
        Regular 1.9.4 behaves differently (User report for a teacher shows total with hidden items being included in aggregation. User report for a student has total that does not include hidden items).

        Please let me know if screenshots are needed.
        Thank you!

        Show
        Elena Ivanova added a comment - Nicolas already fixed the issue when students could see hidden grades/items and categories in the gradebook when "Show hidden items" is set to No. (Though there is a misalignment issue) I have just noticed some other things. 1. Go to Settings > User Report > Show hidden items > Set to Yes. Students will now see the hidden grade item AND the grades. I believe they shouldn't see the actual grade, just an item. 2. Go to Settings > User Report > Show hidden items > Set to Hidden until Hidden grade items will now be hidden completely from students, but items that are set to be Hidden until", will be displayed to a student grayed out with a grade assigned. 3. Even though the item is hidden from the student view, it is still included into the aggregation, and such total is displayed on the User Report for a student. Regular 1.9.4 behaves differently (User report for a teacher shows total with hidden items being included in aggregation. User report for a student has total that does not include hidden items). Please let me know if screenshots are needed. Thank you!
        Hide
        Nicolas Connault added a comment -

        Elena,

        Many thanks for the bug reports and useful feedback. I've just fixed items 1 and 2 in your list. Item 3 needs an explanation:

        In 1.9.4, we were getting lots of people complaining that the course totals were inconsistent for students and teachers. It is clear that some people prefer to aggregate the hidden items, while others don't. Each approach has merits. Unfortunately we cannot implement both without adding a User Report Setting controlled by teachers/admins for the whole course. This requires a new database field for the user report, so we can't add it in 1.9+ and will have to wait for 2.0.

        In the meantime, we are aggregating hidden items for the user report. If you don't want your users to see how hidden items are affecting their course total, you could maybe hide the course total until all hidden items are shown, or put your hidden items in a category of their own, with a weight of 0.

        Show
        Nicolas Connault added a comment - Elena, Many thanks for the bug reports and useful feedback. I've just fixed items 1 and 2 in your list. Item 3 needs an explanation: In 1.9.4, we were getting lots of people complaining that the course totals were inconsistent for students and teachers. It is clear that some people prefer to aggregate the hidden items, while others don't. Each approach has merits. Unfortunately we cannot implement both without adding a User Report Setting controlled by teachers/admins for the whole course. This requires a new database field for the user report, so we can't add it in 1.9+ and will have to wait for 2.0. In the meantime, we are aggregating hidden items for the user report. If you don't want your users to see how hidden items are affecting their course total, you could maybe hide the course total until all hidden items are shown, or put your hidden items in a category of their own, with a weight of 0.
        Hide
        Nicolas Connault added a comment -

        Updated the patch once more

        Show
        Nicolas Connault added a comment - Updated the patch once more
        Hide
        Jesus Martin added a comment -

        Again getting an error at Categories and Items page: Fatal error: Cannot redeclare class grade_edit_tree_column_category in /var/www/moodle/grade/edit/tree/lib.php on line 1454

        I applied the latest patch: 15/Apr/09 09:43 PM

        Thanks. J. Martín

        Show
        Jesus Martin added a comment - Again getting an error at Categories and Items page: Fatal error: Cannot redeclare class grade_edit_tree_column_category in /var/www/moodle/grade/edit/tree/lib.php on line 1454 I applied the latest patch: 15/Apr/09 09:43 PM Thanks. J. Martín
        Hide
        Nicolas Connault added a comment -

        If you've already applied the patch on your code, you cannot apply it again, otherwise the new files (such as grade/edit/tree/lib.php) will append to themselves, duplicating most of their code. You must first revert the patch or do a fresh checkout of MOODLE_19_STABLE

        Show
        Nicolas Connault added a comment - If you've already applied the patch on your code, you cannot apply it again, otherwise the new files (such as grade/edit/tree/lib.php) will append to themselves, duplicating most of their code. You must first revert the patch or do a fresh checkout of MOODLE_19_STABLE
        Hide
        Jesus Martin added a comment -

        Hi again.
        First of all, I replaced all the files with the Moodle 1.9.4+ (built weekly) with cp -r * /var/www/moodle

        Then I applied the new patch.

        This is the code from 1.431 to 1.454 (/var/www/moodle/grade/edit/tree/lib.php):

        abstract class grade_edit_tree_column_category extends grade_edit_tree_column {

        public $forced;
        public $advanced;

        public function __construct($name) {
        global $CFG;
        $this->forced = (int)$CFG->

        {"grade_$name"."_flag"} & 1;
        $this->advanced = (int)$CFG->{"grade_$name"."_flag"}

        & 2;
        }

        public function is_hidden($mode='simple') {
        global $CFG;
        if ($mode == 'simple')

        { return $this->advanced; }

        elseif ($mode == 'advanced') {
        if ($this->forced && $CFG->grade_hideforcedsettings)

        { return true; }

        else

        { return false; }

        }
        }
        }

        Thanks again.

        Show
        Jesus Martin added a comment - Hi again. First of all, I replaced all the files with the Moodle 1.9.4+ (built weekly) with cp -r * /var/www/moodle Then I applied the new patch. This is the code from 1.431 to 1.454 (/var/www/moodle/grade/edit/tree/lib.php): abstract class grade_edit_tree_column_category extends grade_edit_tree_column { public $forced; public $advanced; public function __construct($name) { global $CFG; $this->forced = (int)$CFG-> {"grade_$name"."_flag"} & 1; $this->advanced = (int)$CFG->{"grade_$name"."_flag"} & 2; } public function is_hidden($mode='simple') { global $CFG; if ($mode == 'simple') { return $this->advanced; } elseif ($mode == 'advanced') { if ($this->forced && $CFG->grade_hideforcedsettings) { return true; } else { return false; } } } } Thanks again.
        Hide
        Elena Ivanova added a comment -

        Thank you, Nicolas.
        I agree that we want to make the views as close to each other as possible, so it seems like a good temp solution.

        Something weird is happening with hidden totals though. I was hiding/unhiding them to check what the student will see, and some totals became "stuck" to be hidden.

        1. Go to Cat&Items > Hide Total for a Sub-category (or a course). The whole category will become hidden there, plus from students on the User report. ( I believe that students still should the category with grades). However, if you will check the Grader report, it will still show items in this category as non-hidden=not gray.

        2. Now, on Cat&Items unhide the Sub-category back (Not the Total). Total for the category will also become unhidden. Or at least the icon will indicate so.
        Go to the Grader report. Notice that Total for that subcategory still looks hidden there.
        Click Turn Editing on. You will see that Total itself is indeed unhidden (eye open), but all individual totals below are hidden.
        It seems to me that this should not be happening, though I am not sure if there is an easy fix.

        Show
        Elena Ivanova added a comment - Thank you, Nicolas. I agree that we want to make the views as close to each other as possible, so it seems like a good temp solution. Something weird is happening with hidden totals though. I was hiding/unhiding them to check what the student will see, and some totals became "stuck" to be hidden. 1. Go to Cat&Items > Hide Total for a Sub-category (or a course). The whole category will become hidden there, plus from students on the User report. ( I believe that students still should the category with grades). However, if you will check the Grader report, it will still show items in this category as non-hidden=not gray. 2. Now, on Cat&Items unhide the Sub-category back (Not the Total). Total for the category will also become unhidden. Or at least the icon will indicate so. Go to the Grader report. Notice that Total for that subcategory still looks hidden there. Click Turn Editing on. You will see that Total itself is indeed unhidden (eye open), but all individual totals below are hidden. It seems to me that this should not be happening, though I am not sure if there is an easy fix.
        Hide
        Nicolas Connault added a comment -

        I just updated the patch again, because it contained PHP5-only keywords

        Show
        Nicolas Connault added a comment - I just updated the patch again, because it contained PHP5-only keywords
        Hide
        Nicolas Connault added a comment -

        Jesus,
        This file only has 965 lines. In addition to replacing existing files with those from 1.9.4, you must also DELETE manually the 4 files that are NEW in 1.9.5:

        • grade/edit/tree/lib.php
        • grade/edit/tree/functions.js
        • grade/edit/tree/tree.css
        • grade/report/styles.css
        Show
        Nicolas Connault added a comment - Jesus, This file only has 965 lines. In addition to replacing existing files with those from 1.9.4, you must also DELETE manually the 4 files that are NEW in 1.9.5: grade/edit/tree/lib.php grade/edit/tree/functions.js grade/edit/tree/tree.css grade/report/styles.css
        Hide
        Nicolas Connault added a comment - - edited

        Elena,

        All this is working fine, actually.

        A note of explanation: the hide/show icons have additional function than the Hidden setting. They also cascade the Hidden setting down to all children categories and items. For example:

        • When you click the hide icon of the course category, Everything becomes hidden in your course
        • When you click the hide icon of the course category's total, only the total and its grades become hidden. The category itself remains visible including all its children
        • When you change the course category's status to Hidden in the category edit form, no cascading occurs.

        Also important: categories do not have a hidden status, only their grade item does. The confusion here is that the eye icon represents a status for grade items and an action for categories. I'm not sure how to resolve that without using two different icons.

        Show
        Nicolas Connault added a comment - - edited Elena, All this is working fine, actually. A note of explanation: the hide/show icons have additional function than the Hidden setting. They also cascade the Hidden setting down to all children categories and items. For example: When you click the hide icon of the course category, Everything becomes hidden in your course When you click the hide icon of the course category's total, only the total and its grades become hidden. The category itself remains visible including all its children When you change the course category's status to Hidden in the category edit form, no cascading occurs. Also important: categories do not have a hidden status, only their grade item does. The confusion here is that the eye icon represents a status for grade items and an action for categories. I'm not sure how to resolve that without using two different icons.
        Hide
        Nicolas Connault added a comment - - edited

        I think a good solution would be to change the action of the category total's icon: make it hide all the grades, but not hide the item itself. The problem then becomes: what state do we show for the category total's hidden icon? It could potentially look hidden when in fact it is not. This will be subject to discussion for 2.0, or maybe 1.9.6.

        If you want to only hide the category total, but not the category itself:
        1. Click the hide icon of the category total
        2. Click the show icon of the category

        Show
        Nicolas Connault added a comment - - edited I think a good solution would be to change the action of the category total's icon: make it hide all the grades, but not hide the item itself. The problem then becomes: what state do we show for the category total's hidden icon? It could potentially look hidden when in fact it is not. This will be subject to discussion for 2.0, or maybe 1.9.6. If you want to only hide the category total, but not the category itself: 1. Click the hide icon of the category total 2. Click the show icon of the category
        Hide
        Nicolas Connault added a comment -

        I've committed all these improvements into HEAD

        Show
        Nicolas Connault added a comment - I've committed all these improvements into HEAD
        Hide
        Elena Ivanova added a comment -

        "When you click the hide icon of the course category's total, only the total and its grades become hidden. The category itself remains visible including all its children "

        It is not doing that in the current test environment. Please login as a student, and you will see that the whole category is gone from the user report

        Show
        Elena Ivanova added a comment - "When you click the hide icon of the course category's total, only the total and its grades become hidden. The category itself remains visible including all its children " It is not doing that in the current test environment. Please login as a student, and you will see that the whole category is gone from the user report
        Hide
        Elena Ivanova added a comment -

        And, yes, I was indeed expecting that if Cat&Items interface shows Category Total as Unhidden, then the Grader/User reports will reflect that and all total grades will be unhidden too.
        Since it is not the case now, it looks like a bug.

        I am a little bit lost with the last suggestion

        "category total's icon: make it hide all the grades, but not hide the item itself."
        Do you mean that we will show words and column "category total", but still hide the grades inside from students? If so, I think the icon should look like a closed eye, and words "Category total" should be grayed out on the User report.
        Though, yes, the behavior would be different from regular gradable items. Should we create a separate control: "Show hidden Totals: Only Hidden until"?

        Show
        Elena Ivanova added a comment - And, yes, I was indeed expecting that if Cat&Items interface shows Category Total as Unhidden, then the Grader/User reports will reflect that and all total grades will be unhidden too. Since it is not the case now, it looks like a bug. I am a little bit lost with the last suggestion "category total's icon: make it hide all the grades, but not hide the item itself." Do you mean that we will show words and column "category total", but still hide the grades inside from students? If so, I think the icon should look like a closed eye, and words "Category total" should be grayed out on the User report. Though, yes, the behavior would be different from regular gradable items. Should we create a separate control: "Show hidden Totals: Only Hidden until"?
        Hide
        Nicolas Connault added a comment -

        We should move this discussion to a different tracker issue. This will NOT get resolved in 1.9.5.

        Show
        Nicolas Connault added a comment - We should move this discussion to a different tracker issue. This will NOT get resolved in 1.9.5.
        Hide
        Jesus Martin added a comment -

        Thanks for your help with the patch, Nicolas.

        In the outcomes report, the activity column shows the complete name of the outcome, not the name of the activity.

        J. Martin

        Show
        Jesus Martin added a comment - Thanks for your help with the patch, Nicolas. In the outcomes report, the activity column shows the complete name of the outcome, not the name of the activity. J. Martin
        Hide
        Elena Ivanova added a comment -

        Hi,
        I am sorry to re-post. But since we had a lot of items in this ticket, I was a little worried that my last comment on the bug can be lost. %)
        Basically, once you hide category's total, the whole category becomes hidden for a student (not just the total and its grades). As far as I can tell from the note above this is not the intended behavior.
        Screenshot (1 part: settings; 2-instructor view of the user report; 3 - student view of the user report with missing Category 1):
        http://umn.edu/moodle/samples/hiddenornot.png

        Sorry, it if already on the to-do list, or if it is for the next version to fix.

        Show
        Elena Ivanova added a comment - Hi, I am sorry to re-post. But since we had a lot of items in this ticket, I was a little worried that my last comment on the bug can be lost. %) Basically, once you hide category's total, the whole category becomes hidden for a student (not just the total and its grades). As far as I can tell from the note above this is not the intended behavior. Screenshot (1 part: settings; 2-instructor view of the user report; 3 - student view of the user report with missing Category 1): http://umn.edu/moodle/samples/hiddenornot.png Sorry, it if already on the to-do list, or if it is for the next version to fix.
        Hide
        Nicolas Connault added a comment -

        Gradebook patch applied, resolving this issue.

        Show
        Nicolas Connault added a comment - Gradebook patch applied, resolving this issue.
        Hide
        Petr Škoda added a comment -

        reopening, going to add more fixes over the weekend

        Show
        Petr Škoda added a comment - reopening, going to add more fixes over the weekend
        Hide
        Vlas Voloshin added a comment -

        Hello! Recently I've updated CVS on my test moodle installation and came upon the new grader report interface - it's nice, but there's a problem: one of our courses has about 60 grade items, some have even more. Currently the page seems to run a JS to create a tooltip for every grade table cell: even if I restrict user per page count to 20, for example, it's still 1200 tooltips in the array. This causes browser to hang up for about a minute to create all tooltips after loading the page. If there's no performance solution for this, I suggest adding a setting to disable generating tooltips in grader report would be reasonable.

        Show
        Vlas Voloshin added a comment - Hello! Recently I've updated CVS on my test moodle installation and came upon the new grader report interface - it's nice, but there's a problem: one of our courses has about 60 grade items, some have even more. Currently the page seems to run a JS to create a tooltip for every grade table cell: even if I restrict user per page count to 20, for example, it's still 1200 tooltips in the array. This causes browser to hang up for about a minute to create all tooltips after loading the page. If there's no performance solution for this, I suggest adding a setting to disable generating tooltips in grader report would be reasonable.
        Hide
        Petr Škoda added a comment -

        hmm, alternative solution could be to attach the tooltips later when page is fully loaded, right?

        Show
        Petr Škoda added a comment - hmm, alternative solution could be to attach the tooltips later when page is fully loaded, right?
        Hide
        Nicolas Connault added a comment -

        Yes Petr, I propose to use AJAX to load the tooltip on-the-fly (keeping them cached).

        Show
        Nicolas Connault added a comment - Yes Petr, I propose to use AJAX to load the tooltip on-the-fly (keeping them cached).
        Hide
        Robert Russo added a comment -

        We have the same issue as Vlas. We have some courses with over 800 students and 20+ grade items which kills every browser we've tried it on.

        The AJAX solution would probably be the best solution for the problem at hand.

        Show
        Robert Russo added a comment - We have the same issue as Vlas. We have some courses with over 800 students and 20+ grade items which kills every browser we've tried it on. The AJAX solution would probably be the best solution for the problem at hand.
        Hide
        Robert Russo added a comment -

        I should add, that some instructors like to view those pages with 100 students per page, or alternatively, all members of a group, which can be large as well. Definitely the AJAX solution is preferred in case someone has 60 items and 800 students and decides to show all of them.

        Show
        Robert Russo added a comment - I should add, that some instructors like to view those pages with 100 students per page, or alternatively, all members of a group, which can be large as well. Definitely the AJAX solution is preferred in case someone has 60 items and 800 students and decides to show all of them.
        Hide
        Matt Gibson added a comment -

        I think this can be fixed by using event bubbling and a single listener on the root element.

        Show
        Matt Gibson added a comment - I think this can be fixed by using event bubbling and a single listener on the root element.
        Hide
        Matt Gibson added a comment -

        Sorry this isn't a proper patch - I've not worked out how to get NetBeans to do one on Windows yet (any tips appreciated)

        Replacing index.php and styles.php with these ones switches the tooltips to a single listener on the user-grades table, so there's no looping when the page loads. I've also made it use YUI panel, not YUI tooltip so that very long tooltips can now be read even if they go off the page (MDL-13433, linked). You can also move your mouse over them and they are not fixed width, so long assignment titles are not poking out. This was causing me a problem because tooltips for items with long feedback or with comment inline enabled were disappearing as soon as they appeared and couldn't be read.

        I've noticed two remaining issues - with no javascript, the tooltips are messy as they are full of HTML tags, so not very accessible (can't think of an easy fix for that) and also feedback with tags like br ul and li seem to have those stripped out, which is a shame as some of the comments I'm using rely on those heavily and the YUI panel can deal with them fine.

        Show
        Matt Gibson added a comment - Sorry this isn't a proper patch - I've not worked out how to get NetBeans to do one on Windows yet (any tips appreciated) Replacing index.php and styles.php with these ones switches the tooltips to a single listener on the user-grades table, so there's no looping when the page loads. I've also made it use YUI panel, not YUI tooltip so that very long tooltips can now be read even if they go off the page ( MDL-13433 , linked). You can also move your mouse over them and they are not fixed width, so long assignment titles are not poking out. This was causing me a problem because tooltips for items with long feedback or with comment inline enabled were disappearing as soon as they appeared and couldn't be read. I've noticed two remaining issues - with no javascript, the tooltips are messy as they are full of HTML tags, so not very accessible (can't think of an easy fix for that) and also feedback with tags like br ul and li seem to have those stripped out, which is a shame as some of the comments I'm using rely on those heavily and the YUI panel can deal with them fine.
        Hide
        Nicolas Connault added a comment -

        Matt, thank you heaps for this patch. I've tried it and it works wonders. I'm creating a new subtask for these tooltips (MDL-18963) so that we don't pollute this meta-issue with it.

        Show
        Nicolas Connault added a comment - Matt, thank you heaps for this patch. I've tried it and it works wonders. I'm creating a new subtask for these tooltips ( MDL-18963 ) so that we don't pollute this meta-issue with it.
        Hide
        Petr Škoda added a comment -

        should be done now, please file regressions as separate issues
        thanks for testing

        Show
        Petr Škoda added a comment - should be done now, please file regressions as separate issues thanks for testing
        Hide
        Elena Ivanova added a comment -

        btw, gradebook on http://test.moodle.org/1.9 gives the error: "XML Parsing Error: not well-formed"

        Show
        Elena Ivanova added a comment - btw, gradebook on http://test.moodle.org/1.9 gives the error: "XML Parsing Error: not well-formed"
        Hide
        Helen Foster added a comment -

        Thanks Elena, the XML parsing error is reported in MDL-19029.

        Show
        Helen Foster added a comment - Thanks Elena, the XML parsing error is reported in MDL-19029 .
        Hide
        Petr Škoda added a comment -

        updating test server

        Show
        Petr Škoda added a comment - updating test server
        Hide
        Petr Škoda added a comment -

        upgrade failed :-D reinstalling ...

        Show
        Petr Škoda added a comment - upgrade failed :-D reinstalling ...
        Hide
        Paul Taylor added a comment -

        I have several sites that are running Moodle on Windows servers (some IIS, some Apache) and they are seeing the gradebook issue where the comment box is full of html code. I don't see it on any Linux based sites. I assume it is fixed for Linux, but what extra needs to be done for Windows based servers? Happy for login access to see this.

        Show
        Paul Taylor added a comment - I have several sites that are running Moodle on Windows servers (some IIS, some Apache) and they are seeing the gradebook issue where the comment box is full of html code. I don't see it on any Linux based sites. I assume it is fixed for Linux, but what extra needs to be done for Windows based servers? Happy for login access to see this.
        Hide
        Paul Taylor added a comment -

        HTML code showing in feedback cells only on Windows based sites.

        Show
        Paul Taylor added a comment - HTML code showing in feedback cells only on Windows based sites.
        Hide
        Helen Foster added a comment -

        Paul, as this issue has been marked resolved, please could you report the problem of html code in the comment box as a new issue.

        Show
        Helen Foster added a comment - Paul, as this issue has been marked resolved, please could you report the problem of html code in the comment box as a new issue.
        Hide
        Rossiani Wijaya added a comment -

        Resolved. Closing

        Show
        Rossiani Wijaya added a comment - Resolved. Closing

          People

          • Votes:
            2 Vote for this issue
            Watchers:
            17 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: