Tickets :: [#8128] Enhancements for String::substr

6.0.0-RC7

6/18/26

History
Attachments
Comment
Watch
Download

Summary	Enhancements for String::substr
Queue	Horde Framework Packages
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	jan (at) horde (dot) org
Requester	jens (at) peino (dot) de
Created	3/30/09 (6289 days ago)
Due
Updated	3/30/09 (6289 days ago)
Assigned
Resolved	3/30/09 (6289 days ago)
Milestone
Patch	Yes

413	CVS Commit	Comment #8	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/Util/String.php?rt=horde&r1=1.43.6.36&r2=1.43.6.37&ty=u http://cvs.horde.org/diff.php/horde/docs/CHANGES?rt=horde&r1=1.515.2.516&r2=1.515.2.517&ty=u

373	Jan Schneider	Comment #7 Assigned to Jan Schneider State ⇒ Resolved	Reply to this comment
Committed, thanks.

93	CVS Commit	Comment #6	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/framework/Util/String.php?rt=horde&r1=1.94&r2=1.95&ty=u

332	Jan Schneider	Deleted Original Message

401	jens (at) peino (dot) de	Comment #5 New Attachment: framework_utils_String[1].patch	Reply to this comment
New patch attached. Previous had bad whitespaces and windows lineendings. Sorry.

321	Jan Schneider	Comment #4	Reply to this comment
patch framework_cvs/Util/String.php < framework_utils_String.patch (Stripping trailing CRs from patch.) patching file framework_cvs/Util/String.php Hunk #1 FAILED at 340. patch unexpectedly ends in middle of line Hunk #2 FAILED at 358. 2 out of 2 hunks FAILED -- saving rejects to file framework_cvs/Util/String.php.rej

331	jens (at) peino (dot) de	Comment #3	Reply to this comment
The patch doesn't apply. What do _you_ mean? Is broken? Or not is it not appropriate? - Jens

3812	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
The patch doesn't apply.

558	jens (at) peino (dot) de	Comment #1 State ⇒ New New Attachment: framework_utils_String.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ Horde Framework Packages Summary ⇒ Enhancements for String::substr Type ⇒ Enhancement Priority ⇒ 1. Low	Reply to this comment
1.) iconv_substr throws an error notice when then source string contains an illegal character. -> Use same suppression like done for the case mbstring is enabled. 2.) iconv_substr returns anyway -> Go ahead in case of error and try the other available ways.