Re: [commits] Horde branch master updated. f8e275b289d8314fcfa5ad66d0a8609f02c2a31e

Jan Schneider <[email protected]>
Newsgroups gmane.comp.horde.devel
Message-ID <[email protected]>
Zitat von Michael M Slusarz <[email protected]>:

> Quoting Jan Schneider <[email protected]>:
>
>>> 37a91d8 Use Horde_Imap_Client_Namespace_List object return
>>> fba625b Optimization: If searching all mailboxes, no need to add  
>>> individual namespaces to the query
>>
>> Same procedure as always: please stop refactoring of code while we  
>> are in bug fixing mode for a stable release! This is completely  
>> untested code, that now requires a new Horde_Imap_Client release  
>> that has to be tested against stable application releases, yet  
>> another beta release for IMP 6.2 because this is more than a bug,  
>> and further delayed final releases for the Horde 5.2 series.
>
> Except this is ALL related to a single bug fix.  See:
>
> http://bugs.horde.org/ticket/13212#c3
> http://bugs.horde.org/ticket/13212#c4
>
> So this code needs to be tested somewhere.  Might as well do it  
> right the first time rather than having to hack a unmanageable  
> solution, just to back it out one version later.
>
> Not to mention that this is way better than trying to fix in IMP,  
> since now we get unit tests to ensure that the code works properly.
>
> michael
>
> ___________________________________
> Michael Slusarz [[email protected]]

Except that it doesn't.

2014-05-27T16:32:55+02:00 NOTICE: HORDEDEVELOP [imp] PHP ERROR:  
Undefined variable: ns_val [pid 22523 on line 87 of  
"/home/jan/horde-develop/imp/lib/Ftree/Account/Imap.php"]
2014-05-27T16:33:02+02:00 NOTICE: HORDEDEVELOP [imp] PHP ERROR: Trying  
to get property of non-object [pid 22523 on line 36 of  
"/home/jan/horde-develop/imp/lib/Ftree/IteratorFilter/Expanded.php"]
2014-05-27T16:33:02+02:00 NOTICE: HORDEDEVELOP [imp] PHP ERROR: Trying  
to get property of non-object [pid 22523 on line 29 of  
"/home/jan/horde-develop/imp/lib/Ftree/IteratorFilter/Invisible.php"]
2014-05-27T16:33:02+02:00 NOTICE: HORDEDEVELOP [imp] PHP ERROR: Trying  
to get property of non-object [pid 22523 on line 48 of  
"/home/jan/horde-develop/imp/lib/Ftree/IteratorFilter/Subscribed.php"]
2014-05-27T16:33:02+02:00 NOTICE: HORDEDEVELOP [imp] PHP ERROR: Trying  
to get property of non-object [pid 22523 on line 49 of  
"/home/jan/horde-develop/imp/lib/Ftree/IteratorFilter/Subscribed.php"]

- INBOX is no longer localized
- is sorted inside the folder list
- cannot be accessed from the sidebar anymore

Sure it needs to be tested somewhere, but not while we want to reach RCs.

-- 
Jan Schneider
The Horde Project
http://www.horde.org/
https://www.facebook.com/hordeproject

-- 
dev mailing list
Frequently Asked Questions: http://wiki.horde.org/FAQ
To unsubscribe, mail: [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.