RApiHalHelperSiteContent::createContent() - Code Metrics - Inspection of "#8: use redcore to connect joomla CMS" - redCOMPONENT-COM/redCORE - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — develop (#924)

unknown

created 2022-10-31 04:02 UTC

RApiHalHelperSiteContent::createContent() A

↳ Parent: RApiHalHelperSiteContent

Complexity

Conditions	5
Paths	4

Size

Total Lines	32
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	23
c	1
b	0
f	0
dl	0
loc	32
rs	9.2408
cc	5
nc	4
nop	1

<?php
/**
 * @package     Redcore
 * @subpackage  Api
 *
 * @copyright   Copyright (C) 2008 - 2021 redWEB.dk. All rights reserved.
 * @license     GNU General Public License version 2 or later, see LICENSE.
 */

defined('JPATH_BASE') or die;

/**
 * Api Helper class for overriding default methods
 *
 * @package     Redcore
 * @subpackage  Api Helper
 * @since       1.8
 */
class RApiHalHelperSiteContent
{
	/**
	 * Service for creating content.
	 *
	 * @param string $data content
	 *
	 * @return  boolean         True on success. False otherwise.
	 * @throws Exception
	 */
	public function save($data): string
	{
		if (version_compare(JVERSION, '3.0', 'lt')) {
			JTable::addIncludePath(JPATH_PLATFORM . 'joomla/database/table');
		}
		
		$data = (object) $data;
		
		if (isset($data->job_type) && $data->job_type == 'create')
		{
			return $this->createContent($data);

		}
		
		if (isset($data->job_type) && $data->job_type == 'edit')
		{
			return $this->editContent($data);

		}
interface ReturnsInt {
    public function returnsIntHinted(): int;
}

class MyClass implements ReturnsInt {
    public function returnsIntHinted(): int
    {
        if (foo()) {
            return 123;
        }
        // here: null is implicitly returned
    }
}
	}
	
	/**
	 * @throws Exception
	 */
	private function createContent($data): string
	{
		$article = JTable::getInstance('content');
		$article->title            = $data->title;

		$article->alias            = JFilterOutput::stringURLSafe($data->title);
		$article->introtext        = '<p>'.$data->description.'</p>';
		$article->created          = JFactory::getDate()->toSQL();;
		$article->created_by_alias = $data->user;

		$article->state            = 1;
		$article->access           = 1;

		$article->metadata         = '{"page_title":"'.$data->title.'","author":"'.$data->user.'","robots":""}';
		$article->language         = '*';

		
		if (!$article->check()) {
			throw new Exception($article->getError());
			throw new Exception(/** @scrutinizer ignore-deprecated */ $article->getError());
			return false;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
		}
		
		if ($article->store(TRUE)) {
			
			if (isset($data->image) && $data->image)
			{
				$articleId  = $article->get('id');
				$imgSrc     = $this->checkImages($data->image, $articleId);
				$article->set('introtext', '<p>'.$data->description.'</p>' . $imgSrc);
				$article->store();
			}
			return json_encode($article->getProperties());
			
		}else {
			throw new Exception($article->getError());
			throw new Exception(/** @scrutinizer ignore-deprecated */ $article->getError());
			return false;
		}
	}
	
	/**
	 * @throws Exception
	 */
	private function editContent($data): string
	{
		if (empty($data->article_id))
		{
			return false;

		}
		
		$article = JTable::getInstance('content');
		$id      = $data->article_id;
		$imgSrc  = '';
		$article->load($id);
		$article->set('title', $data->title);
		
		if (isset($data->image) && $data->image)
		{
			$imgSrc     = $this->checkImages($data->image, $id);
		}
		
		$article->set('introtext', '<p>'.$data->description.'</p>' . $imgSrc);
		
		if (!$article->store()) {
			throw new Exception($article->getError());
			throw new Exception(/** @scrutinizer ignore-deprecated */ $article->getError());
			return FALSE;
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}
		}
		;
		return json_encode($article->getProperties());
	}
	
	
	private function checkImages($image, $articleId)
	{
		$baseImgs  = $image;
		$baseImgs  = json_decode($baseImgs);
		$imgSrc    = '';
		
		foreach ($baseImgs as $baseImg)
		{
			$imgSrc .= $this->processImages($baseImg, $articleId);
		}
		return $imgSrc;
	}
	
	private function processImages($imgBase64, $articleId)
	{
		$imageInfo  = explode(";base64,", $imgBase64);
		$imgExt     = str_replace('data:image/', '', $imageInfo[0]);
		$image      = str_replace(' ', '+', $imageInfo[1]);
		$md5Name    = md5($imgBase64);
		$imageName  = $md5Name.".".$imgExt;
		$path       = JPATH_BASE.'/images/aesir/'.$articleId.'/';
	
		if (!is_dir($path)) {
			mkdir($path,0777,true);
		}
		
		if (file_put_contents($path . $imageName, base64_decode($image)))
		{
			return '<p><img src="images/aesir/'.$articleId.'/'.$imageName.'" /></p>';
		}
	}
}



redCOMPONENT-COM / redCORE

Pull Request — develop (#924)

RApiHalHelperSiteContent::createContent() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like