USAGOV-2197 phpstan ssg postprocessing

web/modules/custom/usagov_ssg_postprocessing/src/Controller/SsgStatController.php

@@ -15,7 +15,10 @@ public function __construct(
   ) {
   }
 
-  public function content() {
+  /**
+   * @return array<mixed>
+   */
+  public function content(): array {
     $date = $this->state()->get('ssg_stat_date');
     $msg = $this->state()->get('ssg_stat_msg');
 
@@ -30,11 +33,13 @@ public function content() {
     return ['#markup' => $markup, '#cache' => ['max-age' => 0]];
   }
 
-  /*
-   * This is a utility use in order to test what the WAF and proxies will do with wait-timeouts.
-   * See ticket USAGOV-1927.
+  /**
+   * This is a utility. Use in order to test what the WAF and proxies will do with wait-timeouts
+   * See ticket USAGOV-1927
+   *
+   * @return array<mixed>
    */
-  public function siteLagTest() {
+  public function siteLagTest(): array {
 
     $request = $this->requestStack->getCurrentRequest();
     $waitParam = $request->query->get('wait');

web/modules/custom/usagov_ssg_postprocessing/src/Data/PublishedPagesRow.php

@@ -45,6 +45,9 @@ public function __construct(
     public readonly string $pageLanguage,
   ) {}
 
+  /**
+   * @param array<mixed> $data
+   */
   private static function getHierarchy(array $data): int {
     $texts = array_filter($data, fn($key) => str_starts_with($key, 'Taxonomy_URL_'), ARRAY_FILTER_USE_KEY);
     return count(array_unique($texts));

web/modules/custom/usagov_ssg_postprocessing/src/Drush/Commands/PublishedPagesCommands.php

@@ -5,6 +5,7 @@
 use Drupal\Core\Breadcrumb\ChainBreadcrumbBuilderInterface;
 use Drupal\Core\Config\ConfigFactoryInterface;
 use Drupal\Core\Entity\EntityTypeManagerInterface;
+use Drupal\Core\Language\LanguageManager;
 use Drupal\Core\Language\LanguageManagerInterface;
 use Drupal\Core\Routing\RouteMatch;
 use Drupal\Core\Routing\RouteMatchInterface;
@@ -30,6 +31,9 @@
  */
 final class PublishedPagesCommands extends DrushCommands {
 
+  /**
+   * @var array<string> $csvHeader
+   */
   private array $csvHeader = [
     "Hierarchy Level",
     "Page Type",
@@ -71,7 +75,7 @@ public function __construct(
     parent::__construct();
   }
 
-  public static function create(ContainerInterface $container) {
+  public static function create(ContainerInterface $container): static {
     return new static(
       entityTypeManager: $container->get('entity_type.manager'),
       configFactory: $container->get('config.factory'),
@@ -93,7 +97,7 @@ public static function create(ContainerInterface $container) {
     name: 'usagov_ssg_postprocessing:published-csv',
     description: 'Usage description')
   ]
-  public function publishedCsv($outfile) {
+  public function publishedCsv(mixed $outfile): void {
     $this->output()->writeln('<info>Publishing CSV to ' . $outfile . '</info>');
 
     $out = fopen($outfile, 'w');
@@ -113,7 +117,7 @@ public function publishedCsv($outfile) {
     fclose($out);
   }
 
-  protected function saveNodeRows($out): void {
+  protected function saveNodeRows(mixed $out): void {
     $nids = $this->entityTypeManager
       ->getStorage('node')
       ->getQuery()
@@ -168,7 +172,7 @@ protected function saveNodeRows($out): void {
     }
   }
 
-  protected function saveNodeRow($out, $node, $row): void {
+  protected function saveNodeRow(mixed $out, Node $node, mixed $row): void {
 
     $row = array_map(fn($col) => trim($col), $row);
     fputcsv($out, $row);
@@ -187,7 +191,7 @@ protected function saveNodeRow($out, $node, $row): void {
     }
   }
 
-  protected function saveWizardRows($out): void {
+  protected function saveWizardRows(mixed $out): void {
     $tids = $this->entityTypeManager
       ->getStorage('taxonomy_term')
       ->getQuery()
@@ -263,6 +267,10 @@ protected function getWizardRow(Term $wizard): PublishedPagesRow {
     return PublishedPagesRow::datalayerForWizard($data, $wizard, $baseURL);
   }
 
+  /**
+   * @param array<mixed> $data
+   * @return array<mixed>
+   */
   private function alterDatalayer(array $data): array {
     // Let other modules add to the datalayer payload.
     $datalayerEvent = new DatalayerAlterEvent($data);

web/modules/custom/usagov_ssg_postprocessing/src/EventSubscriber/PagerPathSubscriber.php

@@ -21,7 +21,7 @@ class PagerPathSubscriber implements EventSubscriberInterface {
    * @param \Drupal\tome_static\Event\ModifyDestinationEvent $event
    *   The event.
    */
-  public function modifyDestination(ModifyDestinationEvent $event) {
+  public function modifyDestination(ModifyDestinationEvent $event): void {
     $destination = $event->getDestination();
     $new_destination = $this->modifyUrl($destination);
     if ($destination != $new_destination) {
@@ -35,7 +35,7 @@ public function modifyDestination(ModifyDestinationEvent $event) {
    * @param \Drupal\tome_static\Event\ModifyHtmlEvent $event
    *   The event.
    */
-  public function modifyHtml(ModifyHtmlEvent $event) {
+  public function modifyHtml(ModifyHtmlEvent $event): void {
     $html = $event->getHtml();
     $path = $event->getPath();
 

web/modules/custom/usagov_ssg_postprocessing/src/EventSubscriber/TomeEventSubscriber.php

@@ -63,7 +63,7 @@ public function __construct(EntityTypeManagerInterface $entity_type_manager, Lan
    * @param \Drupal\tome_static\Event\CollectPathsEvent $event
    *   The collect paths event.
    */
-  public function excludeDirectories(CollectPathsEvent $event) {
+  public function excludeDirectories(CollectPathsEvent $event): void {
     $excluded_directories = self::getExcludedDirectories();
     $paths = $event->getPaths(TRUE);
     foreach ($paths as $path => $metadata) {
@@ -117,10 +117,10 @@ public function excludeDirectories(CollectPathsEvent $event) {
   /**
    * Returns per-site excluded directory paths.
    *
-   * @return array
+   * @return array<mixed>
    *   An array of excluded paths.
    */
-  public static function getExcludedDirectories() {
+  public static function getExcludedDirectories(): array {
     $excluded_paths = [];
     $site_paths = Settings::get('usagov_tome_static_path_exclude_directories', []);
     if (is_array($site_paths)) {
@@ -138,7 +138,7 @@ public static function getExcludedDirectories() {
    * @param \Drupal\tome_static\Event\ModifyHtmlEvent $event
    *   The event.
    */
-  public function modifyHtml(ModifyHtmlEvent $event) {
+  public function modifyHtml(ModifyHtmlEvent $event): void {
     $html = $event->getHtml();
     $html5 = new HTML5();
 
@@ -233,6 +233,9 @@ public function addAgencyIndexes(CollectPathsEvent $event): void {
 
   }
 
+  /**
+   * @return array<mixed>
+   */
   private function getLetters(ViewExecutable $view): array {
     $view->execute();
     $letters = [];

web/modules/custom/usagov_ssg_postprocessing/usagov_ssg_postprocessing.module

@@ -4,18 +4,20 @@ use Drupal\Core\Entity\Display\EntityViewDisplayInterface;
 use Drupal\Core\Entity\EntityInterface;
 
 /**
- * Implements hook_entity_view_alter().
+ * Implements hook_entity_view_alter()
  *
  * From Drupal.org -> "If a module wishes to act on the rendered HTML of the
  * ntity rather than the structured content array, it may use this hook to add
  * a #post_render callback."
+ *
+ * @param array<mixed> &$build
 */
-function usagov_ssg_postprocessing_entity_view_alter(array &$build, EntityInterface $entity, EntityViewDisplayInterface $display) {
+function usagov_ssg_postprocessing_entity_view_alter(array &$build, EntityInterface $entity, EntityViewDisplayInterface $display): void {
   usagov_ssg_postprocessing_remove_shortlink($build);
   usagov_ssg_postprocessing_modify_canonical_link($build);
 }
 
-function usagov_ssg_postprocessing_remove_shortlink(array &$attachments) {
+function usagov_ssg_postprocessing_remove_shortlink(array &$attachments): void {
   if (!isset($attachments['#attached']['html_head_link'])) {
     return;
   }
@@ -27,7 +29,7 @@ function usagov_ssg_postprocessing_remove_shortlink(array &$attachments) {
   }
 }
 
-function usagov_ssg_postprocessing_modify_canonical_link(array &$attachments) {
+function usagov_ssg_postprocessing_modify_canonical_link(array &$attachments): void {
   if (!isset($attachments['#attached']['html_head_link'])) {
     return;
   }
@@ -44,10 +46,12 @@ function usagov_ssg_postprocessing_modify_canonical_link(array &$attachments) {
 }
 
 /**
- * Implements hook_page_attachments_alter.
- * We just want to add / to the alternate links on the home pages.
+ * Implements hook_page_attachments_alter
+ *
+ * @param array<mixed> &$attachments
+ * We just want to add / to the alternate links on the home pages
  */
-function usagov_ssg_postprocessing_page_attachments_alter(array &$attachments) {
+function usagov_ssg_postprocessing_page_attachments_alter(array &$attachments): void {
   if (!isset($attachments['#attached']['html_head_link'])) {
     return;
   }
@@ -70,29 +74,29 @@ function usagov_ssg_postprocessing_page_attachments_alter(array &$attachments) {
 /**
 * Static Site Generation Toggle State Form ID
 */
-function usagov_ssg_postprocessing_get_static_state_form_id() {
+function usagov_ssg_postprocessing_get_static_state_form_id(): string {
   return 'toggle_static_site_generation_form';
 }
 
 /**
 * Static Site Generation Toggle State Form Button Name
 */
-function usagov_ssg_postprocessing_get_static_state_button_name() {
+function usagov_ssg_postprocessing_get_static_state_button_name(): string {
   return 'confirm_toggle';
 }
 
 /**
  * Toggle State variable name for sset/sget/sdel
  */
-function usagov_ssg_postprocessing_get_static_state_var() {
+function usagov_ssg_postprocessing_get_static_state_var(): string {
   return 'usagov.tome_run_disabled';
 }
 
 /**
   * Implements hook_form_alter().
   * Static Site Generation form text updates.
   */
-function usagov_ssg_postprocessing_form_alter(&$form, &$form_state, $form_id) {
+function usagov_ssg_postprocessing_form_alter(mixed &$form, mixed &$form_state, string $form_id): void {
   if ($form_id == usagov_ssg_postprocessing_get_static_state_form_id()) {
     $toggle_state = \Drupal::state()->get(usagov_ssg_postprocessing_get_static_state_var()) ? 1 : 0;
     $form_state->setValue(usagov_ssg_postprocessing_get_static_state_button_name(), $toggle_state ? TRUE : FALSE);