Tickets :: [#7482] Undefined function removeBareNewlines

Summary	Undefined function removeBareNewlines
Queue	IMP
Queue Version	HEAD
Type	Bug
State	Resolved
Priority	3. High
Owners	slusarz (at) horde (dot) org
Requester	chuck (at) horde (dot) org
Created	10/12/2008 (6114 days ago)
Due
Updated	10/12/2008 (6114 days ago)
Assigned
Resolved	10/12/2008 (6114 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

10/12/2008 11:37:16 PM	Michael Slusarz	State ⇒ Resolved

10/12/2008 11:36:59 PM	Michael Slusarz	Comment #2	Reply to this comment
My bad - I went back and forth on this and obviously confused myself. Even though the use of the removeNewlines() call in IMP is temporary (it will be totally self-contained in the IMAP lib once we switch over), it might be useful in the future and is best exposed publicly. Fixed.

10/12/2008 05:44:26 PM	Chuck Hagenbuch	Comment #1 Priority ⇒ 3. High Patch ⇒ No Milestone ⇒ Assigned to Michael Slusarz Queue ⇒ IMP Summary ⇒ Undefined function removeBareNewlines Type ⇒ Bug State ⇒ Assigned	Reply to this comment
IMP calls Horde_Imap_Client::removeBareNewlines(), but in the framework package, that function is protected (and prefixed with an underscore). Should IMP be calling another function, or should the function be exposed as public by the imap client?