Bug #19286

unnecessary queries for updating session data

Added by Sebastian Mendel about 12 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2008-09-04
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
4.2
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

currently updating the session data is done in two steps:

- delete session data
- insert new session data

this means two queries and two times updating the index

but with only one update query, there is no need for updating the index

is there any special reason why TYPO3 uses two queries to accomplish this?

(issue imported from #M9287)


Files

class.tslib_feuserauth.php.diff (1.5 KB) class.tslib_feuserauth.php.diff Administrator Admin, 2008-09-19 13:20
class.tslib_feuserauth2.php.diff (1.54 KB) class.tslib_feuserauth2.php.diff Administrator Admin, 2008-09-19 14:55

Related issues

Related to TYPO3 Core - Bug #63922: Correct handling of session data update in TYPO3 CMS 4.5 LTSRejected2014-12-16

Actions
#1

Updated by Netresearch GmbH & Co KG about 12 years ago

This patch resolves the problem, as we have problems on one machine with the mysql DB. It stops sometimes between the delete and insert statement, so sometimes we have no session data on a new request.

#2

Updated by Netresearch GmbH & Co KG about 12 years ago

corrected version, there was one line missed

#3

Updated by Mr. Hudson over 9 years ago

Patch set 1 of change I58624215b65bced7473923eb6ce1966650c02860 has been pushed to the review server.
It is available at http://review.typo3.org/2761

#4

Updated by Mr. Hudson over 9 years ago

Patch set 2 of change I58624215b65bced7473923eb6ce1966650c02860 has been pushed to the review server.
It is available at http://review.typo3.org/2761

#5

Updated by Mr. Hudson over 9 years ago

Patch set 3 of change I58624215b65bced7473923eb6ce1966650c02860 has been pushed to the review server.
It is available at http://review.typo3.org/2761

#6

Updated by Mr. Hudson over 9 years ago

Patch set 4 of change I58624215b65bced7473923eb6ce1966650c02860 has been pushed to the review server.
It is available at http://review.typo3.org/2761

#7

Updated by Mr. Hudson over 9 years ago

Patch set 5 of change I58624215b65bced7473923eb6ce1966650c02860 has been pushed to the review server.
It is available at http://review.typo3.org/2761

#8

Updated by Mr. Hudson over 9 years ago

Patch set 6 of change I58624215b65bced7473923eb6ce1966650c02860 has been pushed to the review server.
It is available at http://review.typo3.org/2761

#9

Updated by Stefan Neufeind over 9 years ago

  • Status changed from New to Resolved
  • % Done changed from 0 to 100
#10

Updated by Benni Mack about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF