Project

General

Profile

Actions
Copy link

Bug #39248

closed

fetchUserRecord don't work without username

Added by Robert Heel over 11 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
-
Target version:
-
Start date:
2012-07-25
Due date:
% Done:

100%

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

Description

e.g. fetchUserRecord($dbUser, false, 'fe_users.uid=1') don't work because $usernameClause is empty and is followed by " AND" in the sql query.

It should be set to "1=1", or $extraWhere should moved after $usernameClause.

Patch attached.

Files

fetchUserRecord_wo_username.diff (717 Bytes) fetchUserRecord_wo_username.diff Robert Heel, 2012-07-25 13:20
Actions
Copy link
 #1

Updated by Markus Klein over 11 years ago

Hi Robert!

1=1 is the only right solution, as $extraWhere is required to begin with ' AND'.

Actions
Copy link
 #2

Updated by Gerrit Code Review over 11 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13041

Actions
Copy link
 #3

Updated by Gerrit Code Review over 11 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13041

Actions
Copy link
 #4

Updated by Gerrit Code Review over 11 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/13041

Actions
Copy link
 #5

Updated by Gerrit Code Review about 11 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/13041

Actions
Copy link
 #6

Updated by Gerrit Code Review about 11 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/19774

Actions
Copy link
 #7

Updated by Robert Heel about 11 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 50 to 100
Actions
Copy link
 #8

Updated by Gerrit Code Review about 11 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19775

Actions
Copy link
 #9

Updated by Gerrit Code Review about 11 years ago

Patch set 2 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/19775

Actions
Copy link
 #10

Updated by Gerrit Code Review about 11 years ago

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/19776

Actions
Copy link
 #11

Updated by Robert Heel about 11 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #12

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF