thedevdojo/voyager

MenuItemPolicy fails with edit page urls

Open

#3.443 aberto em 3 de ago. de 2018

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)PHP (11.817 stars) (2.652 forks)batch import
help wantedtype:enhancement

Description

Version information

  • Laravel: v5.6
  • Voyager: v1.1.3
  • PHP: 7.2

Description

When a route with parameters or a static url which include more than 1 segment added to admin menu, policy fails to check permission because slug fetched wrong (like pages1edit instead of pages)

Guia do colaborador

Pilha de tecnologia
php
Domain
backendauthorization
Tipo Issue
bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 hours
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
PHPLaravelVoyager admin
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35
Direção de pesquisa
Investigate the MenuItemPolicy class (likely in app/Policies or within the Voyager package) to understand how the slug is extracted from the admin menu URL. Focus on the method that splits the URL segments; it may be incorrectly concatenating segments when there are multiple segments. Check the Voyager repository for any similar bug fixes or comments. Also review the issue comments for any additional context.