Introduce annotation to reset breadcrumb trail about apybreadcrumbtrailbundle HOT 4 CLOSED

IMHO resetting the trail by passing an emtpy breadcrumb is an hard to understand workaround that should be deprecated asap, and be removed v2 of this bundle.

Introducing an #[ResetBreadcrumbTrail] attribute would be a way more explicit solution.

Let me know what you think, and if you are able to help in introducing the annotation and deprecating the current behavior.

from apybreadcrumbtrailbundle.

I'm not sure creating a new annotation will be easier to use.
I would prefer the #[Breadcrum] attribute with a reset param.
#[Breadcrumb(reset: true)]

And of course i can help with that

from apybreadcrumbtrailbundle.

Not having the default value on title anymore is a BC break though, so will fix that on the v1.x branch.

Resetting the trail is an entirely different concern as adding a breadcrumb and parsing it's values. The code related to their behavior thereby should explicitly be splitted, which is not possible when such resetting behavior would get mixed with the existing #[Breadcrumb] attribute.

An example benefit, your IDE will allow you to easily find a project's usages of the ResetBreadcrumbTrail annotation.

from apybreadcrumbtrailbundle.

@phillamarmotte https://github.com/APY/APYBreadcrumbTrailBundle/releases/tag/v1.8.0 was tagged with a fix for the issue

from apybreadcrumbtrailbundle.