Bug #20932

Backend record locking should be on session basis

Added by Steffen Gebert over 9 years ago. Updated about 1 year ago.

Status:
Under Review
Priority:
Should have
Assignee:
Category:
Locking / Session Handling
Target version:
-
Start date:
2009-08-26
Due date:
% Done:

0%

TYPO3 Version:
4.3
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

If a user opens a record, a lock is saved in sys_lockedrecords.

The opendocs module saves document in BE session (so different browsers -> different open documents). Furthermore, opendocs doesn't update sys_lockedrecords, if a document is closed.

Attached is a part of the needed patch. As I think it's too late for 4.3, I only want to document this here.

What still has to be considered:
  • What should be done, if a user creates more locks on the same document? Should then at releasing one lock all other locks released or only the one of the current session? Latter would cause confusion, if e.g. a web browser crashes and then user opens the record again. The first lock would remain for 2 hours.
  • Sure sth. more...
    (issue imported from #M11796)

opendocs-locked-records.patch View (3.9 KB) Administrator Admin, 2009-08-26 23:41

History

#1 Updated by Alexander Opitz over 5 years ago

  • Status changed from New to Needs Feedback
  • Target version deleted (0)

The issue is very old, does this issue exists in newer versions of TYPO3 CMS (4.5 or 6.1)?

#2 Updated by Steffen Gebert over 5 years ago

Yes, this issue still exists.

#3 Updated by Alexander Opitz over 5 years ago

Can you provide a new patch?

#4 Updated by Steffen Gebert over 5 years ago

Hi Alexander,

thanks a lot for looking at the old issues! I appreciate it!

Unfortunately I cannot provide an updated patch as my interested shifted away from TYPO3 CMS Core.

I would be happy if one could pick up this topic.

Yours
Steffen

#5 Updated by Alexander Opitz over 5 years ago

  • Status changed from Needs Feedback to New

#6 Updated by Mathias Schreiber almost 4 years ago

  • Target version set to 7.4 (Backend)
  • Is Regression set to No

#7 Updated by Susanne Moog over 3 years ago

  • Target version changed from 7.4 (Backend) to 7.5

#8 Updated by Benni Mack about 3 years ago

  • Target version changed from 7.5 to 7 LTS

#9 Updated by Mathias Schreiber about 3 years ago

  • Target version deleted (7 LTS)

#10 Updated by Markus Sommer almost 2 years ago

  • Assignee set to Markus Sommer

#11 Updated by Gerrit Code Review almost 2 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51448

#12 Updated by Gerrit Code Review almost 2 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51448

#13 Updated by Gerrit Code Review almost 2 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51448

#14 Updated by Gerrit Code Review almost 2 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51448

#15 Updated by Gerrit Code Review almost 2 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/51448

#16 Updated by Susanne Moog about 1 year ago

  • Category set to Locking / Session Handling

Also available in: Atom PDF