Skip to content
GitLab
    • Why GitLab
    • Pricing
    • Contact Sales
    • Explore
  • Why GitLab
  • Pricing
  • Contact Sales
  • Explore
  • Sign in
  • Get free trial
  • AppsByTheRealRyanBonham
  • Open-RealtyOpen-Realty
  • Issues
  • #149

Remove Adodb

Summary

Use PDO and remove Adodb dependency.

Phase 1

  • Adds PDO Extension to compose
  • Adds PDO & PDO Mysql Extenstion test to installer.
  • Setups PDO Connection in OpenRealty\API namespace
  • Moves retrieval of $config to a new SettingsApi
  • Updates Installer to write DB Settings to a ini file in the api folder.
  • Update All API Commands to accept the PDO dbh as at construction.
  • Update All API Commands to accept setings at construction.
  • Update all Open-Realty Classes to accept Settings at construction.
  • Remove all references to global $config.
  • Update all OpenRealty Classes to accept PDO dbh at construction
  • Update all API Calls to pass in PDO db and Settings.
  • Update Hooks to get passed $dbh and $config (This is done, but not correctly needs rework before merging. Should pass by class construct instead of each function.)
  • Update Addons to get passed $dbh and $config

Phase 2

  • Replace all uses of adodb in API with PDO
    • BlogAPi
    • FieldsApi
    • ListingApi
    • LogApi
    • MediaApi #257
    • MenuApi #258
    • PageApi #259
    • PClassApi #261
    • SettingsApi
    • TwitterApi
    • UserApi

Phase 3

  • Work to replace all non API calls to database with API calls.
  • Remove Adodb

Extras (Test feasibility)

  • Consolidate index.php, ajax.php, admin/index.php, admin/ajax.php, aand pingback.php into a single file..

/cc @therealryanbonham

Edited Feb 26, 2023 by Ryan Bonham
Assignee
Assign to
Time tracking