Laclic
/
Souke
ウォッチ 5
スター 0
フォーク 1
コード 課題 1 プルリクエスト 0 リリース 0 Wiki アクティビティ
ソースを参照

Essai architecture refactoring services

refactoring
Guillaume 1年前
親
7ef2670718
コミット
208bf17468
11個のファイルの変更、14055行の追加、229行の削除
分割表示 差分情報を表示
  1. +19
    -0
      common/components/BusinessLogic.php
  2. +6
    -1
      common/config/main.php
  3. +19
    -0
      common/containers/ProducerContainer.php
  4. +19
    -0
      common/containers/UserContainer.php
  5. +5
    -32
      common/models/Producer.php
  6. +18
    -0
      common/repositories/ProducerRepository.php
  7. +8
    -0
      common/repositories/UserRepository.php
  8. +92
    -0
      common/services/ProducerService.php
  9. +44
    -0
      common/services/UserService.php
  10. +186
    -196
      frontend/models/SignupForm.php
  11. +13639
    -0
      producer/runtime/logs/app.log

+ 19
- 0
common/components/BusinessLogic.php ファイルの表示

@@ -0,0 +1,19 @@
<?php

namespace common\components;

use common\containers\UserContainer;
use common\containers\ProducerContainer;

class BusinessLogic
{
public function getUserContainer()
{
return new UserContainer();
}

public function getProducerContainer()
{
return new ProducerContainer();
}
}

+ 6
- 1
common/config/main.php ファイルの表示

@@ -36,6 +36,8 @@
* termes.
*/

use common\components\BusinessLogic;

$serverName = $_SERVER['SERVER_NAME'];

return [
@@ -116,7 +118,10 @@ return [
'enableStrictParsing' => false,
'rules' => [
],
]
],
'logic' => function() {
return new BusinessLogic();
}
],
'language' => 'fr-FR',
];

+ 19
- 0
common/containers/ProducerContainer.php ファイルの表示

@@ -0,0 +1,19 @@
<?php

namespace common\containers;

use common\repositories\ProducerRepository;
use common\services\ProducerService;

class ProducerContainer
{
public function getService()
{
return new ProducerService();
}

public function getRepository()
{
return new ProducerRepository();
}
}

+ 19
- 0
common/containers/UserContainer.php ファイルの表示

@@ -0,0 +1,19 @@
<?php

namespace common\containers;

use common\repositories\UserRepository;
use common\services\UserService;

class UserContainer
{
public function getService()
{
return new UserService();
}

public function getRepository()
{
return new UserRepository();
}
}

+ 5
- 32
common/models/Producer.php ファイルの表示

@@ -456,6 +456,11 @@ class Producer extends ActiveRecordCommon
];
}

public function addUser($idUser, $idProducer) {
$producerContainer = Yii::$app->logic->getProducerContainer();
$producerContainer->getService()->addUser($idUser, $idProducer);
}

/**
* Retourne la liste des établissements pour l'initialisation d'une liste
* sélective.
@@ -694,38 +699,6 @@ class Producer extends ActiveRecordCommon
}
}

/**
* Lie un utilisateur à un producteur.
*
* @param integer $id_user
* @param integer $id_producer
* @return UserProducer
*/
public static function addUser($idUser, $idProducer, $bookmark = 1)
{
$userProducer = UserProducer::searchOne([
'user_producer.id_user' => $idUser,
'user_producer.id_producer' => $idProducer
]);

if (!$userProducer) {
$newUserProducer = new UserProducer;
$newUserProducer->id_producer = $idProducer;
$newUserProducer->id_user = $idUser;
$newUserProducer->credit = 0;
$newUserProducer->active = 1;
$newUserProducer->bookmark = (int)$bookmark;
$newUserProducer->save();
} else {
if (!$userProducer->active) {
$userProducer->active = 1;
$userProducer->save();
}
}

return $userProducer;
}

