Jingga/oms-News
1
0
Fork 0
mirror of https://github.com/Karaka-Management/oms-News.git synced 2026-02-17 08:58:42 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

impl. todos or move to Project.md

Browse Source
This commit is contained in:
Dennis Eichhorn 2021-09-27 23:14:01 +02:00
parent a11aa945ee
commit f41638aa1d
2 changed files with 1 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
CONTRIBUTING.md
View File

@ -12,14 +12,7 @@ If you have a good idea for improvement feel free to create a new issue with all
### Issues ### Issues
Feel free to grab any open issue implement it and create a new pull request. Most issues can be found in the code and have the following structure: Feel free to grab any open issue implement it and create a new pull request. Most issues can be found in the `Project.md` file in the `Docs` repository.
```php
/**
* @todo Orange-Management/Orange-Management#ISSUE_NUMBER [d:difficulty]
* Description for the issue
*/
```
The issue information can be used to provide additional information such as priority, difficulty and type. For your first issue try to find a issue marked `[d:first]` or `[d:beginner]`. The issue information can be used to provide additional information such as priority, difficulty and type. For your first issue try to find a issue marked `[d:first]` or `[d:beginner]`.

5
Models/NewsArticleMapper.php
View File

@ -27,11 +27,6 @@ use phpOMS\DataStorage\Database\DataMapperAbstract;
* @license OMS License 1.0 * @license OMS License 1.0
* @link https://orange-management.org * @link https://orange-management.org
* @since 1.0.0 * @since 1.0.0
*
* @todo Orange-Management/oms-News#???
* Too complicated select.
* I think the default getAll etc. is too complicated and has too many joins which are not really required.
* Check and fix!
*/ */
final class NewsArticleMapper extends DataMapperAbstract final class NewsArticleMapper extends DataMapperAbstract
{ {