Tickets :: [#2748] PEAR xml2sql dependency pulls php 5 out

6.0.0-alpha12

6/7/25

Summary	PEAR xml2sql dependency pulls php 5 out
Queue	Horde Base
Queue Version	FRAMEWORK_3
Type	Enhancement
State	Resolved
Priority	1. Low
Owners
Requester	glen (at) delfi (dot) ee
Created	10/08/2005 (7182 days ago)
Due
Updated	10/25/2005 (7165 days ago)
Assigned
Resolved	10/25/2005 (7165 days ago)
Milestone
Patch	No

10/25/2005 09:37:26 AM	Jan Schneider	Comment #9 State ⇒ Resolved	Reply to this comment
Committed, thanks.

10/24/2005 09:42:02 PM	glen (at) delfi (dot) ee	Comment #8	Reply to this comment
just for information. patch is working fine for me, without any reports so far.

10/11/2005 06:53:58 PM	glen (at) delfi (dot) ee	Comment #7	Reply to this comment
oh, i'm yet, so inexperienced of reading minds, i'll try to improve! :) updated patch!

10/11/2005 05:16:44 PM	Jan Schneider	Comment #6	Reply to this comment
I know that it works. I expected you to create an updated patch.

10/10/2005 04:03:17 PM	glen (at) delfi (dot) ee	Comment #5	Reply to this comment
yes, that could do. just tested. $ php -r '$a = @include_once("nothing.inc"); $b = @include_once("nothing.inc"); $c = @include_once("not-there.inc"); echo "$a, $b, $c\n";' 1, 1,

10/09/2005 05:11:32 PM	Jan Schneider	Comment #4	Reply to this comment
How about: @include_once 'XML/xml2sql.php';

10/09/2005 05:00:50 PM	glen (at) delfi (dot) ee	Comment #3	Reply to this comment
yeah sure, in fact the patch doesn't fix, just sample where i found the problem. i don't know much of Horde code. i'm sure Horde has utility function to detect it properly without hardwiring paths :)

10/08/2005 07:16:38 PM	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
PEAR may be installed in any directory.

10/08/2005 06:21:45 PM	glen (at) delfi (dot) ee	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ PEAR xml2sql dependency pulls php 5 out Queue ⇒ Horde Base State ⇒ New	Reply to this comment
Horde seems to require XML/xml2sql.php PEAR package. to be exact, the Serialize class. Of currently installed horde packages on my system (all stable Horde packages + few others) i did not find any piece of code which acutally uses such serialize $mode. and the PEAR/xml2sql itself is not compatible with php 5 due domxml library change. see also http://pear.php.net/bugs/bug.php?id=904 so here's a patch which makes it optional ie returns false if pear package is not installed: http://cvs.pld-linux.org/cgi-bin/cvsweb/SOURCES/horde-sql2xml.patch