PHP changing fetch_field() to mysqli

Question

My php knowledge is fairly limited but I\'ve recently needed to update a number of web pages from an older version of php 5.2 to php 7.3.

I\'ve managed to update mo

Answer 1

The original function is wrong on so many levels. And there is no point in recreating its functionality.

Basically all you are bargaining for here is just a few SQL keywords. But these keywords contribute for readability.

For some reason you decided to outsmart several generations of programmers who are pretty happy with SQL syntax, and make unreadable gibberish

$row = selectonerow("some, foo, bar", "baz", "id", [$uniquevalue]);

instead of almost natural English

$row = selectonerow("SELECT some, foo, bar FROM baz WHERE id=?", [$uniquevalue]);

Come on. It doesn't worth.

Make your function accept a regular SQL query, not a limited unintelligible mess.

function selectonerow(mysqli $conn, string $sql, array $params = []): array
{
    if ($params) {
        $stmt = $conn->prepare($sql);
        $stmt = $mysqli->prepare($sql);
        $stmt->bind_param(str_repeat("s", count($params), ...$params);
        $stmt->execute();
        $result = $stmt->get_result()
    } else {
        $result = $conn->query($sql);
    }
    return $result->fetch_assoc();
}

This function will let you to use any query. For example, need a row with max price?

$row = selectonerow("SELECT * FROM baz ORDER BY price DESC LIMIT 1");

Need a more complex condition? No problem

$sql = "SELECT * FROM baz WHERE email=? AND activated > ?";
$row = selectonerow($sql, [$email, $date]);

and so on. Any SQL. Any condition.

Answer 2

I would recommend to get rid of this function or replace it with a function suggested by YCS.

If you really want to continue using this function consider the following fixes. You made the code inside extremely complicated and you forgot to pass the connection variable into the function. I have simplified it:

// open the DB connection properly inside Connections/connAsh.php
mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
$connAsh = new mysqli('host', 'user', 'pass', $database_connAsh);
$connAsh->set_charset('utf8mb4');

// your function:
function selectonerow(mysqli $connAsh, $fieldsarray, $table, $uniquefield, $uniquevalue): array
{
    //The required fields can be passed as an array with the field names or as a comma separated value string
    if (is_array($fieldsarray)) {
        $fields = implode(", ", $fieldsarray);
    } else {
        $fields = $fieldsarray;
    }

    //performs the query
    $stmt = $connAsh->prepare("SELECT $fields FROM $table WHERE $uniquefield = ?");
    $stmt->bind_param('s', $uniquevalue);
    $stmt->execute();
    return $stmt->get_result()->fetch_assoc();
}

This is your function, but with a lot of noise removed. I added $connAsh in the function's signature, so you must pass it in every time you call this function. The function will always return an array; if no records are fetched the array will be empty. This is the recommended way. Also remember to always use prepared statements!