Bug #20932

Backend record locking should be on session basis

Added by Steffen Gebert over 9 years ago. Updated over 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 almost 6 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 almost 6 years ago

Yes, this issue still exists.

#3 Updated by Alexander Opitz almost 6 years ago

Can you provide a new patch?

#4 Updated by Steffen Gebert almost 6 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 almost 6 years ago

  • Status changed from Needs Feedback to New

#6 Updated by Mathias Schreiber about 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 over 3 years ago

  • Target version changed from 7.5 to 7 LTS

#9 Updated by Mathias Schreiber over 3 years ago

  • Target version deleted (7 LTS)

#10 Updated by Markus Sommer about 2 years ago

  • Assignee set to Markus Sommer

#11 Updated by Gerrit Code Review about 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 about 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 about 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 about 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 about 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 over 1 year ago

  • Category set to Locking / Session Handling

Also available in: Atom PDF