diff --git a/modules/rest_example/src/Controller/RestExampleClientController.php b/modules/rest_example/src/Controller/RestExampleClientController.php index 195542415322cdc8eb01ca92cb68e3a62a0d45e0..2bc9ea1d64e5cbb2ae1a6f209035a72b652ef1d7 100644 --- a/modules/rest_example/src/Controller/RestExampleClientController.php +++ b/modules/rest_example/src/Controller/RestExampleClientController.php @@ -2,46 +2,69 @@ namespace Drupal\rest_example\Controller; -use Drupal\Core\Controller\ControllerBase; +use Drupal\Core\Config\ConfigFactoryInterface; +use Drupal\Core\DependencyInjection\ContainerInjectionInterface; use Drupal\Core\Link; +use Drupal\Core\Messenger\MessengerTrait; +use Drupal\Core\StringTranslation\StringTranslationTrait; use Drupal\rest_example\RestExampleClientCalls; use Symfony\Component\DependencyInjection\ContainerInterface; /** - * Controller routines for rest example routes. + * Controller class for the REST Example routes. * * @ingroup rest_example */ -class RestExampleClientController extends ControllerBase { +class RestExampleClientController implements ContainerInjectionInterface { + + use MessengerTrait; + use StringTranslationTrait; + + /** + * The config factory. + * + * @var \Drupal\Core\Config\ConfigFactoryInterface + */ + protected $configFactory; /** - * RestExampleClientCalls object. + * The service to make REST calls. * * @var \Drupal\rest_example\RestExampleClientCalls */ - private $restExampleClientCalls; + protected $restClient; /** - * RestExampleClientController constructor. + * Constructs a new \Drupal\rest_example\Controller\RestExampleClientController object. * - * @param \Drupal\rest_example\RestExampleClientCalls $rest_example_client_calls - * RestExampleClientCalls service. + * @param \Drupal\rest_example\RestExampleClientCalls $rest_client + * The service to make REST calls. */ - public function __construct(RestExampleClientCalls $rest_example_client_calls) { - $this->restExampleClientCalls = $rest_example_client_calls; + public function __construct( + ConfigFactoryInterface $config_factory, + RestExampleClientCalls $rest_client, + ) { + $this->configFactory = $config_factory; + $this->restClient = $rest_client; } /** * {@inheritdoc} */ public static function create(ContainerInterface $container) { - return new static( + $controller = new static( + $container->get('config.factory'), $container->get('rest_example_client_calls') ); + + $controller->setMessenger($container->get('messenger')); + $controller->setStringTranslation($container->get('string_translation')); + + return $controller; } /** - * Retrieve a list of all nodes available on the remote site. + * Retrieves the list of all nodes available on the remote site. * * Building the list as a table by calling the RestExampleClientCalls::index() * and builds the list from the response of that. @@ -50,14 +73,6 @@ public static function create(ContainerInterface $container) { * @throws \GuzzleHttp\Exception\GuzzleException */ public function indexAction() { - - if (NULL === $this->configFactory->get('rest_example.settings')->get('server_url')) { - $this->messenger()->addWarning($this->t('The remote endpoint service address have not been set. Please go and provide the credentials and the endpoint address on the <a href="@url">config page</a>.', ['@url' => base_path() . 'examples/rest-client-settings'])); - } - $build = []; - - $nodes = $this->restExampleClientCalls->index(); - $build['intro'] = [ '#markup' => $this->t('This is a list of nodes, of type <em>Rest Example Test</em>, on the remote server. From here you can create new node, edit and delete existing ones.'), ]; @@ -71,17 +86,27 @@ public function indexAction() { $this->t('Edit'), $this->t('Delete'), ], - '#empty' => t('There are no items on the remote system yet'), + '#empty' => $this->t('There are no items on the remote system yet'), ]; - if (!empty($nodes)) { - foreach ($nodes as $delta => $node) { - $build['node_table'][$delta]['title']['#plain_text'] = $node['title']; - $build['node_table'][$delta]['type']['#plain_text'] = $node['type']; - $build['node_table'][$delta]['created']['#plain_text'] = $node['created']; - $build['node_table'][$delta]['edit']['#plain_text'] = Link::createFromRoute($this->t('Edit'), 'rest_example.client_actions_edit', ['id' => $node['nid']])->toString(); - $build['node_table'][$delta]['delete']['#plain_tex'] = Link::createFromRoute($this->t('Delete'), 'rest_example.client_actions_delete', ['id' => $node['nid']])->toString(); + if ($this->configFactory->get('rest_example.settings')->get('server_url')) { + $this->messenger()->addWarning( + $this->t('The remote endpoint service address has not been set. Please go and provide the credentials and the endpoint address on the <a href="@url">config page</a>.', ['@url' => base_path() . 'examples/rest-client-settings']) + ); + } + else { + $nodes = $this->restClient->index(); + + if (!empty($nodes)) { + foreach ($nodes as $delta => $node) { + $build['node_table'][$delta]['title']['#plain_text'] = $node['title']; + $build['node_table'][$delta]['type']['#plain_text'] = $node['type']; + $build['node_table'][$delta]['created']['#plain_text'] = $node['created']; + $build['node_table'][$delta]['edit']['#plain_text'] = Link::createFromRoute($this->t('Edit'), 'rest_example.client_actions_edit', ['id' => $node['nid']])->toString(); + $build['node_table'][$delta]['delete']['#plain_tex'] = Link::createFromRoute($this->t('Delete'), 'rest_example.client_actions_delete', ['id' => $node['nid']])->toString(); + } } + } return $build; diff --git a/modules/rest_example/src/RestExampleClientCalls.php b/modules/rest_example/src/RestExampleClientCalls.php index d978884eda7663ce2a42dd78726487a6ba08c024..cba3e65af1aaab42892db68e64ae85a36fc0b762 100644 --- a/modules/rest_example/src/RestExampleClientCalls.php +++ b/modules/rest_example/src/RestExampleClientCalls.php @@ -59,7 +59,7 @@ class RestExampleClientCalls { * @param \GuzzleHttp\ClientInterface $client * The HTTP client. * @param \Drupal\Core\Config\ConfigFactoryInterface $config_factory - * The Config Factory. + * The config factory. */ public function __construct(ClientInterface $client, ConfigFactoryInterface $config_factory) { $this->client = $client;