thedevdojo/voyager

MenuItemPolicy fails with edit page urls

Open

#3443 aperta il 3 ago 2018

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)PHP (2652 fork)batch import
help wantedtype:enhancement

Metriche repository

Star
 (11.817 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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)

Guida contributor

Direzione di ricerca
Esamina la classe MenuItemPolicy e traccia come lo slug viene estratto dall'URL della route. Concentrati sulla logica che estrae lo slug quando l'URL contiene più segmenti o parametri. Scrivi un test per confermare il bug e poi modifica l'estrazione dello slug per identificare correttamente lo slug della voce di menu.
Tech stack
phplaravel
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PHPLaravel
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75