Skip to content
Snippets Groups Projects
Commit d860cfd1 authored by Dhirendra Pandey's avatar Dhirendra Pandey Committed by Daniele Piaggesi
Browse files

Issue #3134827 by nitk.dhirendra, g0blin79: Drupal 9 Deprecated code report

parent 8c656c66
No related branches found
No related tags found
No related merge requests found
......@@ -2,5 +2,6 @@ name: Previous & Next
type: module
description: Add a "Previous/Next" links to the node display.
core: 8.x
core_version_requirement: ^8 || ^9
package: Other
configure: prevnext.admin_settings
......@@ -83,6 +83,13 @@ function prevnext_entity_extra_field_info() {
*/
function prevnext_node_view(array &$build, Node $node, EntityViewDisplayInterface $display, $view_mode) {
// To check if current node is configured for prevnext or not.
$config = \Drupal::config('prevnext.settings');
$enabled_nodetypes = $config->get('prevnext_enabled_nodetypes');
if (empty($enabled_nodetypes[$node->getType()])) {
return;
}
/** @var \Drupal\prevnext\PrevnextService $prevnext */
$prevnext = \Drupal::service('prevnext.service');
$previous_next = $prevnext->getPreviousNext($node);
......
services:
prevnext.service:
class: Drupal\prevnext\PrevnextService
arguments: ["@entity.query"]
arguments: ["@entity_type.manager"]
......@@ -38,7 +38,7 @@ class PrevnextSettingsForm extends ConfigFormBase {
'#description' => $this->t('Check node types enabled for Previous/Next'),
'#type' => 'checkboxes',
'#options' => node_type_get_names(),
'#default_value' => $config->get('prevnext_enabled_nodetypes'),
'#default_value' => !empty($config->get('prevnext_enabled_nodetypes')) ? $config->get('prevnext_enabled_nodetypes') : array(),
);
return parent::buildForm($form, $form_state);
......
......@@ -2,8 +2,9 @@
namespace Drupal\prevnext;
use Drupal\Core\Entity\Query\QueryFactory;
use Drupal\Core\Entity\EntityTypeManager;
use Drupal\node\Entity\Node;
use Drupal\node\NodeInterface;
/**
* Class PrevnextService.
......@@ -13,11 +14,11 @@ use Drupal\node\Entity\Node;
class PrevnextService implements PrevnextServiceInterface {
/**
* The entity query factory.
* The entity manager.
*
* @var \Drupal\Core\Entity\Query\QueryFactory
* @var Drupal\Core\Entity\EntityTypeManager
*/
protected $queryFactory;
protected $entityTypeManager;
/**
* Previous / Next nids.
......@@ -29,11 +30,11 @@ class PrevnextService implements PrevnextServiceInterface {
/**
* PrevnextService constructor.
*
* @param QueryFactory $query
* The entity query instance.
* @param EntityTypeManager $entityTypeManager
* The entity type manager instance.
*/
public function __construct(QueryFactory $query) {
$this->queryFactory = $query;
public function __construct(EntityTypeManager $entityTypeManager) {
$this->entityTypeManager = $entityTypeManager;
}
/**
......@@ -60,10 +61,10 @@ class PrevnextService implements PrevnextServiceInterface {
* An array of nodes filtered by type, status and language.
*/
protected function getNodesOfType(Node $node) {
$query = $this->queryFactory->get('node');
$query = $this->entityTypeManager->getStorage('node')->getQuery();
$bundle = $node->bundle();
$langcode = $node->language()->getId();
$nodes = $query->condition('status', NODE_PUBLISHED)
$nodes = $query->condition('status', NodeInterface::PUBLISHED)
->condition('type', $bundle)
->condition('langcode', $langcode)
->addMetaData('type', $bundle)
......
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