Retornando códigos HTTP alternativos para o nó não publicado

8

Estou tentando retornar a página 404 em vez da resposta 403 para nós não publicados no Drupal 8.

Testei o assinante de resposta do kernel , mas achei que o código que eu estava usando alteraria apenas o código de status para 404 de 403, e não exibia a página 404. Então, talvez alguém possa me mostrar como gerar um objeto de resposta de página 404 lá?

Este é o código que eu estava usando:

class ResponseSubscriber implements EventSubscriberInterface {

  /**
   * {@inheritdoc}
   */
  public static function getSubscribedEvents() {
    return [KernelEvents::RESPONSE => [['alterResponse']]];
  }

  /**
   * Change status code to 404 from 403 if page is an unpublished node.
   *
   * @param FilterResponseEvent $event
   *   The route building event.
   */
  public function alterResponse(FilterResponseEvent $event) {
    if ($event->getResponse()->getStatusCode() == 403) {
      /** @var \Symfony\Component\HttpFoundation\Request $request */
      $request = $event->getRequest();
      $node = $request->attributes->get('node');
      if ($node instanceof Node && !$node->isPublished()) {
        $response = $event->getResponse();
        // This changes the code, but doesn't return a 404 page.
        $response->setStatusCode(404);

        $event->setResponse($response);
      }
    }
  }

}

Finalmente, resolvi remover completamente esse assinante de resposta e usei hook_node_access assim:

use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Drupal\Core\Access\AccessResult;

function unpublished_404_node_access(\Drupal\node\NodeInterface $node, $op, \Drupal\Core\Session\AccountInterface $account) {

  if ($op == 'view' && !$node->isPublished()) {
    if (\Drupal::moduleHandler()->moduleExists('workbench_moderation') && $account->hasPermission('view any unpublished content')) {
      return AccessResult::neutral();
    }
    elseif (\Drupal::routeMatch()->getRouteName() == 'entity.node.canonical' && \Drupal::routeMatch()->getRawParameter('node') == $node->id()) {
      throw new NotFoundHttpException();
      return AccessResult::neutral();
    }
  }

  return AccessResult::neutral();
}

Isso parece estar alinhado com várias respostas neste site para o Drupal 7. Mas eu queria ver se alguém tem uma maneira melhor de fazer isso com um assinante KernelEvent, em vez de hook_node_access. Parece que o que eu quero fazer é testar se um nó está retornando um 403 e gerar uma nova resposta com a página 404 e o código de status 404. Não sei bem como fazer isso.

oknate
fonte

Respostas:

6

Você pode tentar fazer isso anteriormente em uma exceção, em vez de um assinante de resposta. Estenda HttpExceptionSubscriberBase, então você precisa de menos código para fazer isso. Em seguida, substitua o 403 por uma exceção 404 pelo método$event->setException()

/src/EventSubscriber/Unpublished404Subscriber.php

<?php

namespace Drupal\mymodule\EventSubscriber;

use Drupal\Core\EventSubscriber\HttpExceptionSubscriberBase;
use Symfony\Component\HttpKernel\Event\GetResponseForExceptionEvent;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;

class Unpublished404Subscriber extends HttpExceptionSubscriberBase {

  protected static function getPriority() {
    // set priority higher than 50 if you want to log "page not found"
    return 0;
  }

  protected function getHandledFormats() {
    return ['html'];
  }

  public function on403(GetResponseForExceptionEvent $event) {
    $request = $event->getRequest();
    if ($request->attributes->get('_route') == 'entity.node.canonical') {
      $event->setException(new NotFoundHttpException());
    }
  }

}

mymodule.services.yml:

services:
  mymodule.404:
    class: Drupal\mymodule\EventSubscriber\Unpublished404Subscriber
    arguments: []
    tags:
      - { name: event_subscriber }

Isso substitui todas as 403 exceções para rotas de nó canônico. Você pode obter o objeto do nó $request->attributes->get('node')se desejar verificar se é realmente porque o nó não está publicado.

4k4
fonte
Obrigado, eu testei e funciona muito bem! Este é exatamente o tipo de coisa que eu estava procurando.
oknate