Leantime/leantime

Move all repo calls from controllers to services

Open

#1,532 创建于 2023年3月31日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)PHP (957 fork)batch import
Help Wantedgood-first-issue

仓库指标

Star
 (9,827 star)
PR 合并指标
 (平均合并 12小时 22分钟) (30 天内合并 61 个 PR)

描述

The controllers should not be calling the repos directly. Instead they should call a service method. The service layer should be the layer calling the repo.

As such the task is to go through all controllers and update repository calls to be service calls. If the method does not exist in the service a wrapper service method should be created.

贡献者指南

研究方向
识别项目中所有控制器文件。对于每个直接调用仓库（例如 `$this >repo >method()`）的控制器方法，找到对应的服务类（如果有的话）。如果服务已经有一个包装了仓库调用的方法，将直接仓库调用替换为服务方法。如果不存在这样的服务方法，在适当的服务中创建一个新方法，委托给仓库，然后更新控制器以使用该服务。确保服务被注入到控制器中。更改后测试应用程序是否仍然工作。
技术栈
php
领域
backend
议题类型
重构
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
3
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-2 天
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
PHPGit
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70