TopviewDigital\TranslationHelper\Service\Translation

Complexity

Total Complexity

12

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	randomUserAgent()	0	8	1
A	__construct()	0	11	1
B	translation()	0	39	7
A	handle()	0	9	3

<?php

namespace TopviewDigital\TranslationHelper\Service;

use Campo\UserAgent;
use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Foundation\Bus\Dispatchable;
use Illuminate\Queue\InteractsWithQueue;
use Illuminate\Queue\SerializesModels;
use Stichoza\GoogleTranslate\GoogleTranslate;
use TopviewDigital\TranslationHelper\Model\VocabTerm;

class Translation implements ShouldQueue
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by TopviewDigital\TranslationHelper\Service\Translation: $id, $class, $relations

    protected $break = 0;
    protected $term;
    protected $locales;
    protected $translation;

    public function __construct(VocabTerm $term = null, array $locales = [])
    {
        $this->locales = array_unique(
            array_merge($locales, [
                app()->getLocale(),

The method getLocale() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

                config('app.locale'),
                config('app.fallback_locale'),
                config('app.faker_locale'),
            ])
        );
        $this->term = $term;
    }

    public function handle()
    {
        if (empty($this->term)) {
            sweep();
            foreach (VocabTerm::get()->all() as $u) {
                $this->translation($u, $this->locales);

The call to TopviewDigital\TranslationHelper\Service\Translation::translation() has too many arguments starting with $this->locales.

            }
        } else {
            $this->translation($this->term);
        }
    }

    private function randomUserAgent()
    {
        sleep(1);
        $this->called++;

        return [
            'headers' => [
                'User-Agent' => UserAgent::random(),
            ],
        ];
    }

    private function translation(VocabTerm $term)
    {
        $row = 0;
        while ($row < 1) {
            $this->called = 0;

The property called does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

            try {
                $language = new GoogleTranslate();
                // $language->setOptions($this->randomUserAgent())
                //     ->setSource()
                //     ->setTarget(config('app.locale'))
                //     ->translate($term->term);
                // $default_locale = $language->setOptions($this->randomUserAgent())->getLastDetectedSource();
                $translation = empty($term->translation) ? [] : $term->translation;

The property translation does not seem to exist on TopviewDigital\TranslationHelper\Model\VocabTerm. Are you sure there is no database migration missing?

                foreach ($this->locales as $locale) {
                    if (!array_key_exists($locale, $term->translation)) {
                        $translation[$locale] = $language
                            ->setOptions($this->randomUserAgent())
                            ->setSource()
                            ->setTarget($locale)
                            ->translate($term->term);

The property term does not seem to exist on TopviewDigital\TranslationHelper\Model\VocabTerm. Are you sure there is no database migration missing?

                        $this->called++;
                    }
                }
                $locale = $language->setOptions($this->randomUserAgent())->getLastDetectedSource();
                if (!array_key_exists($locale, $term->translation)) {
                    $translation[$locale] = $term->term;
                }
                $term->translation = $translation;
                $language = null;

The assignment to $language is dead and can be removed.

                $term->save();
                $row++;
            } catch (Exception $e) {

The type TopviewDigital\TranslationHelper\Service\Exception was not found. Did you mean Exception? If so, make sure to prefix the type with \.

                $this->break++;
                $mins = rand(
                    floor($this->called / ($row + 1)),
                    floor($this->called / ($row + 1) * 2)
                ) * $this->break;
                sleep($mins * 60);
            }
        }
    }
}