bytic / database

src/Query/Update.php

<?php

namespace Nip\Database\Query;

/**
 * Class Update
 * @package Nip\Database\Query
 */
class Update extends AbstractQuery
{
    /**
     * @return string
     */
    public function assemble()
    {
        $query = 'UPDATE ' . $this->protect($this->getTable()) . ' SET ' . $this->parseUpdate();

Are you sure $this->parseUpdate() of type false|string can be used in concatenation?

        $query .= $this->assembleWhere();
        $query .= $this->assembleLimit();

        return $query;

The expression return $query returns the type string which is incompatible with the return type mandated by Nip\Database\Query\AbstractQuery::assemble() of null.

    }

    /**
     * @return bool|string
     */
    public function parseUpdate()
    {
        if (!$this->parts['data']) {
            return false;
        }
        $fields = [];
        foreach ($this->parts['data'] as $data) {

The expression $this->parts['data'] of type void is not traversable.

            foreach ($data as $key => $values) {
                if (!is_array($values)) {
                    $values = [$values];
                }
                $value = $values[0];
                $quote = isset($values[1]) ? $values[1] : null;

                if ($value === null) {
                    $value = 'NULL';
                } elseif (!is_numeric($value)) {
                    if (is_null($quote)) {
                        $quote = true;
                    }
                    if ($quote) {
                        $value = $this->getManager()->getAdapter()->quote($value);
                    }
                }

                $fields[] = "{$this->protect($key)} = $value";
            }
        }

        return implode(", ", $fields);
    }
}