+global\sn_module_payment::db_insert()

Complexity

Conditions	7
Paths	48

Size

Total Lines	54
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes	2
Bugs	0	Features	0

<?php
  public function db_insert() {
    $this->payment_test = !empty($this->config['test']) || $this->payment_test;

The property $payment_test was declared of type integer, but !empty($this->config['test']) || $this->payment_test is of type boolean. Maybe add a type cast?

    $payment = array(
      'payment_module_name' => $this->manifest['name'],

      'payment_status' => $this->payment_status,
      // 'payment_date' => $this->payment_date, // Не нужно

50% of this comment could be valid code. Did you maybe forget this after debugging?

      'payment_provider_id' => $this->payment_provider_id,
      'payment_account_id' => $this->payment_account_id,
      'payment_account_name' => $this->payment_account_name,
      'payment_user_id' => $this->payment_user_id,
      'payment_user_name' => $this->payment_user_name,

      'payment_dark_matter_paid' => $this->payment_dark_matter_paid,
      'payment_dark_matter_gained' => $this->payment_dark_matter_gained,

      'payment_amount' => $this->payment_amount,
      'payment_currency' => $this->payment_currency,

      'payment_external_id' => $this->payment_external_id, // TODO
      'payment_external_amount' => $this->payment_external_amount,
      'payment_external_currency' => $this->payment_external_currency,
      'payment_external_date' => $this->payment_external_date, // TODO

      'payment_test' => $this->payment_test ? 1 : 0, // Boolean -> int

      'payment_comment' => $this->description_generated[PAYMENT_DESCRIPTION_MAX],

      'payment_external_lots' => $this->payment_dark_matter_paid / get_mm_cost(),
    );

    $replace = false;
    if($this->payment_id) {
      $payment['payment_id'] = $this->payment_id;
      $replace = true;
    }

    $query = array();
    foreach($payment as $key => $value) {
      $value = is_string($value) ? '"' . db_escape($value) . '"' : $value;
      $query[] = "`{$key}` = {$value}";
    }

    $longQuery = ' INTO `{{payment}}` SET ' . implode(',', $query) . ';';
    if($replace) {
      $this->db->doReplace('REPLACE' . $longQuery);
    } else {
      $this->db->doInsert('INSERT' . $longQuery);
    }

    return $this->db_get_by_id($this->db->db_insert_id());
  }