Facebook\ObjectDebugger\Command\RefreshCommand

Complexity

Total Complexity

14

Size/Duplication

Total Lines	133
Duplicated Lines	9.02 %

Coupling/Cohesion

Components	2
Dependencies	5

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	34	1
A	scrape()	0	15	2
B	execute()	12	47	11

<?php

/**
 * @file RefreshCommand.php
 * @brief This file contains the RefreshCommand class.
 * @details
 * @author Filippo F. Fadda
 */


namespace Facebook\ObjectDebugger\Command;


use Symfony\Component\Console\Exception\InvalidOptionException;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Output\OutputInterface;
use Symfony\Component\Console\Input\InputOption;

use Facebook\Facebook;


/**
 * @brief Scrapes pubs' information from WhatPub.
 * @nosubgrouping
 */
class RefreshCommand extends AbstractCommand {

  /**
   * @var Facebook $fb
   */
  private $fb;

  /**
   * @var InputInterface $input
   */
  private $input;

  /**
   * @var OutputInterface $output
   */
  private $output;


  /**
   * @brief Fetches new scrape information and update the Facebook cache.
   */
  protected function configure() {
    $this->setName("refresh");
    $this->setDescription("Fetches new scrape information and update the Facebook cache");

    $this->addOption("file",
      'i',
      InputOption::VALUE_REQUIRED,
      "Text file which contains a list of URLs, one per row");

    $this->addOption("url",
      'u',
      InputOption::VALUE_REQUIRED,
      "URL of a single page to scrape");

    $this->addOption("id",
      'd',
      InputOption::VALUE_REQUIRED,
      "Facebook id");

    $this->addOption("secret",
      's',
      InputOption::VALUE_REQUIRED,
      "Facebook secret");

    $this->addOption("token",
      't',
      InputOption::VALUE_REQUIRED,
      "Facebook App|User access token");

    $this->addOption("encode",
      'e',
      InputOption::VALUE_NONE,
      "Encode the provided URL");
  }


  /**
   * @brief Scrapes the information.
   * @param string $url The page URL to scrape.
   */
  private function scrape($url) {
    $this->output->write($url);

    if ($this->input->getOption('encode'))
      $url = urlencode($url);

    $params = [
      'id' => $url,
      'scrape' => 'true'
    ];

    $response = $this->fb->post('/', $params);

$response is not used, you could remove the assignment.

    $this->output->writeln(' => done');
  }


  /**
   * @brief Executes the command.
   * @param InputInterface $input The input interface
   * @param OutputInterface $output The output interface
   * @return string
   */
  protected function execute(InputInterface $input, OutputInterface $output) {
    $this->input = $input;
    $this->output = $output;

    $config = $this->getApplication()->getConfig();

It seems like you code against a specific sub-type and not the parent class Symfony\Component\Console\Application as the method getConfig() does only exist in the following sub-classes of Symfony\Component\Console\Application: Facebook\ObjectDebugger\Console. Maybe you want to instanceof check for one of these explicitly?

    if ($input->getOption('id'))
      $appId = $input->getOption('id');
    elseif (array_key_exists('appId', $config))
      $appId = $config['appId'];
    else
      throw new InvalidOptionException('Facebook Open Graph requires an App ID.');

    if ($input->getOption('secret'))

This code seems to be duplicated across your project.

      $appSecret = $input->getOption('secret');
    elseif (array_key_exists('appSecret', $config))
      $appSecret = $config['appSecret'];
    else
      throw new InvalidOptionException('Facebook Open Graph requires an App Secret.');

    if ($input->getOption('token'))

This code seems to be duplicated across your project.

      $accessToken = $input->getOption('token');
    elseif (array_key_exists('appAccessToken', $config))
      $accessToken = $config['appAccessToken'];
    else
      throw new InvalidOptionException('Facebook Open Graph requires an App|User Access Token.');

    $this->fb = new Facebook([
      'app_id' => $appId,
      'app_secret' => $appSecret,
      'default_access_token' => $accessToken,
      'default_graph_version' => 'v2.8',
    ]);

    if ($url = $input->getOption('url')) {
      $this->scrape($url);
    }
    elseif ($fileName = $input->getOption('file')) {
      $urls = file($fileName, FILE_IGNORE_NEW_LINES);

      if ($urls === FALSE)
        throw new \RuntimeException('Cannot open the file.');

      foreach ($urls as $url)
        $this->scrape($url);
    }
  }

}