Solr\Bridge\Util

Complexity

Total Complexity

5

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	convertLocationCoordinates()	0	10	1
A	convertDateTime()	0	4	1
A	convertSolrDateToPhpDateTime()	0	12	3

<?php
/**
 * YAWIK
 *
 * @filesource
 * @copyright (c) 2013 - 2016 Cross Solution (http://cross-solution.de)
 * @license   MIT
 */

namespace Solr\Bridge;

use Jobs\Entity\Location;
use DateTime;

/**
 * Static utility to do conversion
 *
 * @author Anthonius Munthi <me@itstoni.com>
 * @author Miroslav Fedeleš <miroslav.fedeles@gmail.com>
 * @since  0.26
 * @package Solr\Bridge
 */
class Util
{
    /**
     * Convert Location coordinate into acceptable solr document format
     * @param Location $location
     * @return string
     */
    static public function convertLocationCoordinates(Location $location)

As per PSR2, the static declaration should come after the visibility declaration.

    {
        $coordinates = $location->getCoordinates()->getCoordinates();

It seems like you code against a specific sub-type and not the parent class GeoJson\GeoJson as the method getCoordinates() does only exist in the following sub-classes of GeoJson\GeoJson: GeoJson\Geometry\Geometry, GeoJson\Geometry\GeometryCollection, GeoJson\Geometry\LineString, GeoJson\Geometry\LinearRing, GeoJson\Geometry\MultiLineString, GeoJson\Geometry\MultiPoint, GeoJson\Geometry\MultiPolygon, GeoJson\Geometry\Point, GeoJson\Geometry\Polygon, Geo\Entity\Geometry\Point. Maybe you want to instanceof check for one of these explicitly?

        $coordinate = doubleval($coordinates[0]).'%'.doubleval($coordinates[1]);
        $coordinate = strtr($coordinate,[
            '%'=>',',
            ','=>'.'
        ]);
        return $coordinate;
    }

    /**
     * Convert date time into acceptable solr document format
     *
     * @param DateTime $date
     * @return string
     */
    static public function convertDateTime(DateTime $date)

As per PSR2, the static declaration should come after the visibility declaration.

    {
        return $date->setTimezone(new \DateTimeZone('UTC'))->format(Manager::SOLR_DATE_FORMAT);
    }

    /**
     * Convert Solr date into a PHP DateTime object
     *
     * @param string $solrDate
     * @return DateTime
     */
    static public function convertSolrDateToPhpDateTime($solrDate)

As per PSR2, the static declaration should come after the visibility declaration.

    {
        $solrDate = trim($solrDate);
        $dateTime = DateTime::createFromFormat(Manager::SOLR_DATE_FORMAT, $solrDate);
        $valid = $dateTime && ($dateTime->format(Manager::SOLR_DATE_FORMAT) === $solrDate);
        
        if (!$valid) {
            throw new \InvalidArgumentException(sprintf('invalid format of Solr date passed: "%s"', $solrDate));
        }
        
        return $dateTime;
    }
}