sulu / SuluArticleBundle

Bundle for managing localized content-rich entities like blog-posts in the Sulu content management system
MIT License
52 stars 77 forks source link

Refresh routePath for page_tree_route based Article #667

Open Olivier127 opened 3 months ago

Olivier127 commented 3 months ago
Q A
Bug? yes
New Feature? no
SuluArticleBundle Version 2.5.3
Sulu Version 2.5.13

Actual Behavior

The "refresh URL" button to update the slug of an article using a routePath based on page_tree_route calls an unknown route.

Expected Behavior

A correctly updated slug.

Steps to Reproduce

Create an article with the page_tree.txt

put "page test" inside title section. clic on the slug and put "page-test" inside clic on refresh url

le browser call http://localhost/admin/undefined with this payload

{
    "parts": {
        "title": "page test"
    },
    "resourceKey": "articles",
    "locale": "fr",
    "id": null,
    "entityClass": null,
    "routeSchema": "/{implode('-', object)}"
}

Possible Solutions

This page should have the same behavior as an article using the routePath of type route and calling the URL admin/api/routes?action=generate with the correct parameters.

alexander-schranz commented 3 weeks ago

Did it work on 2.5.12? Then it might be related to changes in https://github.com/sulu/sulu/pull/7271. /cc @wachterjohannes

Olivier127 commented 3 weeks ago

@alexander-schranz no error with the "route" type. Only with the "page_tree_route".

I have test with sulu 2.5.19 and sulu article 2.6.0

See this sceen shoot

image