Skip to content
Snippets Groups Projects
Commit dfba9738 authored by Jether Paradero's avatar Jether Paradero Committed by Morten Bartvig
Browse files

Issue #3363728 by Priya_Degwekar, paraderojether: Fix the issues reported by phpcs

parent c36b84f9
No related branches found
No related tags found
No related merge requests found
......@@ -14,4 +14,3 @@ fieldset#edit-api .prepublish-is-error {
background-color: #f9c9bf;
padding: 5px 10px;
}
......@@ -2,7 +2,7 @@
* @file
* Styles for Siteimprove toolbar.
*/
.siteimprove-toolbar .siteimprove-link {
.siteimprove-toolbar .siteimprove-link {
padding-left: 3.25em;
position: relative;
}
......
......@@ -42,7 +42,7 @@ function siteimprove_update_8101() {
*/
function siteimprove_update_8102() {
$settings = \Drupal::configFactory()->getEditable('siteimprove.settings');
$settings->set('prepublish_enabled', false);
$settings->set('prepublish_enabled', FALSE);
$settings->set('api_username', '');
$settings->set('api_key', '');
$settings->set('enabled_content_types', []);
......@@ -55,6 +55,6 @@ function siteimprove_update_8102() {
*/
function siteimprove_update_8103() {
$settings = \Drupal::configFactory()->getEditable('siteimprove.settings');
$settings->set('use_latest_experience', true);
$settings->set('use_latest_experience', TRUE);
$settings->save();
}
......@@ -8,6 +8,9 @@
use Drupal\Core\Entity\EntityInterface;
use Drupal\Core\Url;
use Drupal\Component\Utility\NestedArray;
use Drupal\node\Entity\Node;
use Drupal\taxonomy\Entity\Term;
use Drupal\Core\Entity\ContentEntityInterface;
/**
* Implements hook_toolbar().
......@@ -23,7 +26,7 @@ function siteimprove_toolbar() {
$taxonomy = \Drupal::routeMatch()->getParameter('taxonomy_term');
$bundle = '';
//In event node parameter is not a node object.
// In event node parameter is not a node object.
if (is_string($node)) {
$node = \Drupal::entityTypeManager()->getStorage('node')->load($node);
}
......@@ -38,16 +41,14 @@ function siteimprove_toolbar() {
$toolbar = [];
// Get bundle from either node or taxonomy.
if (is_object($node) && $node instanceof \Drupal\node\Entity\Node) {
$bundle = $node->bundle();
if (is_object($node) && $node instanceof Node) {
$bundle = $node->bundle();
}
elseif (is_object($taxonomy) && $taxonomy instanceof \Drupal\taxonomy\Entity\Term) {
elseif (is_object($taxonomy) && $taxonomy instanceof Term) {
$bundle = $taxonomy->bundle();
}
/**
* Prepublish toolbar item.
*/
// Prepublish toolbar item.
$prepublish_access = $user->hasPermission('use siteimprove prepublish');
$prepublish_available = FALSE;
$config = \Drupal::service('config.factory')->get('siteimprove.settings');
......@@ -84,15 +85,13 @@ function siteimprove_toolbar() {
];
}
/**
* Recheck toolbar item.
*/
// Recheck toolbar item.
$recheck_access = $user->hasPermission('use siteimprove');
$enabled_route_names = [
'entity.node.canonical',
'entity.node.edit_form',
'entity.taxonomy_term.canonical',
'entity.taxonomy_term.edit_form'
'entity.taxonomy_term.edit_form',
];
$current_route_name = \Drupal::routeMatch()->getRouteName();
if ($recheck_access && in_array($current_route_name, $enabled_route_names)) {
......@@ -110,9 +109,7 @@ function siteimprove_toolbar() {
$toolbar_items['siteimprove_toolbar']['tray']['#attached']['drupalSettings']['siteimprove']['recheck'] = \Drupal::service('siteimprove.utils')->getSiteimproveSettings($urls, 'recheck', FALSE);
}
/**
* Settings page toolbar item.
*/
// Settings page toolbar item.
if ($user->hasPermission('administer siteimprove')) {
$toolbar_items['siteimprove_toolbar']['tray']['actions']['#items']['settings'] = [
'#type' => 'link',
......@@ -131,7 +128,7 @@ function siteimprove_toolbar() {
}
// Current entity or null.
$current_entity = $node ? $node : ($taxonomy ? $taxonomy : null);
$current_entity = $node ? $node : ($taxonomy ? $taxonomy : NULL);
// Add current viewed entity as cacheable dependency.
$renderer = \Drupal::service('renderer');
......@@ -145,6 +142,7 @@ function siteimprove_toolbar() {
* Basic toolbar array.
*
* @return array[]
* Toolbar array.
*/
function _siteimprove_toolbar_basic() {
return [
......@@ -164,7 +162,7 @@ function _siteimprove_toolbar_basic() {
'#url' => Url::fromUserInput('#'),
'#attributes' => [
'class' => ['siteimprove-link'],
]
],
],
'#wrapper_attributes' => [
'class' => ['siteimprove-toolbar'],
......@@ -178,10 +176,8 @@ function _siteimprove_toolbar_basic() {
'class' => ['toolbar-menu'],
],
],
'urls' => [
],
'empty' => [
]
'urls' => [],
'empty' => [],
],
],
];
......@@ -241,14 +237,15 @@ function siteimprove_page_attachments(array &$attachments) {
$attachments['#attached']['library'][] = $siteimprove->getSiteimproveOverlayLibrary($use_latest_experience);
$attachments['#attached']['library'][] = $siteimprove->getSiteimproveLibrary();
// If node pages or taxonomy term pages, add input method, else domain method.
// If node pages or taxonomy term pages, add input method, else domain
// method.
$enabled_route_names = [
'entity.node.canonical',
'entity.node.latest_version',
'entity.node.edit_form',
'entity.taxonomy_term.canonical',
'entity.taxonomy_term.latest_version',
'entity.taxonomy_term.edit_form'
'entity.taxonomy_term.edit_form',
];
$route_match = \Drupal::routeMatch();
$current_route_name = $route_match->getRouteName();
......@@ -263,11 +260,11 @@ function siteimprove_page_attachments(array &$attachments) {
$bundle = '';
$parameters = $route_match->getParameters();
foreach ($parameters as $parameter) {
if (is_object($parameter) && $parameter instanceof \Drupal\node\Entity\Node) {
if (is_object($parameter) && $parameter instanceof Node) {
$type = 'enabled_content_types';
$bundle = $parameter->bundle();
}
elseif (is_object($parameter) && $parameter instanceof \Drupal\taxonomy\Entity\Term) {
elseif (is_object($parameter) && $parameter instanceof Term) {
$type = 'enabled_taxonomies';
$bundle = $parameter->bundle();
}
......@@ -280,7 +277,7 @@ function siteimprove_page_attachments(array &$attachments) {
if (!empty($method)) {
$parameters = \Drupal::routeMatch()->getParameters();
foreach ($parameters as $param) {
if (is_object($param) && $param instanceof \Drupal\Core\Entity\ContentEntityInterface) {
if (is_object($param) && $param instanceof ContentEntityInterface) {
if ($param->in_preview !== TRUE) {
/** @var \Drupal\Core\Entity\ContentEntityType $type */
$entity_type = $param->getEntityType()->id();
......
......@@ -17,7 +17,7 @@ function siteimprove_domain_access_help($route_name, RouteMatchInterface $route_
$output = '';
$output .= '<h3>' . t('About') . '</h3>';
$output .= '<p>' . t('Add a integration between Siteimprove module and Domain Access. For now only a Siteimprove Domain plugin automatically using the Domain Access domains as frontend domains.') . '</p>';
$output .= '<p><a href="https://www.drupal.org/docs/contributed-modules/siteimprove" target="_blank" title="Siteimprove module documentation guide on Drupal.org">' . t('Find more help in the documentation guide on drupal.org') . '.</a></p>';
$output .= '<p>' . t('Find more help in the @documentation_guide.', ['@documentation_guide' => '<a href="https://www.drupal.org/docs/contributed-modules/siteimprove" target="_blank" title="Siteimprove module documentation guide on Drupal.org">documentation guide on drupal.org</a>']) . '</p>';
return $output;
default:
......
......@@ -365,7 +365,7 @@ class SettingsForm extends ConfigFormBase {
*/
protected function setRepublish($username, $key) {
$url = 'https://api.siteimprove.com/v2/settings/content_checking';
$res = $this->httpClient->request('POST', $url, [
$this->httpClient->request('POST', $url, [
'auth' => [$username, $key],
'headers' => [
'Accept' => 'application/json',
......
......@@ -147,8 +147,9 @@ class SiteimproveUtils {
/**
* Return Siteimprove js library.
*
* @param $use_latest_experience
* @param bool $use_latest_experience
* Use latest version of overlay when set to true.
*
* @return string
* Siteimprove js library.
*/
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment