Project

General

Profile

Actions

Bug #9789

open

Record Lock: release to early on 'leave page'

Added by Carsten Rose over 4 years ago. Updated 3 months ago.

Status:
New
Priority:
Normal
Assignee:
Carsten Rose
Target version:
Start date:
17.12.2019
Due date:
% Done:

100%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

Description

  • Form oeffnen
  • Record veraendern - lock wird angefordert: ok
  • Auf irgendeinen Link klicken (Form verlassen)
  • Sofort wird ein 'Release Lock' gesendet (und ausgefuehrt), obwohl der User via Dialog gefragt wird ob er die Daten verlieren moechte: BAD
  • Anschliessend arbeitet das Form ohne Lock weiter - das ist nicht gut!
  • Min. in der Version 19.7.0 war das Verhalten noch ok (w16.math.uzh.ch/ort)
  • Die Screenshots zeigen das Verhalten bei 19.7.0 und 19.12.0. In der 19.7.0 ist der Dialog 'leave site' offen, ohne das 'dirty release' getriggert wurde, bei 19.12.0 wurde dirty release faelschlicherweise getriggert.

Files

lock-fail.png View lock-fail.png 204 KB Carsten Rose, 17.12.2019 20:56
lock-ok.png View lock-ok.png 375 KB Carsten Rose, 17.12.2019 20:56
RecordLock TestsExcel.xlsx RecordLock TestsExcel.xlsx 6.3 KB Enis Nuredini, 24.12.2021 16:03

Related issues

Related to QFQ - Bug #10081: Stale record lock after 'forbidden' characterNewCarsten Rose10.02.2020

Actions
Related to QFQ - Bug #9173: Stale Record Lock: Firefox NewCarsten Rose18.09.2019

Actions
Related to QFQ - Feature #8702: Load Record which is locked: missing user infoClosedCarsten Rose09.07.2019

Actions
Actions #1

Updated by Benjamin Baer over 4 years ago

  • Status changed from New to Priorize
Actions #2

Updated by Benjamin Baer over 4 years ago

  • Tracker changed from Support to Bug
Actions #3

Updated by Benjamin Baer over 4 years ago

  • Status changed from Priorize to Closed
  • % Done changed from 0 to 100
Actions #4

Updated by Benjamin Baer over 4 years ago

Problem:

Events:
beforeUnload ist zu frueh
unload ist zu spaet und wird nicht von jedem browser beachtet
pagehide ist halb zu spaet - funktioniert generell ausser das man 2 mal die seite reloaden muss

Es ist nicht moeglich beforeUnload dialog antworten abzufangen, dh. ich kann es nicht praeventativ entfernen und wieder ein neues Lock erstellen falls der User cancel / no drueckt.

Also scheint pagehide unsere beste option zu sein.

Im branch: b9789-earlyRelease

Actions #5

Updated by Benjamin Baer over 4 years ago

  • Status changed from Closed to Priorize
Actions #6

Updated by Benjamin Baer over 4 years ago

  • Status changed from Priorize to In Progress
Actions #7

Updated by Benjamin Baer over 4 years ago

  • Status changed from In Progress to Closed

Im Branch wurde jetzt der Release Lock bei der "Seite Verlassen" anfrage wieder eingefuegt.

  • PageHide war auch zu spaet bei Firefox
  • Funktioniert im Chrome

Alternative Idee: Erstellen eines halb releases den wir beim Seiten Verlassen absenden - wurde aber wieder verworfen

Mit CO besprochen:
  • Wichtiger als ein faelschlich verlorenes Lock ist ein faelschlich behaltenes. (Faellt dem User schneller auf)
  • Lock wird wieder angefragt wenn er tippt
  • Keinen grossen Vorteil stur auf dem Lock zu bleiben - vorallem da dies nur auftritt wenn er versucht die Seite zu verlassen
Actions #8

Updated by Carsten Rose about 4 years ago

  • Status changed from Closed to New
  • Assignee changed from Benjamin Baer to Carsten Rose
  • Target version set to next5
Actions #9

Updated by Carsten Rose about 4 years ago

  • Status changed from New to Priorize
Actions #10

Updated by Carsten Rose almost 3 years ago

  • Target version changed from next5 to next4
Actions #11

Updated by Carsten Rose almost 3 years ago

  • Target version changed from next4 to next3
Actions #12

Updated by Carsten Rose almost 3 years ago

  • Priority changed from High to Normal
Actions #13

Updated by Carsten Rose over 2 years ago

  • Assignee changed from Carsten Rose to Enis Nuredini
  • Priority changed from Normal to High

Details mit CR besprechen

Actions #14

Updated by Carsten Rose over 2 years ago

  • Target version changed from next3 to next2
Actions #15

Updated by Carsten Rose over 2 years ago

  • Target version changed from next2 to 355
Actions #16

Updated by Enis Nuredini over 2 years ago

  • Status changed from Priorize to New
Actions #17

Updated by Enis Nuredini over 2 years ago

  • Status changed from New to Priorize
Actions #18

Updated by Enis Nuredini over 2 years ago

  • Status changed from Priorize to ToDo
Actions #19

Updated by Enis Nuredini over 2 years ago

  • Status changed from ToDo to Priorize
Actions #20

Updated by Enis Nuredini over 2 years ago

  • Status changed from Priorize to ToDo
Actions #21

Updated by Carsten Rose over 2 years ago

  • Related to Bug #10081: Stale record lock after 'forbidden' character added
Actions #22

Updated by Carsten Rose over 2 years ago

  • Related to Bug #9173: Stale Record Lock: Firefox added
Actions #23

Updated by Carsten Rose over 2 years ago

  • Related to Feature #8702: Load Record which is locked: missing user info added
Actions #24

Updated by Enis Nuredini over 2 years ago

Aktuelle Erkenntnisse:

Fehler trifft unter Linux in Firefox zufällig auf und nicht immer regelmässig. Unter Mac OS ebenfalls in Firefox tritt der Fehler sehr häufig bis immer auf. Bei Chrome tritt der Fehler nie auf, egal in welchem OS. Eine Tabelle mit allen Tests wurde erstellt.

Folgende Abläufe konnten zu dem Record Lock Fehler führen:
1. Edit > Chars im Textfeld löschen > Close > no save
2. Edit > Chars im Textfeld löschen > Save > Close >Edit >Änderungen im Textfeld > Close > no save

Wie schon erwähnt trifft der Fehler nicht immer unter der exakt gleichen Eingabe auf.

Die Excel Test Tabelle ist angehängt.

Actions #25

Updated by Carsten Rose over 2 years ago

Am besten mit Benj besprechen.

Actions #26

Updated by Carsten Rose over 1 year ago

  • Target version changed from 355 to Check if 'high' is still necessary
Actions #27

Updated by Carsten Rose 7 months ago

  • Priority changed from High to Normal
  • Target version changed from Check if 'high' is still necessary to CodingWeek2023
  • Prio Planung set to No
Actions #28

Updated by Carsten Rose 6 months ago

  • Status changed from In Progress to New
Actions #29

Updated by Carsten Rose 3 months ago

  • Target version changed from CodingWeek2023 to 24.4.0
Actions

Also available in: Atom PDF