[Phplib-trackers] [ phplib-Patches-487847 ] Bigger patch

[<no...@so...>]

Patches item #487847, was opened at 2001-12-01 08:22
You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=403613&aid=487847&group_id=31885

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Dieter Steinwedel (dsteinwe)
Assigned to: Nobody/Anonymous (nobody)
Summary: Bigger patch

Initial Comment:
Hi folks,

I've posted here a set of my patched files of the 
phplib. The files "db_oracle.inc" and "db_oci8.inc" 
are rewritten and therefore renamed as "db_*_ds.inc". 
Additionally, the "db_sql" class has got the new 
method "column_names()". This method gives you the 
column names of a query in an array. The class "Table" 
uses this function to avoid seeking in the query 
result, because some DBs don't like "seeking". I 
hope "table_ds.inc" (after renaming as "table.inc") 
will be a part of the phplib. This *requires* the 
additional implementation of the method "column_names
()" in all other "db_*.inc" files. The implementation 
should be quite simple and is - in my eyes - very 
useful. 

By the way, I've fixed some further bugs found in the 
messages: #445385, #445386, #445389, #472136, #482372, 
#445388 and #450706 (my new version 
of "db_oci8_ds.inc" makes possibly this message 
superfluous).

All modified files contains further informations about 
the changes. Changes are marked with a "DS1".

I would be glad about feedback!!! 
Please, mail and help to make the phplib perfect.


Dieter

----------------------------------------------------------------------

Comment By: Gaetano Giunta (ggiunta)
Date: 2002-01-10 04:33

Message:
Logged In: YES 
user_id=114789

my 2 eurocents:

in page.inc, better use 'if (isset($user) and 
(is_object($user)))'

Gaetano

----------------------------------------------------------------------

Comment By: Dieter Steinwedel (dsteinwe)
Date: 2001-12-29 09:20

Message:
Logged In: YES 
user_id=373951

Unfortunally, there is an error occurred on uploading. 
Please, don't download the file "PAGE.INC" (I cannot delete 
it). All you need is "patch.zip".

----------------------------------------------------------------------

Comment By: Dieter Steinwedel (dsteinwe)
Date: 2001-12-29 09:15

Message:
Logged In: YES 
user_id=373951

Missing files added ...

----------------------------------------------------------------------

You can respond by visiting: 
http://sourceforge.net/tracker/?func=detail&atid=403613&aid=487847&group_id=31885