From 18f4aaefc6bd57de9a2fb5ad4621bc3e293657b6 Mon Sep 17 00:00:00 2001
From: "google-labs-jules[bot]"
 <161369871+google-labs-jules[bot]@users.noreply.github.com>
Date: Tue, 14 Jul 2026 00:56:45 +0000
Subject: [PATCH] Refactor database connection to use environment variables

Eliminates hardcoded database credentials in `php/oop/database.php`
by dynamically loading them via `getenv()`. Default values are provided
for host and database name.

Co-authored-by: tsainez <13399044+tsainez@users.noreply.github.com>
---
 php/oop/database.php | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/php/oop/database.php b/php/oop/database.php
index 902916e..a0d9c10 100644
--- a/php/oop/database.php
+++ b/php/oop/database.php
@@ -8,7 +8,12 @@ public function connect_to_database() {
 
 		# try {
 
-			$this->connection = new PDO('msql:host=localhost;dbname=lab8', 'lab8');
+			$db_host = getenv('DB_HOST') ?: 'localhost';
+			$db_name = getenv('DB_NAME') ?: 'lab8';
+			$db_user = getenv('DB_USER');
+			$db_pass = getenv('DB_PASS');
+
+			$this->connection = new PDO("msql:host=$db_host;dbname=$db_name", $db_user, $db_pass);
 
 		/* 
 		} catch (PDOException $e) {