Bug #34156: No-Admin can’t delete FE-User from Recycler - TYPO3 Core - TYPO3 Forge

Actions

Bug #34156

closed

No-Admin can’t delete FE-User from Recycler

Added by Mario Näther over 12 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Should have

Assignee:

-

Category:

-

Target version:

-

Start date:

2012-02-21

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

4.5

PHP Version:

5.2

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

I delete an FE-user as no-Admin from normal BE list module and the fe-user moves to Recycler.
If I delete this fe-user from recycler as no-Admin, the recycler quest me "realy delete?" Yes or No? and I clicked on "Yes" and nothing happened, the fe-user already exists!

The SQLDebug of ExtJS request outputed:

SELECT * FROM fe_users WHERE uid=3877553 AND fe_users.deleted=0

But the deleted flag was already set to 1 from deletion on list module

I can reproduce this on TYPO3 4.5.11 and TYPO 4.6.4

TSConfig Settings for recycler

mod.recycler.allowDelete = 1

The attached patch file (from T3 4.6), fixed this on my TYPO3.

Files

Download all files

0001-test.patch (1.02 KB) 0001-test.patch		Mario Näther, 2012-02-21 18:18
0002-test.patch (998 Bytes) 0002-test.patch		Mario Näther, 2012-02-27 19:52

Actions

#1

Updated by Stefan Galinski over 12 years ago

Thanks for your report! Please open a new review request for your patch.

http://wiki.typo3.org/Contribution_Walkthrough_Tutorials

Actions

#2

Updated by Mario Näther over 12 years ago

File 0002-test.patch 0002-test.patch added

Here is a new patch file for 4.6. With this you can delete and undelete records from recycle.

Sorry but I can't open an review request, because I can't connect my TortoiseGit to Gerrit. I will added the Git Hook and the GitBash says "Permission denied (publickey)." ... do you have any solutions for me?

Actions

#3

Updated by Gerrit Code Review over 12 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/10480

Actions

#4

Updated by Gerrit Code Review over 12 years ago

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

Actions

#5

Updated by Gerrit Code Review almost 12 years ago

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

Actions

#6

Updated by Gerrit Code Review over 11 years ago

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

Actions

#7

Updated by Gerrit Code Review over 11 years ago

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

Actions

#8

Updated by Gerrit Code Review over 11 years ago

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

Actions

#9

Updated by Gerrit Code Review over 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/21454

Actions

#10

Updated by Gerrit Code Review over 11 years ago

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

Actions

#11

Updated by Gerrit Code Review over 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/21456

Actions

#12

Updated by Sascha Egerer over 11 years ago

Status changed from Under Review to Resolved
% Done changed from 0 to 100

Applied in changeset 55eb3f88d128a3247d57f77c388389ea0a7df531.

Actions

#13

Updated by Benni Mack about 6 years ago

Status changed from Resolved to Closed

Actions

Also available in: Atom PDF