EPDOStatement::execute() - Code Metrics - Inspection of "Add interpolator query PDO PHP" - dimaslanjaka/universal-framework - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 3b2975...5480a6 )

by Dimas
created 2020-07-04 17:40 UTC
EPDOStatement::execute() A

↳ Parent: EPDOStatement
Complexity

Conditions	3
Paths	4
Size

Total Lines	26
Code Lines	14
Duplication

Lines	0
Ratio	0 %
Importance

Changes	1
Bugs	0	Features	0
Metric	Value
cc	3
eloc	14
nc	4
nop	1
dl	0
loc	26
rs	9.7998
c	1
b	0
f	0
<?php

/**
 * Copyright 2018 github.com/noahheck.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

namespace DB;

use PDO as PDO;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use PDOStatement as PDOStatement;
use Psr\Log\LoggerAwareInterface;
use Psr\Log\LoggerInterface;

class EPDOStatement extends PDOStatement implements LoggerAwareInterface
{
  const WARNING_USING_ADDSLASHES = 'addslashes is not suitable for production logging, etc. Please consider updating your processes to provide a valid PDO object that can perform the necessary translations and can be updated with your e.g. package management, etc.';

  /**
   * @var \PDO
   */
  protected $_pdo = '';

  /**
   * @var LoggerInterface
   */
  private $logger;

  /**
   * @var string - will be populated with the interpolated db query string
   */
  public $fullQuery;

  /**
   * @var array - array of arrays containing values that have been bound to the query as parameters
   */
  protected $boundParams = [];

  /**
   * The first argument passed in should be an instance of the PDO object. If so, we'll cache it's reference locally
   * to allow for the best escaping possible later when interpolating our query. Other parameters can be added if
   * needed.
   *
   * @param \PDO $pdo
   */
  protected function __construct(PDO $pdo = null)
  {
    if ($pdo) {
      $this->_pdo = $pdo;
    }
  }

  /**
   * {@inheritdoc}
   */
  public function setLogger(LoggerInterface $logger)
  {
    $this->logger = $logger;
  }

  /**
   * Overrides the default \PDOStatement method to add the named parameter and it's reference to the array of bound
   * parameters - then accesses and returns parent::bindParam method.
   *
   * @param string $param
   * @param mixed  $value
   * @param int    $datatype
   * @param int    $length
   * @param mixed  $driverOptions
   *
   * @return bool - default of \PDOStatement::bindParam()
   */
  public function bindParam($param, &$value, $datatype = PDO::PARAM_STR, $length = 0, $driverOptions = false)
  {
    $this->debug(
      'Binding parameter {param} (as parameter) as datatype {datatype}: current value {value}',
      [
        'param' => $param,
        'datatype' => $datatype,
        'value' => $value,
      ]
    );

    $this->boundParams[$param] = [
      'value' => &$value, 'datatype' => $datatype,
    ];

    return parent::bindParam($param, $value, $datatype, $length, $driverOptions);
  }

  /**
   * Overrides the default \PDOStatement method to add the named parameter and it's value to the array of bound values
   * - then accesses and returns parent::bindValue method.
   *
   * @param string $param
   * @param mixed  $value
   * @param int    $datatype
   *
   * @return bool - default of \PDOStatement::bindValue()
   */
  public function bindValue($param, $value, $datatype = PDO::PARAM_STR)
  {
    $this->debug(
      'Binding parameter {param} (as value) as datatype {datatype}: value {value}',
      [
        'param' => $param,
        'datatype' => $datatype,
        'value' => $value,
      ]
    );

    $this->boundParams[$param] = [
      'value' => $value, 'datatype' => $datatype,
    ];

    return parent::bindValue($param, $value, $datatype);
  }

  /**
   * Copies $this->queryString then replaces bound markers with associated values ($this->queryString is not modified
   * but the resulting query string is assigned to $this->fullQuery).
   *
   * @param array $inputParams - array of values to replace ? marked parameters in the query string
   *
   * @return string $testQuery - interpolated db query string
   */
  public function interpolateQuery($inputParams = null)
  {
    $this->debug('Interpolating query...');

    $testQuery = $this->queryString;

    $params = ($this->boundParams) ? $this->boundParams : $inputParams;

    if ($params) {
      ksort($params);

      foreach ($params as $key => $value) {
        $replValue = (is_array($value)) ? $value
          : [
            'value' => $value,
            'datatype' => PDO::PARAM_STR,
          ];

        $replValue = $this->prepareValue($replValue);

        $testQuery = $this->replaceMarker($testQuery, $key, $replValue);
      }
    }

    $this->fullQuery = $testQuery;

    $this->debug('Query interpolation complete');
    $this->debug('Interpolated query: {query}', ['query' => $testQuery]);

    return $testQuery;
  }

  /**
   * Overrides the default \PDOStatement method to generate the full query string - then accesses and returns
   * parent::execute method.
   *
   * @param array $inputParams
   *
   * @return bool - default of \PDOStatement::execute()
   */
  public function execute($inputParams = [])
  {
    /** migration \DB\statement */
    $this->_debugValues = $inputParams;

    $this->interpolateQuery($inputParams);

    try {
      $response = parent::execute($inputParams);

      if (!$response) {
        $this->error('Failed executing query: {query}', ['query' => $this->fullQuery]);

        return $response;
      }
    } catch (\Exception $e) {
      $this->error('Exception thrown executing query: {query}', ['query' => $this->fullQuery]);
      $this->error($e->getMessage(), ['exception' => $e]);

      throw $e;
    }

    $this->debug('Query executed: {query}', ['query' => $this->fullQuery]);
    $this->info($this->fullQuery);

    return $response;
  }

  private function replaceMarker($queryString, $marker, $replValue)
  {
    /*
     * UPDATE - Issue #3
     * It is acceptable for bound parameters to be provided without the leading :, so if we are not matching
     * a ?, we want to check for the presence of the leading : and add it if it is not there.
     */
    if (is_numeric($marker)) {
      $marker = "\?";
    } else {
      $marker = (preg_match('/^:/', $marker)) ? $marker : ':' . $marker;
    }

    $this->debug(
      'Replacing marker {marker} with value {value}',
      [
        'marker' => $marker,
        'value' => $replValue,
      ]
    );

    $testParam = "/({$marker}(?!\w))(?=(?:[^\"']|[\"'][^\"']*[\"'])*$)/";

    // Back references may be replaced in the resultant interpolatedQuery, so we need to sanitize that syntax
    $cleanBackRefCharMap = ['%' => '%%', '$' => '$%', '\\' => '\\%'];

    $backReferenceSafeReplValue = strtr($replValue, $cleanBackRefCharMap);

    $interpolatedString = preg_replace($testParam, $backReferenceSafeReplValue, $queryString, 1);

    return strtr($interpolatedString, array_flip($cleanBackRefCharMap));
  }

  /**
   * Prepares values for insertion into the resultant query string - if $this->_pdo is a valid PDO object, we'll use
   * that PDO driver's quote method to prepare the query value. Otherwise:.
   *
   *      addslashes is not suitable for production logging, etc. You can update this method to perform the necessary
   *      escaping translations for your database driver. Please consider updating your processes to provide a valid
   *      PDO object that can perform the necessary translations and can be updated with your e.g. package management,
   *      etc.
   *
   * @param array $value - an array representing the value to be prepared for injection as a value in the query string
   *                     with it's associated datatype:
   *                     ['datatype' => PDO::PARAM_STR, 'value' => 'something']
   *
   * @return string $value - prepared $value
   */
  private function prepareValue($value)
  {
    if (null === $value['value']) {
      $this->debug("Value is null: returning 'NULL'");

      return 'NULL';
    }

    if (PDO::PARAM_INT === $value['datatype']) {
      $this->debug('Preparing value {value} as integer', ['value' => $value]);

      return (int) $value['value'];
    }

    if (!$this->_pdo) {
      $this->debug('Preparing value {value} using addslashes', ['value' => $value]);
      $this->warn(self::WARNING_USING_ADDSLASHES);

      return "'" . addslashes($value['value']) . "'";
    }

    $this->debug('Preparing value {value} as string', ['value' => $value]);

    return  $this->_pdo->quote($value['value']);
  }

  /**
   * @param string $message
   * @param array  $context
   */
  private function debug($message, $context = [])
  {
    if ($this->logger) {
      $this->logger->debug($message, $context);
    }
  }

  /**
   * @param string $message
   * @param array  $context
   */
  private function warn($message, $context = [])
  {
    if ($this->logger) {
      $this->logger->warning($message, $context);
    }
  }

  /**
   * @param string $message
   * @param array  $context
   */
  private function error($message, $context = [])
  {
    if ($this->logger) {
      $this->logger->error($message, $context);
    }
  }

  private function info($message, $context = [])
  private function info($message, /** @scrutinizer ignore-unused */ $context = [])
  {
    if ($this->logger) {
      $this->logger->info($message);
    }
  }

  /**
   * Migrate from \DB\statement.
   */
  protected $_debugValues = null;

  public function _debugQuery($replaced = true)
  {
    $q = $this->queryString;

    if (!$replaced) {
      return $q;
    }

    return preg_replace_callback('/:([0-9a-z_]+)/i', [$this, '_debugReplace'], $q);
  }

  protected function _debugReplace($m)
  {
    $v = $this->_debugValues[$m[1]];
    if (null === $v) {
      return 'NULL';
    }
    if (!is_numeric($v)) {
      $v = str_replace("'", "''", $v);
    }

    return "'" . $v . "'";
  }
}

dimaslanjaka / universal-framework

Push — master ( 3b2975...5480a6 )

EPDOStatement::execute() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
