Adding annotation doesn't alert if you change page without saving - leaves empty block on the page

Bug #1929357 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Medium
Dianne Tennent

Bug Description

After clicking the dot on the smart evidence page to display the annotation modal you decide to reload the page before saving the form or exiting out with the 'X' button the empty annotation block still exists on the corresponding page.

When you do this multiple times from the smart evidence page, multiple annotations with the same evaluation item are created on the corresponding page.

There needs to be form change checker alert if we are adding the block (but not when editing) to alert one when leaving the page. And on leaving to clean up the half finished block.

Tags: bite-sized
Robert Lyon (robertl-9)
Changed in mahara:
status: New → Confirmed
importance: Undecided → Medium
tags: added: bite-sized
Changed in mahara:
status: Confirmed → In Progress
assignee: nobody → Dianne Tennent (dianne-t)
Revision history for this message
Dianne Tennent (dianne-t) wrote (last edit ):

Steps to reproduce:
- Learner makes a copy of SmartEvidence template made available by their institution
- Learner creates a page and adds this to the template collection
- Admin to ensure the blocktype/annotation plugin is enabled
- Reviewer or learner makes annotation on matrix page for one of the criteria
- Reload page before saving or pressing the X on the annotation modal
- Check newly added page

Expected result - no annotations appear on this page

Actual result - Annotation heading appears on this page

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/12044

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "main" branch: https://reviews.mahara.org/12124

Changed in mahara:
milestone: none → 22.04.0
Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "main" branch: https://reviews.mahara.org/12175

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/12124
Committed: https://git.mahara.org/mahara/mahara/commit/69306ce352c131f79c7bb54ccc9dd504da8cd883
Submitter: Robert Lyon (<email address hidden>)
Branch: main

commit 69306ce352c131f79c7bb54ccc9dd504da8cd883
Author: Dianne Tennent <email address hidden>
Date: Wed Sep 29 16:56:19 2021 +1300

Bug 1929357: Do not create block when leaving annotation

If people close the annotation edit window or leave
without saving, no block will be created.

Change-Id: Ic37c42bc628fa5fb8f027efa975f321ba6ca6e99

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/c/mahara/+/12585
Committed: https://git.mahara.org/mahara/mahara/commit/707bba07b1ca1290e6d1ba0ad53549d853333a26
Submitter: "Robert Lyon <email address hidden>"
Branch: main

commit 707bba07b1ca1290e6d1ba0ad53549d853333a26
Author: Robert Lyon <email address hidden>
Date: Thu Apr 7 14:43:13 2022 +1200

Bug 1929357 - adding back in the hide dock

So that when others are giving feedback the dock closes

Change-Id: I71c9f36784ad6b42cf0fd2bcf6fc3e1c212953e3
Signed-off-by: Robert Lyon <email address hidden>

Gold (gold.catalyst)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.