Author Topic: [0.2 stable] The auto-checkmail doesn't work anymore  (Read 16986 times)

Offline Napsty

  • Jr. Member
  • **
  • Posts: 20
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #15 on: January 08, 2009, 08:55:19 AM »
Works for me with both Firefox (3.0.5) and IE7 (7.0.5730.13) on Windows XP Pro SP3.
Installed roundcube today with the tar.gz file downloaded on January 5th.

Update:
Tested it also with IE6 (6.0.2900.5512) on Windows XP Home SP3 -> works.
« Last Edit: January 08, 2009, 01:55:59 PM by Napsty »

Offline rosali

  • Hero Member
  • *****
  • Posts: 2,533
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #16 on: January 08, 2009, 01:13:50 PM »
Could you please check if it works when ...

./program/steps/mail/check_recent.inc
Code: [Select]

        //if ($header->recent)


... is out commented ?
« Last Edit: January 08, 2009, 03:55:19 PM by rosali »
Regards,
Rosali
__________________
MyRoundcube Project (commercial)

Offline Konishi

  • Newbie
  • *
  • Posts: 5
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #17 on: January 08, 2009, 03:46:53 PM »
Quote from: rosali;16153
Could you please check if it works when ...

./program/steps/mail/check_recent.inc
Code: [Select]

        //if ($header->recent)


... is out commented ?


That did not fix it for me.  This symptom only occurs with specific accounts and I cannot find any correlation between them (size, age, frequency of use).  There are no javascript errors and it happens with Opera, IE6, and IE7.  I suspect that there is another necessary step for those of us upgrading from previous versions of roundcube that would resolve this.
« Last Edit: January 08, 2009, 03:55:27 PM by rosali »

Offline rosali

  • Hero Member
  • *****
  • Posts: 2,533
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #18 on: January 08, 2009, 04:07:57 PM »
Specific accounts? Please disable caching in addition ...

I was able to reproduce the issue by using an old v0.2 beta installation of a backup which I updated now. I commented out what I've posted above ... Browser cache has been cleared of course. Worked for me ... But caching was and is disabled in the updated installation ...

I noticed that $header->recent is false even if there is a "recent" message. May be messages are touched before it runs into check_recent.inc include ?

Strange ...

___
I'm a mod not a dev !!!
Regards,
Rosali
__________________
MyRoundcube Project (commercial)

Offline Konishi

  • Newbie
  • *
  • Posts: 5
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #19 on: January 08, 2009, 04:16:15 PM »
Quote from: rosali;16157
Specific accounts? Please disable caching in addition ...

I was able to reproduce the issue by using an old v0.2 beta installation of a backup which I updated now. I commented out what I've posted above ... Browser cache has been cleared of course. Worked for me ... But caching was and is disabled in the updated installation ...

I noticed that $header->recent is false even if there is a "recent" message. May be messages are touched before it runs into check_recent.inc include ?

Strange ...

___
I'm a mod not a dev !!!


I started seeing it in accounts that had previously worked now, but your post pointed me to the solution.  Apparently messages ARE being touched (but not being marked as read) by standard email clients.  If thunderbird or outlook is also checking for messages, roundcube will not see them.  Can't believe I didn't catch this earlier, but regardless the devs should possibly look into another method for deciding whether or not to download a new header.

tl;dr-- Stop any other email clients from accessing your accounts at the same time as roundcube.

Offline lucaferr

  • Newbie
  • *
  • Posts: 7
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #20 on: January 09, 2009, 05:57:22 PM »
I tried commenting it out... Nothing....

Maybe I found the problem in the logs (it happens every minute):
[Fri Jan 09 17:22:33 2009] [error] [client 192.168.0.4] PHP Notice:  Undefined index:  HTTPS in /WEB/egos_mail/program/include/iniset.php on line 56, referer: http://192.168.0.1:25902/egos_mail/?_task=mail&_mbox=INBOX&_refresh=1


Could it be the cause? How can I solve it?
Thanks!

Offline rosali

  • Hero Member
  • *****
  • Posts: 2,533
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #21 on: January 10, 2009, 01:28:30 AM »
Replace line 56 by ...
Code: [Select]

if  (isset($_SERVER['HTTPS'])) {
   ini_set('session.cookie_secure', ($_SERVER['HTTPS'] && ($_SERVER['HTTPS'] != 'off'))?1:0);
} else {
   ini_set('session.cookie_secure', 0);
}

... but I don't not believe that's the reason ...

The URL for the recent messages check should be:
./?_task=mail&_action=check-recent&_t=1231568689322&_remote=1

Quote from trunk:
_________________
Quote

Changeset 2211
 Timestamp: 01/02/09 08:48:10 (8 days ago)
Author: till
Message: * fixing notice, moved error_reporting one up, added @todo
« Last Edit: January 10, 2009, 01:31:47 AM by rosali »
Regards,
Rosali
__________________
MyRoundcube Project (commercial)

Offline lucaferr

  • Newbie
  • *
  • Posts: 7
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #22 on: January 11, 2009, 09:13:39 AM »
You were right... Now my log is totally empty, but the auto check-mail and the check mail button don't work yet! :(
Any ideas?

