1.5 beta 2 - Delete mails from server after x days

[skypilot]

In the account settings panel the option to delete messages from the server has a strange behavior.

Could not be set to "No" : after validating changes, it remains on "Yes".
The number of days could only be change by setting delete to "No" (but it will stay to "Yes" state anyway)

Set the option to "No" and set days to 0, it will remains on "Yes" but days will say "0".

Now, let's set the option to "Yes" and days to 7 : remains on "Yes" and days are still "0".
Set delete to "No" and days to 7 : remains on "Yes" and days are "7"
Set delete to "No" and days to 0 : still remains on "Yes" and days are "7"

 

Only tried with a POP account.

[alvarotm01]

Hi,

a quick fix for that, could be:
in 'application/views/mail/add_account.php' replace line 199 with this one:

Code: [Select]

<?php echo yes_no_widget('mailAccount[del_mails_from_server]', 'mailAccountDelMailFromServer', $del_from_server > 0, lang('yes'), lang('no'), 60) ?>
and in 'application/controllers/MailController.class.php' replace lines 800 and 962 with this one

Code: [Select]

if (!array_var($mailAccount_data, 'del_mails_from_server', false)) $mailAccount_data['del_from_server'] = 0;

this will be fixed for next release.
regards.

[skypilot]

Thanks a lot !
I'll try and tell you tomorrow....

[skypilot]

I applied your fix, it now works as expected.
Thanks again for you reactivity.