sakai icon indicating copy to clipboard operation
sakai copied to clipboard

SAK-47669 - Rubric should only be locked if the association is active

Open maurercw opened this issue 2 years ago • 4 comments

maurercw avatar Sep 01 '22 02:09 maurercw

I don't know if this is right. If an instructor accidentally switches an association to another rubric and wants to switch back, what happens when the original rubric has been deleted? That original rubric may even have evaluations associated with it. The reason the active flag was put in was to handle that case and help prevent student evaluations being lost. So, if this needs to happen, you should probably check that you can't deleted a rubric with evaluations. That way at least any student evaluations are protected.

adrianfish avatar Sep 01 '22 09:09 adrianfish

I was just going off of what the jira was asking for! But it sounds like the criteria for if a rubric is locked is more complicated than that. Is there a person or document somewhere that is the authoritative source? It sounds like the current criteria is incomplete though.

maurercw avatar Sep 01 '22 12:09 maurercw

@maurercw No probs! I just know a lot about rubrics, for my sins. It seems to me that if we want to unlock a rubric that isn't part of an active association, then we just need to make sure it hasn't got any evaluations against the inactive tool associations. Then we unlock it and it can be deleted or whatever. But, we then need to make sure that when the rubrics is deleted all the inactive associations are removed too.

adrianfish avatar Sep 01 '22 12:09 adrianfish

@adrianfish Finally coming back to this! So, because everything is separate, how is a Rubric supposed to know if there are any evaluations? It seems like the objects are related, but have no direct linkage. So I can't go from Rubric -> ToolItemRubricAssociation -> Evaluation. But since the locked status is really part of the Rubric, I can't make a service call to find the evaluations.

maurercw avatar Sep 26 '22 23:09 maurercw

Closing this as this is likely not the appropriate fix.

ern avatar Nov 09 '23 15:11 ern