Offline rosali

  • Hero Member
  • *****
  • Posts: 2,533
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #23 on: January 11, 2009, 02:31:22 PM »
Are you sure not to have  a n y  Javascript errors on the page?
Regards,
Rosali
__________________
MyRoundcube Project (commercial)

Offline cboshuizen

  • Newbie
  • *
  • Posts: 1
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #24 on: January 11, 2009, 07:03:15 PM »
I can confirm that this issue occurs for me when there are other IMAP clients touching the file, as pointed out on page 2.  

Make sure your thunderbird, etc are closed when testing this feature.

Chris

Offline Konishi

  • Newbie
  • *
  • Posts: 5
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #25 on: January 11, 2009, 07:15:44 PM »
Quote from: cboshuizen;16238
I can confirm that this issue occurs for me when there are other IMAP clients touching the file, as pointed out on page 2.  

Make sure your thunderbird, etc are closed when testing this feature.

Chris


Yes, maybe everyone overlooked my post since it was the last on Page 2, but this does seem to be the cause of the problem.  Could everyone else please check if this is the case?

Offline lucaferr

  • Newbie
  • *
  • Posts: 7
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #26 on: January 12, 2009, 05:05:38 PM »
It could be... I only use RoundCube as client, but I have "imapsync" that synchronizes the mailbox on my own server with the mailbox on the provider's one every 4 minutes. I read that it synchronizes everything but the "/recent" attribute, cause it can't be modified by a client...
I can't disable it, otherwise I wouldn't receive e-mails at all...
Can you help me? I'm getting desperate...
Maybe we could find a way to only check on mails without "/seen" instead of mails with "/recent"...?
I dumped some javascript errors I have, don't know if it helps:
Quote

Avviso: anonymous function does not always return a value
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/common.js
Riga: 85
Codice sorgente:
},get_mouse_pos:function(e){
Avviso: function rcube_layer does not always return a value
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/common.js
Riga: 274, Colonna: 1
Codice sorgente:
};
Avviso: anonymous function does not always return a value
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 386, Colonna: 22
Codice sorgente:
};
Avviso: test for equality (==) mistyped as assignment (=)?
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 476, Colonna: 55
Codice sorgente:
if(_45=document.getElementById("rcm"+this.env.sort_col)){
Avviso: test for equality (==) mistyped as assignment (=)?
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 479, Colonna: 41
Codice sorgente:
if(_45=document.getElementById("rcm"+_43)){
Avviso: test for equality (==) mistyped as assignment (=)?
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 698, Colonna: 28
Codice sorgente:
if(uid=this.get_single_uid()){
Avviso: variable _64 redeclares argument
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 992, Colonna: 11
Codice sorgente:
var li,pos,_64,_68;
Avviso: test for equality (==) mistyped as assignment (=)?
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 998, Colonna: 27
Codice sorgente:
if(li=this.get_folder_li(k)){
Avviso: anonymous function does not always return a value
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 1144
Codice sorgente:
};
Avviso: test for equality (==) mistyped as assignment (=)?
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 1685, Colonna: 39
Codice sorgente:
if(f=rcube_find_object("_draft_saveid")){
Avviso: test for equality (==) mistyped as assignment (=)?
File sorgente: http://10.8.0.10:25902/egos_mail/program/js/app.js
Riga: 2849, Colonna: 32
Codice sorgente:
if(item=this.get_folder_li(mbox)){
And thousands of other similars... I use firefox with "Enable JS strict warnings" enabled.
Sorry for the parts in Italian... "Riga = row; Colonna = column",

Offline rosali

  • Hero Member
  • *****
  • Posts: 2,533
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #27 on: January 13, 2009, 05:58:02 AM »
@lucaferr:

Adjust ./program/steps/mail/check_recent.inc to refresh the message list if there are unseen messages instead of recent messages ...
Regards,
Rosali
__________________
MyRoundcube Project (commercial)

Offline pcrew

  • Newbie
  • *
  • Posts: 2
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #28 on: January 20, 2009, 02:32:43 PM »
Quote from: rosali;16306
@lucaferr:

Adjust ./program/steps/mail/check_recent.inc to refresh the message list if there are unseen messages instead of recent messages ...


what do we adjust exactly to try and resolve this problem?
should  we be changing:
if ($header->recent)

to

if ($header->unseen)

or some other option of "RECENT" to "UNSEEN" (there are many "RECENT" occurrence in caps)

Offline rosali

  • Hero Member
  • *****
  • Posts: 2,533
[0.2 stable] The auto-checkmail doesn't work anymore
« Reply #29 on: January 20, 2009, 03:06:03 PM »
First try to  o u t c o m m e n t ...

Code: [Select]

// if ($header->recent)


If it does not help then replace RECENT by UNSEEN (only uppercase occurrencies within this file)!


http://www.roundcubeforum.net/deutsch/4102-0-2-stable-autocheck-nach-neuen-emails-funktioniert-nicht.html
Regards,
Rosali
__________________
MyRoundcube Project (commercial)