Project

General

Profile

Actions

Feature #33703

closed

Auto deactivation of scheduler task for saltedpasswords

Added by Christian Nölle about 12 years ago. Updated over 5 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.

Actions #1

Updated by Philipp Gampe about 12 years ago

I am working on this :)

Actions #2

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

Actions #3

Updated by Gerrit Code Review about 12 years ago

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

Actions #4

Updated by Christian Nölle about 12 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.

Actions #5

Updated by Gerrit Code Review about 12 years ago

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

Actions #6

Updated by Gerrit Code Review almost 12 years ago

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

Actions #7

Updated by Gerrit Code Review almost 12 years ago

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

Actions #8

Updated by Philipp Gampe almost 12 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF