Auto Responder API Issues

Submitted by izoox on Mon, 12/27/2010 - 13:35

So I was trying to add the --autoreply-period to my roundcube vacation plugin because now that users have access to easily manage their autoreplies, I am getting complaints about autoreply loops. It doesn't appear that the --autoreply-period option in the api actually works. I have tried it as below:

virtualmin modify-user --domain domain.com --user test --local --autoreply "Test" --autoreply-period 1

as well as

virtualmin modify-user --domain domain.com --user test --autoreply-period 1

Status: 
Closed (fixed)

Comments

Submitted by JamieCameron on Mon, 12/27/2010 - 23:52

Odd, this works fine for me.

If you look at the user's autoreply text file after using the --autoreply-period flag, do you see a line like :

Reply-Period: 1

Submitted by izoox on Tue, 12/28/2010 - 11:33

I see that in the file, but the form inside virtualmin doesn't show the period. The option "Minimum time between autoreplies" shows no minimum. Shouldn't it show 1?

Submitted by JamieCameron on Tue, 12/28/2010 - 12:07

Ok, I see the bug now .. the issue is that when the API is used, the reply-tracking DB isn't properly setup. I will fix this in the next release.

Also, the parameter to --autoreply-period is in seconds, even though the web UI converts this to minutes.

Submitted by izoox on Tue, 12/28/2010 - 12:10

The command line help states it as --autoreply-period mins, so you might want to update that as well.

Submitted by JamieCameron on Tue, 12/28/2010 - 12:13

Thanks, I will fix that too.

Submitted by izoox on Tue, 12/28/2010 - 12:18

Cool. Thanks. So even though the form doesn't show correctly right now, the option is still being used correctly?

Submitted by JamieCameron on Tue, 12/28/2010 - 12:23

No, the option isn't actually being used correctly unless enabled via the web UI..

Submitted by izoox on Tue, 12/28/2010 - 12:24

If the next release isn't coming out for a while can you provide a patch? With the holidays, I have tons of customers using this and need the replay period working.

Submitted by JamieCameron on Tue, 12/28/2010 - 12:46

Sure .. just replace the file /usr/{share,libexec}/webmin/virtual-server/modify-user.pl with the attached file.

Submitted by izoox on Tue, 12/28/2010 - 12:58

Works perfectly now. Thanks so much, you just made my new year that much better since I wont have to be woken up early with, "I have 4000 messages in my inbox"

Submitted by Issues on Tue, 01/11/2011 - 14:20

Automatically closed -- issue fixed for 2 weeks with no activity.