Feature #33703

Auto deactivation of scheduler task for saltedpasswords

Added by Christian Nölle over 9 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Could have
Assignee:
-
Category:
Authentication
Target version:
-
Start date:
2012-02-06
Due date:
% Done:

100%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

As we all know, the scheduler task for saltedpasswords has a built in auto deactivation, which jumps in, when no more password updates are neccessary. In class.tx_saltedpasswords_tasks_bulkupdate.php one can find

    /**
     * @var boolean Whether or not the task is allowed to deactivate itself after processing all existing user records.
     * @TODO: This could be set with an additional field later on.
     *        The idea is to not disable the task after all initial users where handled.
     *        This could be handy for example if new users are imported regularily from some external source.
     */
    protected $canDeactivateSelf = TRUE;

which would be very handy to have. So I would like to make a request to make this configurable via em for saltedpasswords.

#1

Updated by Philipp Gampe about 9 years ago

I am working on this :)

#2

Updated by Gerrit Code Review about 9 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/9139

#3

Updated by Gerrit Code Review about 9 years ago

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

#4

Updated by Christian Nölle about 9 years ago

Well, seen the patch in gerrit. Nice, thx a lot. But I would suggest to incooporate the part which is including the user counter reset after job completion for version 4.5 as well.

#5

Updated by Gerrit Code Review about 9 years ago

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

#6

Updated by Gerrit Code Review almost 9 years ago

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

#7

Updated by Gerrit Code Review almost 9 years ago

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

#8

Updated by Philipp Gampe almost 9 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
#9

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF