Hi!

I use this module v1.8 and it work: I prune (delete) users after 7 days. But after update to 1.9 users stop deleted. In preview I see them

screenshot

Cron working regular. I try run cron manually: users not deleted

CommentFileSizeAuthor
#9 user_prune.module.patch316 bytesivnish
#6 Bildschirmfoto vom 2018-06-09 13-03-17.png11.16 KBivnish
#5 2978116.patch395 bytesViktor-T
Bildschirmfoto vom 2018-06-07 18-42-44.png19.91 KBivnish

Comments

itcrowd72 created an issue. See original summary.

ivnish’s picture

ivnish
he/him
commented
Issue summary: View changes
ivnish’s picture

ivnish
he/him
commented
Viktor-T’s picture

Viktor-T commented

can confirm, something went wrong. will take a look at this on the weekend.

Viktor-T’s picture

Viktor-T commented
StatusFileSize
new 2978116.patch395 bytes

could you please test this patch. looks like the batch_process call was missing after the user_cancel function. Works on my local, but would be good to have a confirmation.

ivnish’s picture

ivnish
he/him
commented
StatusFileSize
new Bildschirmfoto vom 2018-06-09 13-03-17.png11.16 KB

After patch users deleted, but every run cron get error

screenshot

In watchdog many errors

Viktor-T’s picture

Viktor-T commented

Hi,

I was unable to reproduce this error. Could you please check if there is anything strange with the user id from the error? Something preventing Drupal's batch script to run on that id. Would be the best testing on a fresh install to be sure it is not a possible issue on the site creating the error.

Regards,
Viktor

ivnish’s picture

ivnish
he/him
commented

I try fresh install, one user removed without error. But next cron run get the same errors:

Notice: Undefined index: sets in function batch_process()
Warning: Invalid argument supplied for foreach() in function batch_process()
Notice: Undefined index: sets in function _batch_page()
Warning: Invalid argument supplied for foreach() in function _batch_page()
Notice: Undefined index: success in function _batch_process()

ivnish’s picture

ivnish
he/him
commented
StatusFileSize
new user_prune.module.patch316 bytes

Try my patch, it works without errors (users deleted)

  • itcrowd72 authored 25ede0a on 7.x-1.x 
    Issue #2978116 by itcrowd72, Viktor-T: Module stop prune users after...
Viktor-T’s picture

Viktor-T commented

looks great to me, added to the 1.x branch, will create a release soon

ivnish’s picture

ivnish
he/him
commented

Yeah, thanks!

Viktor-T’s picture

Viktor-T commented
Status: Active » Fixed
Viktor-T’s picture

Viktor-T commented
Status: Fixed » Closed (fixed)
joshuautley’s picture

joshuautley commented

After patching the module I receive the following and am unable to run cron.

An AJAX HTTP error occurred. HTTP Result Code: 500 Debugging information follows.
Path: /batch?id=1008&op=do
Status Text: Service unavailable (with message)

ResponseText:
Location /batch?id=1012&op=do
Referrer /batch?op=start&id=1012
Message Error: Call to a member function claimItem() on null in _batch_process() (line 274 of /web/content/includes/batch.inc).
Severity error