mahara-contributors team mailing list archive

[Lester Caine <lester@xxxxxxxxxxx>]

Public bug reported:

Attached are a pair of drivers to add Firebird as an option on the
supported databases. Class goes to xmldb library, and the adodb driver
replaces the one currently provided.

The next step on this exercise is to work out how to handle wrapping of
field names with double quotes. The normal practice has been to use
'backtick' as a hook to identify both field and table names that need
processing. This is then replaced or removed as required before the SQL
is run. This would - I think - tidy up a problem I am seeing currently
in the code base where some fields have had double quotes added in the
scripts, or bodge case statements adjust things for MySQL or Postgres
.... A generic fix would allow the availability of other databases to be
expanded? And I am happy to look at handling that.

** Affects: mahara
     Importance: Undecided
         Status: New

-- 
Firebird support development
https://bugs.launchpad.net/bugs/572189
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.

Status in Mahara ePortfolio: New

Bug description:
Attached are a pair of drivers to add Firebird as an option on the supported databases. Class goes to xmldb library, and the adodb driver replaces the one currently provided.

The next step on this exercise is to work out how to handle wrapping of field names with double quotes. The normal practice has been to use 'backtick' as a hook to identify both field and table names that need processing. This is then replaced or removed as required before the SQL is run. This would - I think - tidy up a problem I am seeing currently in the code base where some fields have had double quotes added in the scripts, or bodge case statements adjust things for MySQL or Postgres .... A generic fix would allow the availability of other databases to be expanded? And I am happy to look at handling that.