HTML sitemap lists unpublished pages
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 82/100
Direzione di ricerca
Inizia dal blocco delle pagine in src/Web/Grand.Web/Features/Handlers/Common/GetSitemapHandler.cs e confronta i suoi filtri con quelli del gestore della sitemap XML in src/Business/Grand.Business.Messages/Commands/Handlers/Common/GetSitemapXMLCommandHandler.cs. Controlla il comportamento correlato delle pagine in Features/Handlers/Pages/GetPageBlockHandler.cs. Il lavoro è completato quando /sitemap esclude le pagine non pubblicate mantenendo i filtri sitemap e data indicati; verifica il comportamento con i test pertinenti o riproducendolo con una pagina non pubblicata.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
What happens
The storefront HTML sitemap (/sitemap) lists CMS pages that are not published. The title of an unpublished page is shown publicly, and its link leads nowhere because the page itself checks Published and does not render.
Steps to reproduce
- In the admin panel create a page, tick "Include in sitemap" and leave "Published" unticked.
- Open
/sitemapon the storefront. - The page is listed under the pages section; following the link does not show the page.
Cause
src/Web/Grand.Web/Features/Handlers/Common/GetSitemapHandler.cs (pages block, around lines 110-114) takes pages from IPageService.GetAllPages(storeId), which does not filter on Published, and then filters only on IncludeInSitemap and the start/end dates.
The XML sitemap already does this correctly: src/Business/Grand.Business.Messages/Commands/Handlers/Common/GetSitemapXMLCommandHandler.cs (around line 332) filters on t.Published && t.IncludeInSitemap && .... The page view enforces Published in Features/Handlers/Pages/GetPageBlockHandler.cs.
Expected behaviour
Only published pages with "Include in sitemap" ticked (and within their start/end dates) appear on /sitemap, consistent with the XML sitemap.
- Lingua principale
- C#
- Stelle
- 1.3k
- Fork
- 549
- Merge medio
- 7h 58m
- PR unite (30g)
- 52
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di grandnode/grandnode2
-
bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 65/100
grandnode/grandnode2#881 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
grandnode/grandnode2#880 ·
I maintainer di solito rispondono entro 1 giorno
-
Please add the offer systemAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
grandnode/grandnode2#849 ·
I maintainer di solito rispondono entro 1 giorno
-
grandnode/grandnode2#499 ·
I maintainer di solito rispondono entro 1 giorno
-
grandnode/grandnode2#410 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di grandnode/grandnode2
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
stryker-mutator/stryker-net#3892 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
MobiFlight/MobiFlight-Connector#3419 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Kryptos-FR/MarkView.Avalonia#105 ·
I maintainer di solito rispondono entro 1 giorno
-
[辞書]Aperta提案 辞書
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
microsoft/fluentui-blazor#5410 ·
I maintainer di solito rispondono entro 1 giorno