Symfony-docs: Missing explanation of the possible priority values

Created on 23 Oct 2020  路  4Comments  路  Source: symfony/symfony-docs

For example https://symfony.com/doc/current/service_container/tags.html#tagged-services-with-priority
There we can see example of setting priority. But no description possible values of priority and how this value processing, and how it affects order of tagged elements.

But instead of previous article here this description presents.
https://symfony.com/doc/current/reference/dic_tags.html

Also many components has lack of different examples and use cases.

Most helpful comment

Thanks @x8core. Please continue this conversation in a respectful matter. It's free time spend here for all of us, so let's not blame each other when things could be done better.


Thanks for the examples @javiereguiluz, I agree we should add that here as well. @x8core are you open to create a PR fixing this?

All 4 comments

Hi @x8core. The documentation is available on https://github.com/symfony/symfony-docs . Feel free to submit PRs, I'm happy to review and merge them.

I'm going to close this issue because it's the wrong repository and too broad to be actionable.

@x8core thanks for reporting this. In Symfony Docs we usually explain this value. For example, in this page -> https://symfony.com/doc/current/reference/dic_tags.html

The priority is optional and its value is a positive or negative integer that defaults to 0. The higher the number, the earlier that warmers are executed.

In this page -> https://symfony.com/doc/current/event_dispatcher.html

There is an optional attribute for the kernel.event_listener tag called priority, which is a positive or negative integer that defaults to 0 and it controls the order in which listeners are executed (the higher the number, the earlier a listener is executed). 

In this page -> https://symfony.com/doc/current/routing.html#priority-parameter

The priority parameter expects an integer value. Routes with higher priority are sorted before routes with lower priority. The default value when it is not defined is 0.

So, we should add something similar in the page that you mentioned. If you find any other missing place, please tell us.

I'm going to try to transfer this issue to symfony/symfony-docs to continue the discussion there. Thanks!

Thanks @x8core. Please continue this conversation in a respectful matter. It's free time spend here for all of us, so let's not blame each other when things could be done better.


Thanks for the examples @javiereguiluz, I agree we should add that here as well. @x8core are you open to create a PR fixing this?

Was this page helpful?
0 / 5 - 0 ratings

Related issues

weaverryan picture weaverryan  路  3Comments

wouterj picture wouterj  路  4Comments

ManInTheBox picture ManInTheBox  路  4Comments

Kwadz picture Kwadz  路  3Comments

dannyvw picture dannyvw  路  3Comments