Differences

This shows you the differences between two versions of the page.

--- v5:database:for_removal [2020/01/08 03:26] – [Table] mnewnham
+++ v5:database:for_removal [2020/01/09 04:03] (current) – [Functions in adodb-lib.inc.php] mnewnham
@@ Line 47: / Line 47: @@
+ ===== The thing about the PDO driver =====
+Take, for example, pdo_mysql. This driver is really just the mysql driver. It doesn't really have anything to do with PDO. This is the same for all the PDO drivers.
+If you were going to make the drivers loadable classes, it would make no sense to have these as extensions of PDO, they should be an extension of the native driver so that they all have access to the already written methods in the native driver.
+How to link them to the base driver I have no idea.
+===== LDAP Driver =====
+Isn't really the same thing as all the others. It's useful, but doesnt really fit in with the others. Maybe we should break it off to its own directory or something.
+===== ociquercus =====
+This seems to have a lot of complicated cursor stuff in it. I think John was a bit of an oracle hacker. would be a shame to lose it but not quite sure what to do with it
+===== Directory Structure =====
+I think we should break a lot of stuff away from the main directory, perhaps it could match a namespace or something?
+<code php>
+\adodb
+\adodb\adodb
+\adodb\adodb\drivers
+\adodb\adodb\datadict
+\adodb\adodb\perf
+\adodb\adodb\xmlschema
+\adodb\adodb\memcache
+\adodb\adodb\errors
+\adodb\adodb\lang
+\adodb\session
+\adodb\activerecord
+\adodb\time
+\adodb\other - everything else in subdirectories
+</code>
+This would empty the main directory
+===== Functions in adodb-lib.inc.php =====
+There are a lot of functions in this file, for eaxample ''_adodb_column_sql()''. I've never understood why these are functions and not part of the class. I think we should make them class functions, and if necessary, create some kind of helper functions that calls them. so:
+<code php>
+function _adodb_column_sql(&$zthis ..
+becomes ADOconnection::adodbColumnSql
+</code>