6.0.0-beta1
7/5/25

[#7862] ContactAutoCompleter calls non-existent private MIME::_rfc822Encode
Summary ContactAutoCompleter calls non-existent private MIME::_rfc822Encode
Queue Kronolith
Queue Version 2.3
Type Bug
State Resolved
Priority 1. Low
Owners Horde Developers (at) , chuck (at) horde (dot) org
Requester mrubinsk (at) horde (dot) org
Created 01/16/2009 (6014 days ago)
Due
Updated 01/18/2009 (6012 days ago)
Assigned
Resolved 01/18/2009 (6012 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
01/18/2009 03:40:40 AM Chuck Hagenbuch Comment #9
Assigned to Chuck Hagenbuch
State ⇒ Resolved
Reply to this comment
BC in fw3 should be restored now.
01/17/2009 06:59:12 PM Michael Rubinsky Comment #7 Reply to this comment
Another related issue is that the class variable 'rfc822_filter' has 
been removed from MIME.php as well.  This needs to be added back for 
BC reasons, as it is also used (at least) in ContactAutoCompleter.php
01/17/2009 05:27:48 PM Chuck Hagenbuch Comment #6 Reply to this comment
Right - and we can't change it now because that'll just create more BC 
problems with older Horde versions.
01/17/2009 05:12:52 PM Michael Rubinsky Comment #5 Reply to this comment
That sounds reasonable, though I still have a nagging voice in my head 
about client code calling a private method. I guess since FW_3 is 
really only being maintained for bugs at this point though, I guess 
there's not much worth in adding a public method for any possible 
future FW_3 releases as well.
01/17/2009 05:18:24 AM Chuck Hagenbuch Comment #4 Reply to this comment
Continuing my piecemeal analysis, in FW3, the method is still private, 
and called _rfc2822Encode. I *think* what we should do is:



- change the calls in the HEAD apps to use the public rfc2822Encode 
function and new 2nd parameter



- re-add the original _rfc822Encode function to MIME.php in FW3 for 
backwards compatibility with old code.



Any objections or better ideas?
01/17/2009 05:12:06 AM Chuck Hagenbuch Comment #3 Reply to this comment
... and it also affects Whups 1.0.
01/17/2009 05:11:21 AM Chuck Hagenbuch Comment #2 Reply to this comment
Following the changelog, this is a BC break for Kronolith with Horde 3.3+
01/16/2009 11:40:52 PM Michael Rubinsky Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ ContactAutoCompleter calls non-existent private MIME::_rfc822Encode
Queue ⇒ Kronolith
Assigned to Horde DevelopersHorde Developers
Milestone ⇒
Patch ⇒ No
State ⇒ Assigned
Reply to this comment
The private method MIME::_rfc822Encode() is called directly from 
kronolith's ContacatAutoCompleter.php file.  That method was renamed 
to _rfc2822Encode() and then made public (renamed to rfc2822Encode()) 
in MIME.php



The function signature has also changed, and now no longer takes a 
'filter' argument, but a 'type' argument in it's place.



This is a sort-of BC break in that the function was changed in MIME, 
but kronolith should not have been calling a private method on MIME 
directly...

Saved Queries