public function getSpecificDelays()
{
$array = [];

+ 18
- 0
common/repositories/ProducerRepository.php ファイルの表示

@@ -0,0 +1,18 @@
<?php

namespace common\repositories;

use common\models\Producer;

class ProducerRepository
{
public function getOneById($id)
{
return Producer::find()->where(['id' => $id])->one();;
}

public function getOneBySlug($slug)
{
return Producer::findOne($slug);
}
}

+ 8
- 0
common/repositories/UserRepository.php ファイルの表示

@@ -0,0 +1,8 @@
<?php

namespace common\repositories;

class UserRepository
{

}

+ 92
- 0
common/services/ProducerService.php ファイルの表示

@@ -0,0 +1,92 @@
<?php

namespace common\services;

use common\helpers\Password;
use common\models\Producer;
use common\models\UserProducer;

class ProducerService
{
public function createInstance()
{
$producer = new Producer;

$producer->order_deadline = 20;
$producer->order_delay = 1;

return $producer;
}

public function init($producer)
{
$this->initSlug($producer);
$this->initCode($producer);
}

public function initSlug($producer)
{
$producerRepository = Yii::$app->logic->getProducerContainer()->getRepository();
$cptSlug = 0 ;
do {
$slug = \common\helpers\Url::slugify($producer->name) ;
if($cptSlug) {
$slug .= $cptSlug ;
}
$producer->slug = $slug ;
$cptSlug ++ ;
} while($producerRepository->getOneBySlug($producer->slug)) ;
}

public function initCode($producer)
{
$producer->code = Password::generate();
}

public function sendEmailNewProducer($producer)
{
Yii::$app->mailer->compose(
[
'html' => 'new-producer-html',
'text' => 'new-producer-text'
], [
'producer' => $producer
])
->setTo(Yii::$app->params['adminEmail'])
->setFrom([Yii::$app->params['adminEmail'] => 'distrib'])
->setSubject('[Opendistrib] Nouveau producteur')
->send();
}

/**
* Lie un utilisateur à un producteur.
*
* @param integer $id_user
* @param integer $id_producer
* @return UserProducer
*/
public static function addUser($idUser, $idProducer, $bookmark = 1)
{
$userProducer = UserProducer::searchOne([
'user_producer.id_user' => $idUser,
'user_producer.id_producer' => $idProducer
]);

if (!$userProducer) {
$newUserProducer = new UserProducer;
$newUserProducer->id_producer = $idProducer;
$newUserProducer->id_user = $idUser;
$newUserProducer->credit = 0;
$newUserProducer->active = 1;
$newUserProducer->bookmark = (int)$bookmark;
$newUserProducer->save();
} else {
if (!$userProducer->active) {
$userProducer->active = 1;
$userProducer->save();
}
}

return $userProducer;
}
}

+ 44
- 0
common/services/UserService.php ファイルの表示

@@ -0,0 +1,44 @@
<?php

namespace common\services;

use common\models\User;

class UserService
{
public function createInstance()
{
$user = new User();

return $user;
}

public function initPassword($user, $password)
{
$user->setPassword($this->password);
$user->generateAuthKey();
}

public function initProducer($user, $producer)
{
$user->id_producer = $producer->id;
$user->status = User::STATUS_PRODUCER;
}

public function sendEmailSignup($user, $producer)
{
Yii::$app->mailer->compose(
[
'html' => 'signup-html',
'text' => 'signup-text'
],
[
'user' => $user,
'producer' => $producer
])
->setTo($user->email)
->setFrom([Yii::$app->params['adminEmail'] => 'distrib'])
->setSubject('[Opendistrib] Inscription')
->send();
}
}

+ 186
- 196
frontend/models/SignupForm.php ファイルの表示

@@ -1,40 +1,40 @@
<?php

/**
Copyright distrib (2018)
contact@opendistrib.net
Ce logiciel est un programme informatique servant à aider les producteurs
à distribuer leur production en circuits courts.
Ce logiciel est régi par la licence CeCILL soumise au droit français et
respectant les principes de diffusion des logiciels libres. Vous pouvez
utiliser, modifier et/ou redistribuer ce programme sous les conditions
de la licence CeCILL telle que diffusée par le CEA, le CNRS et l'INRIA
sur le site "http://www.cecill.info".
En contrepartie de l'accessibilité au code source et des droits de copie,
de modification et de redistribution accordés par cette licence, il n'est
offert aux utilisateurs qu'une garantie limitée. Pour les mêmes raisons,
seule une responsabilité restreinte pèse sur l'auteur du programme, le
titulaire des droits patrimoniaux et les concédants successifs.
A cet égard l'attention de l'utilisateur est attirée sur les risques
associés au chargement, à l'utilisation, à la modification et/ou au
développement et à la reproduction du logiciel par l'utilisateur étant
donné sa spécificité de logiciel libre, qui peut le rendre complexe à
manipuler et qui le réserve donc à des développeurs et des professionnels
avertis possédant des connaissances informatiques approfondies. Les
utilisateurs sont donc invités à charger et tester l'adéquation du
logiciel à leurs besoins dans des conditions permettant d'assurer la
sécurité de leurs systèmes et ou de leurs données et, plus généralement,
à l'utiliser et l'exploiter dans les mêmes conditions de sécurité.
Le fait que vous puissiez accéder à cet en-tête signifie que vous avez
pris connaissance de la licence CeCILL, et que vous en avez accepté les
termes.
*/
* Copyright distrib (2018)
*
* contact@opendistrib.net
*
* Ce logiciel est un programme informatique servant à aider les producteurs
* à distribuer leur production en circuits courts.
*
* Ce logiciel est régi par la licence CeCILL soumise au droit français et
* respectant les principes de diffusion des logiciels libres. Vous pouvez
* utiliser, modifier et/ou redistribuer ce programme sous les conditions
* de la licence CeCILL telle que diffusée par le CEA, le CNRS et l'INRIA
* sur le site "http://www.cecill.info".
*
* En contrepartie de l'accessibilité au code source et des droits de copie,
* de modification et de redistribution accordés par cette licence, il n'est
* offert aux utilisateurs qu'une garantie limitée. Pour les mêmes raisons,
* seule une responsabilité restreinte pèse sur l'auteur du programme, le
* titulaire des droits patrimoniaux et les concédants successifs.
*
* A cet égard l'attention de l'utilisateur est attirée sur les risques
* associés au chargement, à l'utilisation, à la modification et/ou au
* développement et à la reproduction du logiciel par l'utilisateur étant
* donné sa spécificité de logiciel libre, qui peut le rendre complexe à
* manipuler et qui le réserve donc à des développeurs et des professionnels
* avertis possédant des connaissances informatiques approfondies. Les
* utilisateurs sont donc invités à charger et tester l'adéquation du
* logiciel à leurs besoins dans des conditions permettant d'assurer la
* sécurité de leurs systèmes et ou de leurs données et, plus généralement,
* à l'utiliser et l'exploiter dans les mêmes conditions de sécurité.
*
* Le fait que vous puissiez accéder à cet en-tête signifie que vous avez
* pris connaissance de la licence CeCILL, et que vous en avez accepté les
* termes.
*/

namespace frontend\models;

@@ -68,7 +68,7 @@ class SignupForm extends Model
public $code;
public $type;
// public $free_price ;
public $id_tax_rate_default ;
public $id_tax_rate_default;
public $verifyCode;

/**
@@ -80,7 +80,7 @@ class SignupForm extends Model
['email', 'filter', 'filter' => 'trim'],
['email', 'required', 'message' => 'Champs obligatoire'],
['email', 'email'],
['email', function($attribute, $params) {
['email', function ($attribute, $params) {
$email = $this->$attribute;

$userExist = User::searchOne([
@@ -88,7 +88,7 @@ class SignupForm extends Model
'email' => $email
]);

if($userExist) {
if ($userExist) {
$this->addError($attribute, 'Cet email est déjà utilisé.');
}
}],
@@ -99,107 +99,107 @@ class SignupForm extends Model
['password', 'string', 'min' => 6, 'tooShort' => 'Votre mot de passe doit contenir au moins 6 caractères'],
['is_producer', 'boolean'],
['cgv', 'boolean'],
['cgv', function($attribute, $params) {
$cgv = $this->$attribute;
['cgv', function ($attribute, $params) {
$cgv = $this->$attribute;

if ($this->option_user_producer == 'producer' && !$cgv) {
$this->addError($attribute, 'Vous devez accepter les conditions générales de vente pour vous inscrire.');
}
}],
['postcode', 'required', 'message' => 'Champs obligatoire', 'when' => function($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
if ($this->option_user_producer == 'producer' && !$cgv) {
$this->addError($attribute, 'Vous devez accepter les conditions générales de vente pour vous inscrire.');
}
}],
['postcode', 'required', 'message' => 'Champs obligatoire', 'when' => function ($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
return $('#option-producer').prop('checked') ;
}"],
['id_tax_rate_default', 'required', 'message' => 'Champs obligatoire', 'when' => function($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
['id_tax_rate_default', 'required', 'message' => 'Champs obligatoire', 'when' => function ($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
return $('#option-producer').prop('checked') ;
}"],
['city', 'required', 'message' => 'Champs obligatoire', 'when' => function($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
['city', 'required', 'message' => 'Champs obligatoire', 'when' => function ($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
return $('#option-producer').prop('checked') ;
}"],
['name_producer', 'string'],
['name_producer', 'required', 'message' => 'Champs obligatoire', 'when' => function($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
['name_producer', 'required', 'message' => 'Champs obligatoire', 'when' => function ($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
return $('#option-producer').prop('checked') ;
}"],
['type', 'string'],
['type', 'required', 'message' => 'Champs obligatoire', 'when' => function($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
['type', 'required', 'message' => 'Champs obligatoire', 'when' => function ($model) {
return $model->option_user_producer == 'producer';
}, 'whenClient' => "function (attribute, value) {
return $('#option-producer').prop('checked') ;
}"],
['siret', 'string'],
['siret', function($attribute, $params) {
['siret', function ($attribute, $params) {

$siret = $this->$attribute;
$siret = str_replace(' ', '', $siret);
$siret = $this->$attribute;
$siret = str_replace(' ', '', $siret);

if (strlen($siret) != 14) {
$this->addError($attribute, 'Le numéro SIRET doit contenir 14 caractères');
return;
}
if (strlen($siret) != 14) {
$this->addError($attribute, 'Le numéro SIRET doit contenir 14 caractères');
return;
}

if (!is_numeric($siret)) {
$this->addError($attribute, 'Le numéro SIRET ne doit contenir que des chiffres');
return;
}
if (!is_numeric($siret)) {
$this->addError($attribute, 'Le numéro SIRET ne doit contenir que des chiffres');
return;
}

// on prend chaque chiffre un par un
// si son index (position dans la chaîne en commence à 0 au premier caractère) est pair
// on double sa valeur et si cette dernière est supérieure à 9, on lui retranche 9
// on ajoute cette valeur à la somme totale

$sum = 0;
for ($index = 0; $index < 14; $index ++) {
$number = (int) $siret[$index];
if (($index % 2) == 0) {
if (($number *= 2) > 9)
$number -= 9;
}
$sum += $number;
// on prend chaque chiffre un par un
// si son index (position dans la chaîne en commence à 0 au premier caractère) est pair
// on double sa valeur et si cette dernière est supérieure à 9, on lui retranche 9
// on ajoute cette valeur à la somme totale

$sum = 0;
for ($index = 0; $index < 14; $index++) {
$number = (int)$siret[$index];
if (($index % 2) == 0) {
if (($number *= 2) > 9)
$number -= 9;
}
$sum += $number;
}

// le numéro est valide si la somme des chiffres est multiple de 10
if (($sum % 10) != 0)
$this->addError($attribute, 'Numéro SIRET invalide');
}],
// le numéro est valide si la somme des chiffres est multiple de 10
if (($sum % 10) != 0)
$this->addError($attribute, 'Numéro SIRET invalide');
}],
['id_producer', 'integer'],
['id_producer', function($attribute, $params) {
if ($this->id_producer) {
$producer = Producer::findOne($this->id_producer);
if (!$producer) {
$this->addError($attribute, 'Ce producteur n\'existe pas.');
}
}
}],
['id_producer', 'required', 'message' => 'Champs obligatoire', 'when' => function($model) {
return $this->option_user_producer == 'user' ;
}],
['code', 'required', 'message' => 'Champs obligatoire', 'when' => function($model) {
['id_producer', function ($attribute, $params) {
if ($this->id_producer) {
$producer = Producer::findOne($this->id_producer);
if ($producer) {
return strlen($producer->code);
} else {
return false;
if (!$producer) {
$this->addError($attribute, 'Ce producteur n\'existe pas.');
}
}],
['code', function($attribute, $params) {
$code = $this->$attribute;
$producer = Producer::findOne($this->id_producer);
}
}],
['id_producer', 'required', 'message' => 'Champs obligatoire', 'when' => function ($model) {
return $this->option_user_producer == 'user';
}],
['code', 'required', 'message' => 'Champs obligatoire', 'when' => function ($model) {
$producer = Producer::findOne($this->id_producer);
if ($producer) {
return strlen($producer->code);
} else {
return false;
}
}],
['code', function ($attribute, $params) {
$code = $this->$attribute;
$producer = Producer::findOne($this->id_producer);

if ($producer && strtolower(trim($code)) != strtolower(trim($producer->code))) {
$this->addError($attribute, 'Code incorrect');
}
}],
if ($producer && strtolower(trim($code)) != strtolower(trim($producer->code))) {
$this->addError($attribute, 'Code incorrect');
}
}],
// ['free_price', 'number'],
['id_tax_rate_default', 'exist',
'targetClass' => TaxRate::className(),
'targetAttribute' => ['id_tax_rate_default' => 'id']],
'targetClass' => TaxRate::className(),
'targetAttribute' => ['id_tax_rate_default' => 'id']],
['verifyCode', 'captcha', 'message' => 'Veuillez recopier le code de vérification'],
];
}
@@ -234,95 +234,30 @@ class SignupForm extends Model
*/
public function signup()
{
$userContainer = Yii::$app->logic->getUserContainer();
$producerContainer = Yii::$app->logic->getProducerContainer();

if ($this->validate()) {
$user = new User();
$user->username = $this->email;
$user->email = $this->email;
$user->name = $this->name;
$user->lastname = $this->lastname;
$user->phone = $this->phone;

if ($this->option_user_producer == 'producer') {
// producteur
$producer = new Producer;
$producer->name = $this->name_producer;
$producer->type = $this->type;
$producer->siret = $this->siret;
$producer->postcode = $this->postcode;
$producer->city = $this->city;
$producer->order_deadline = 20;
$producer->order_delay = 1;
// $producer->free_price = (float) abs($this->free_price);
$producer->id_tax_rate_default = $this->id_tax_rate_default ;

$cptSlug = 0 ;
do {
$slug = \common\helpers\Url::slugify($this->name_producer) ;
if($cptSlug) {
$slug .= $cptSlug ;
}
$producer->slug = $slug ;
$cptSlug ++ ;
} while(Producer::findOne(['slug' => $producer->slug])) ;


// génération d'un code
do {
$code = Password::generate();
$producer->code = $code;
} while (Producer::findOne(['code' => $code]));

$producer->save();

// user
$user->id_producer = $producer->id;
$user->status = User::STATUS_PRODUCER;

// envoi d'un email à l'administrateur pour le prévenir
Yii::$app->mailer->compose(
[
'html' => 'new-producer-html',
'text' => 'new-producer-text'
], [
'producer' => $producer
])
->setTo(Yii::$app->params['adminEmail'])
->setFrom([Yii::$app->params['adminEmail'] => 'distrib'])
->setSubject('[distrib] Nouveau producteur')
->send();

$idProducer = $producer->id ;
}
else {
$idProducer = $this->id_producer ;
}

$user->setPassword($this->password);
$user->generateAuthKey();
$user = $userContainer->getService()->createInstance();
$this->populateUser($user);

if ($user->save()) {
if ($this->isProducer()) {
$producer = $this->processFormProducer($user);
$idProducer = $producer->id;
} else {
$idProducer = $this->id_producer;
}

$producer = Producer::find()->where(['id' => $idProducer])->one();
$producer = $producerContainer->getRepository()->getOneById($idProducer);

if ($producer) {
Producer::addUser($user->id, $idProducer) ;

if($this->option_user_producer == 'user') {
// envoi d'un email à l'utilisateur
Yii::$app->mailer->compose(
[
'html' => 'signup-html',
'text' => 'signup-text'
],
[
'user' => $user,
'producer' => $producer
])
->setTo($user->email)
->setFrom([Yii::$app->params['adminEmail'] => 'distrib'])
->setSubject('[distrib] Inscription')
->send();
}
if ($user->save() && $producer) {
// Liaison User / Producer
$producerContainer->getService()->addUser($user->id, $idProducer);

// Envoi d'un email de bienvenue à l'utilisateur
if ($this->isCustomer()) {
$userContainer->getService()->sendEmailSignup($user, $producer);
}

return $user;
@@ -332,4 +267,59 @@ class SignupForm extends Model
return null;
}

public function isCustomer()
{
return $this->option_user_producer == 'user';
}

public function isProducer()
{
return $this->option_user_producer == 'producer';
}

public function processFormProducer($user)
{
$userContainer = Yii::$app->logic->getUserContainer();
$producerContainer = Yii::$app->logic->getProducerContainer();

// Création du producteur
$producer = $producerContainer->getService()->createInstance();
$this->populateProducer($producer);
$producerContainer->getService()->init($producer);
$producer->save();

/*
* Envoi d'un email à l'administrateur pour le prévenir
* qu'un nouveau producteur s'est inscrit
*/
$producerContainer->getService()->sendEmailNewProducer($producer);

// Initialisation de l'utilisateur (statut et liaison au producteur)
$userContainer->getService()->initProducer($user, $producer);

return $producer;
}

public function populateUser($user)
{
$userContainer = Yii::$app->logic->getUserContainer();

$user->username = $this->email;
$user->email = $this->email;
$user->name = $this->name;
$user->lastname = $this->lastname;
$user->phone = $this->phone;

$userContainer->getService()->initPassword($user, $this->password);
}

public function populateProducer($producer)
{
$producer->name = $this->name_producer;
$producer->type = $this->type;
$producer->siret = $this->siret;
$producer->postcode = $this->postcode;
$producer->city = $this->city;
$producer->id_tax_rate_default = $this->id_tax_rate_default;
}
}

+ 13639
- 0
producer/runtime/logs/app.log
ファイル差分が大きすぎるため省略します
ファイルの表示


書き込み プレビュー
読み込み中…
キャンセル
保存