Drupal 11 support

config/install/views.view.localgov_blog_list.yml

@@ -142,7 +142,6 @@ display:
           default_argument_type: fixed
           default_argument_options:
             argument: ''
-          default_argument_skip_url: false
           summary_options:
             base_path: ''
             count: true

config/install/views.view.recent_blogs.yml

@@ -210,7 +210,6 @@ display:
           title: ''
           default_argument_type: node
           default_argument_options: {  }
-          default_argument_skip_url: false
           summary_options:
             base_path: ''
             count: true

localgov_blogs.info.yml

@@ -1,6 +1,6 @@
 name: 'LocalGov Blogs'
 description: Provides blog channel and blog post content types with related views for LocalGov Drupal.
-core_version_requirement: ^9 || ^10
+core_version_requirement: ^10 || ^11
 type: module
 package: LocalGov Drupal
 

localgov_blogs.install

@@ -11,7 +11,7 @@ use Symfony\Component\Yaml\Yaml;
 /**
  * Implements hook_install().
  */
-function localgov_blogs_install($is_syncing) {
+function localgov_blogs_install($is_syncing): void {
   // Configure scheduled transitions if enabled.
   if (!$is_syncing && \Drupal::moduleHandler()->moduleExists('scheduled_transitions')) {
     localgov_blogs_configure_scheduled_transitions();
@@ -21,7 +21,7 @@ function localgov_blogs_install($is_syncing) {
 /**
  * Add the new view for blog authors.
  */
-function localgov_blogs_update_8001() {
+function localgov_blogs_update_8001(): void {
 
   $views_view_localgov_blog_authors_yml = <<<EOY
 langcode: en

localgov_blogs.module

@@ -16,7 +16,7 @@ use Drupal\views\Views;
 /**
  * Implements hook_theme().
  */
-function localgov_blogs_theme($existing, $type, $theme, $path) {
+function localgov_blogs_theme($existing, $type, $theme, $path): array {
   return [
     'localgov_blogs_prev_next_block' => [
       'variables' => [
@@ -33,7 +33,7 @@ function localgov_blogs_theme($existing, $type, $theme, $path) {
 /**
  * Implements hook_modules_installed().
  */
-function localgov_blogs_modules_installed($modules, $is_syncing) {
+function localgov_blogs_modules_installed($modules, $is_syncing): void {
 
   // Configure scheduled transitions if it's being installed.
   if (in_array('scheduled_transitions', $modules, TRUE)) {
@@ -44,7 +44,7 @@ function localgov_blogs_modules_installed($modules, $is_syncing) {
 /**
  * Implements hook_entity_extra_field_info().
  */
-function localgov_blogs_entity_extra_field_info() {
+function localgov_blogs_entity_extra_field_info(): array {
   $fields = [];
 
   // Add a blog listing field to blog channels.
@@ -69,7 +69,7 @@ function localgov_blogs_entity_extra_field_info() {
 /**
  * Implements hook_ENTITY_TYPE_view() for node entities.
  */
-function localgov_blogs_node_view(array &$build, NodeInterface $node, EntityViewDisplayInterface $display, $view_mode) {
+function localgov_blogs_node_view(array &$build, NodeInterface $node, EntityViewDisplayInterface $display, $view_mode): void {
 
   // Add the blog listing view to blog channels.
   if ($display->getComponent('localgov_blog_listing')) {
@@ -92,7 +92,7 @@ function localgov_blogs_node_view(array &$build, NodeInterface $node, EntityView
 /**
  * Implements hook_form_alter().
  */
-function localgov_blogs_form_alter(&$form, FormStateInterface $form_state, $form_id) {
+function localgov_blogs_form_alter(&$form, FormStateInterface $form_state, $form_id): void {
 
   // Add promote to channel field to blog posts.
   if (
@@ -129,7 +129,7 @@ function localgov_blogs_form_alter(&$form, FormStateInterface $form_state, $form
 /**
  * Custom submit handler for localgov_blog_channel_promote field.
  */
-function localgov_blogs_blog_channel_promote_submit(array $form, FormStateInterface $form_state) {
+function localgov_blogs_blog_channel_promote_submit(array $form, FormStateInterface $form_state): void {
 
   if (
     $form_state->getFormObject() instanceof NodeForm &&
@@ -163,7 +163,7 @@ function localgov_blogs_blog_channel_promote_submit(array $form, FormStateInterf
 /**
  * Implements hook_field_widget_complete_form_alter().
  */
-function localgov_blogs_field_widget_complete_form_alter(&$field_widget_complete_form, FormStateInterface $form_state, $context) {
+function localgov_blogs_field_widget_complete_form_alter(&$field_widget_complete_form, FormStateInterface $form_state, $context): void {
 
   // Check the blog channel select options.
   $field_definition = $context['items']->getFieldDefinition();
@@ -279,7 +279,7 @@ function localgov_blogs_localgov_roles_default(): array {
 /**
  * Configure scheduled transitions.
  */
-function localgov_blogs_configure_scheduled_transitions() {
+function localgov_blogs_configure_scheduled_transitions(): void {
   // Configure scheduled transitions for alert banners.
   $scheduled_transitions_config = \Drupal::service('config.factory')->getEditable('scheduled_transitions.settings');
   $bundles = $scheduled_transitions_config->get('bundles');

src/EventSubscriber/PageHeaderSubscriber.php

@@ -16,7 +16,7 @@ class PageHeaderSubscriber implements EventSubscriberInterface {
   /**
    * {@inheritdoc}
    */
-  public static function getSubscribedEvents() {
+  public static function getSubscribedEvents(): array {
     return [
       PageHeaderDisplayEvent::EVENT_NAME => ['setPageHeader', 0],
     ];
@@ -25,7 +25,7 @@ public static function getSubscribedEvents() {
   /**
    * Hide page header block.
    */
-  public function setPageHeader(PageHeaderDisplayEvent $event) {
+  public function setPageHeader(PageHeaderDisplayEvent $event): void {
     if ($event->getEntity() instanceof Node &&
           ($event->getEntity()->bundle() == 'localgov_blog_post')
       ) {

tests/src/Functional/BlogCreationTest.php

@@ -45,7 +45,7 @@ protected function setUp(): void {
   /**
    * Test the blog post create form.
    */
-  public function testBlogChannelSelection() {
+  public function testBlogChannelSelection(): void {
 
     // Check no blog channel message.
     $this->drupalGet('/node/add/localgov_blog_post');

tests/src/Functional/PrevNextBlockTest.php

@@ -54,7 +54,7 @@ protected function setUp(): void {
   /**
    * Test the Previous / Next Navigation block.
    */
-  public function testPrevNextBlock() {
+  public function testPrevNextBlock(): void {
 
     // Create channel_one.
     $channel_one = $this->createNode([
@@ -121,7 +121,7 @@ public function testPrevNextBlock() {
   /**
    * Test the correct next / prev posts appear when all the same date.
    */
-  public function testPrevNextWithSameDate() {
+  public function testPrevNextWithSameDate(): void {
 
     // Create a channel.
     $channel = $this->createNode([