jaumarar commented 3 years ago

I think allowing multiple middlewares on the same route could bring more possibilities to the routes and keeping the code clean if we want to call multiple functions passing through the parameters in order to each middleware.

Features

Multiple middlewares on same route
Calling the middlewares in order
Passing through the route and parameters to each middleware to modify the values
Avoiding the middlewares that have a non expected result

codecov-commenter commented 3 years ago

Codecov Report

Merging #6 into master will not change coverage. The diff coverage is 100.00%.

@@             Coverage Diff             @@
##              master        #6   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity       112       117    +5     
===========================================
  Files             12        12           
  Lines            263       272    +9     
===========================================
+ Hits             263       272    +9

Impacted Files	Coverage Δ	Complexity Δ
Mezon/Router/UrlParser.php	`100.00% <100.00%> (ø)`	`51.00 <0.00> (+5.00)`

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 41a706a...e6f593a. Read the comment docs.

alexdodonov commented 3 years ago

@jaumarar thanks for this pull request ) and especially for unit-tests for your patch )

alexdodonov / mezon-router

Allow multiple middlewares on same route #6

Codecov Report