App\Service\WpPostService

Complexity

Total Complexity

13

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	findOneAlbumByName()	0	7	2
A	findOneLyricByTitle()	0	7	2
A	findOneArtistByName()	0	7	2
A	insertPost()	0	19	4
A	insertMeta()	0	13	2

<?php
namespace App\Service;

class WpPostService
{
    public function __construct($pdo)
    {
        $this->pdo = $pdo;

The property pdo does not exist. Did you maybe forget to declare it?

    }

    public function findOneAlbumByName($albumName)
    {
        $sql = 'SELECT * FROM muu_posts WHERE post_title = "%'.$albumName.'%" AND post_type="album";';
        foreach ($this->pdo->query($sql) as $res) {
            return $res;
        }
    }

    public function findOneLyricByTitle($title)
    {
        $sql = 'SELECT * FROM muu_posts WHERE post_title = "'.$title.'" AND post_type="lyrics";';
        foreach ($this->pdo->query($sql) as $res) {
            return $res;
        }
    }

    public function findOneArtistByName($artistName)
    {
        $sql = 'SELECT * FROM muu_posts WHERE post_title = "'.$artistName.'" AND post_type="artist";';
        foreach ($this->pdo->query($sql) as $res) {
            return $res;
        }
    }

    public function insertPost($params)
    {
        $sql = $this->pdo->prepare('
            INSERT INTO muu_posts
            (post_name, post_author, post_date, post_date_gmt, post_title, post_content, post_status, comment_status, ping_status, post_type)
            VALUES (:post_name, 1, :post_date, :post_gmt_date, :post_title, :post_content, :post_status, :comment_status, :ping_status, :post_type)
        ');
        foreach ($params as $key => $param) {
            $sql->bindValue($key, $param);
        }
        if (!$sql->execute()) {
            var_dump($this->pdo->errorInfo());die;

var_dump($this->pdo->errorInfo()); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

The method insertPost() contains an exit expression.

        }
        $id = $this->pdo->lastInsertId();
        $sql = 'SELECT * FROM muu_posts WHERE ID='.$id;
        foreach ($this->pdo->query($sql) as $res) {
            return $res;
        }
    }

    public function insertMeta($params)
    {
        $sql = $this->pdo->prepare('
            INSERT INTO muu_postmeta
            (post_id, meta_key, meta_value)
            VALUES (:post_id, :key, :value)
        ');
        foreach ($params as $key => $param) {
            $sql->bindValue($key, $param);
        }
        $sql->execute();
        return $this->pdo->lastInsertId();
    }
}