Contrib: Gradebook2
  1. Contrib: Gradebook2
  2. GRBK-1005

Standard Category, changed to Extra Credit Category, cannot be changed back to not Extra Credit after edit of any other part of structure

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.5.0-b11
    • Fix Version/s: 1.5.0-b12, 1.5.0
    • Component/s: Services
    • Labels:
      None
    • Target Version:
    • QA Status:
      PASS
    • Previous Issue Keys:

      Description

      in GB (categories or weighted categories mode) with a standard category (extra credit flag not set) with items
      -- set the category to be extra credit, then save/close
      -- re-edit the category
      ==== result: extra credit setting can be changed (set to not extra credit)
      -- re-edit the category and set it to be extra credit, then save/close
      -- select edit on some other category or item in the gradebook, then close with no change (or a change, it doesn't matter)
      -- select edit on the original category (which was set to extra credit)
      ==== result: can no longer modify the extra credit setting -- grayed out.

      There is no way to make a category not be extra credit once something else has been edited. It must be deleted and re-added to get back to not extra credit status. In some cases, you can't delete the category without first deleting previously un-included or extracredit items have different weights/points from the rest of the items, and drop lowest was set.

        Activity

        Hide
        Joyce Johnstone added a comment - - edited
        This also happened to my category as well. It seems like this should be fixed for v1.5.

        Joyce
        Show
        Joyce Johnstone added a comment - - edited This also happened to my category as well. It seems like this should be fixed for v1.5. Joyce
        Hide
        Thomas Amsler added a comment -
        Show
        Thomas Amsler added a comment - UC Davis code review : https://crucible.ucdavis.edu/cru/CR-SAK-111
        Hide
        Michael Wenk added a comment -
        Testing on firefox 4.0.1 on linux on site 31070c6c-d3ec-4091-8998-53d9d054642b

        - Created category Cat 1 with 100 weight
        - Created item Item 1 in Cat 1 with 48 weight
        - Created item Item 2 in Cat 1 with 52 weight
        - Set Cat 1 to EC.
        - Cleared EC from Cat 1
        - Set Cat 1 to EC
        - Changed points from Item 2 to 45 pts
        - Cleared EC from Cat 1

        Pass
        Show
        Michael Wenk added a comment - Testing on firefox 4.0.1 on linux on site 31070c6c-d3ec-4091-8998-53d9d054642b - Created category Cat 1 with 100 weight - Created item Item 1 in Cat 1 with 48 weight - Created item Item 2 in Cat 1 with 52 weight - Set Cat 1 to EC. - Cleared EC from Cat 1 - Set Cat 1 to EC - Changed points from Item 2 to 45 pts - Cleared EC from Cat 1 Pass
        Hide
        Fernando Socorro added a comment - - edited
        Process:
        1. Dbl click on reg cat
        2. Change it to XCr
        3. Save/Close
        4. Dbl click on the same cat, now XCr
        5. Uncheck XCr
        6. Save/Close

        W7
        FF 3.6.14
        WMS 050 001-006 SQ 2011: cats PASS
        NUT 011 001-010 SQ 2011: wt cats FAIL: cat XCr 2, which was created and had the grayed out checkbox issue before the fix, still cannot be changed to no xcr

        IE 9.0.8122
        WMS 050 001-006 SQ 2011: cats PASS
        NUT 011 001-010 SQ 2011: wt cats FAIL: cat XCr 2, which was created and had the grayed out checkbox issue before the fix, still cannot be changed to no xcr

        Mac 10.6
        Safari 5.0.5
        WMS 050 001-006 SQ 2011: cats PASS
        NUT 011 001-010 SQ 2011: wt cats FAIL: cat XCr 2, which was created and had the grayed out checkbox issue before the fix, still cannot be changed to no xcr

        FF 4.0.1
        Same as above

        So
        Tentaive PASS.
        Show
        Fernando Socorro added a comment - - edited Process: 1. Dbl click on reg cat 2. Change it to XCr 3. Save/Close 4. Dbl click on the same cat, now XCr 5. Uncheck XCr 6. Save/Close W7 FF 3.6.14 WMS 050 001-006 SQ 2011: cats PASS NUT 011 001-010 SQ 2011: wt cats FAIL: cat XCr 2, which was created and had the grayed out checkbox issue before the fix, still cannot be changed to no xcr IE 9.0.8122 WMS 050 001-006 SQ 2011: cats PASS NUT 011 001-010 SQ 2011: wt cats FAIL: cat XCr 2, which was created and had the grayed out checkbox issue before the fix, still cannot be changed to no xcr Mac 10.6 Safari 5.0.5 WMS 050 001-006 SQ 2011: cats PASS NUT 011 001-010 SQ 2011: wt cats FAIL: cat XCr 2, which was created and had the grayed out checkbox issue before the fix, still cannot be changed to no xcr FF 4.0.1 Same as above So Tentaive PASS.
        Hide
        Joyce Johnstone added a comment -
        Fernando, was this tested with with new sites? This would continue to be a problem old sites with categories with grayed out exc
        Show
        Joyce Johnstone added a comment - Fernando, was this tested with with new sites? This would continue to be a problem old sites with categories with grayed out exc
        Hide
        Thomas Amsler added a comment -
        If this is really an old site problem, it won't affect current production because this error was introduced in 1.5.0 as part of GRBK-833.
        Show
        Thomas Amsler added a comment - If this is really an old site problem, it won't affect current production because this error was introduced in 1.5.0 as part of GRBK-833 .
        Hide
        Thomas Amsler added a comment -
        I just checked NUT 011 001-010 SQ 2011 and I don't see the grayed out checkbox on any of the categories. I wonder if this is a browser caching issue?
        Show
        Thomas Amsler added a comment - I just checked NUT 011 001-010 SQ 2011 and I don't see the grayed out checkbox on any of the categories. I wonder if this is a browser caching issue?
        Hide
        Fernando Socorro added a comment -
        @Thomas: The only item that had the grayed out checkbox issue still was XCr 2. I saw it in several browser, Mac & W7. But it could be.

        @Joyce: as per our conversation, I am changing to PASS since it is only a cat that was affected during the period of time this issue was in effect.
        PASS
        Show
        Fernando Socorro added a comment - @Thomas: The only item that had the grayed out checkbox issue still was XCr 2. I saw it in several browser, Mac & W7. But it could be. @Joyce: as per our conversation, I am changing to PASS since it is only a cat that was affected during the period of time this issue was in effect. PASS

          People

          • Assignee:
            Unassigned
            Reporter:
            David L. Woods
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: