phpolar / phpolar / 5256022945
91%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 12
Run time 4s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
5256022945

push

github

web-flow 
refactor: simplify project architecture (#180)

* refactor: simplify project architecture

The project namespace hierarchy should be simplified.  The intent of some of the namespaces and classes was not always clear.

BREAKING CHANGE: All classes in the `Routing` namespace have been moved to the `Http` namespace.  `ModelParamResolver` has been renamed to `ModelResolver`.  The `ModelResolver` now takes the parsed request body as the second constructor argument instead of the server request.  The `PrimaryHandler` has been renamed to `MiddlewareQueueRequestHandler`.  Issue #174.

* refactor: add registered routes in dependency injection configuration

The app is already using dependency injection to obtain the `RouteRegistry`.  Removing the need to call `useRoutes` simplifies the application bootstrap process.

BREAKING CHANGE: Add the `RouteRegistry` in a dependency injection configuration file instead of calling `App::useRoutes`.

* test: fix test and generate docs

* refactor: move the model namespace into it's own project

Moving the `Model` namespace into it's own project will simplify the entire codebase.  Users of the framework now have the option to use any "Model" library they choose or none at all.

BREAKING CHANGE: All classes in the `Model` namespace have been moved to the `phpolar/model` project.  An implementation of `ModelResolverInterface` MUST added to the dependency injection configuration.  Closes #174.

---------

Co-authored-by: Eric Fortmeyer <efortmeyer@bethel.jw.org>

89 of 90 relevant lines covered (98.89%)

6.66 hits per line

Jobs
ID Job ID Ran Files Coverage
1 5256022945.1 0
98.89
 GitHub Action Run
Source Files on build 5256022945
Detailed source file information is not available for this build.