Merge branch 'master' of https://git.unicaen.fr/lib/unicaen/app

CHANGELOG.md

 CHANGELOG
 =========
 
+3.1.15
+------
+- Refactoring de HostLocalizationUnicaen pour éviter d'avoir les adresses des proxies et reverse proxies en dur dans le code. Passage en fichier de config pour que chacun puisse configurer dans son application.
+- Aide de vue FormControlGroup : utilisation des aides de vue ZF standards pour les éléments de formulaire ZF Date et Time
+- Support des fonctions Doctrine compriseEntre() et pasHistorise()
+- [FIX] Modification du répertoire de cache de mPDF car tentatives de supprimer les vieux fichiers ne lui appartenant pas
+- [FIX] Correction de Util::truncatedString boguée
+
 3.1.14
 ------
 - Nouvelle ligne de commande 'run-sql-query' pour exécuter une requête SQL.

config/module.config.php

@@ -6,10 +6,12 @@ use UnicaenApp\Controller\CacheControllerFactory;
 use UnicaenApp\Controller\InstadiaControllerFactory;
 use UnicaenApp\Form\View\Helper\FormControlGroup;
 use UnicaenApp\Form\View\Helper\FormControlGroupFactory;
-use UnicaenApp\HostLocalization\HostLocalizationUnicaen;
-use UnicaenApp\HostLocalization\HostLocalizationUnicaenFactory;
+use UnicaenApp\HostLocalization\HostLocalization;
+use UnicaenApp\HostLocalization\HostLocalizationFactory;
 use UnicaenApp\Message\View\Helper\MessageHelper;
 use UnicaenApp\Message\View\Helper\MessageHelperFactory;
+use UnicaenApp\ORM\Query\Functions\CompriseEntre;
+use UnicaenApp\ORM\Query\Functions\PasHistorise;
 use UnicaenApp\Service\InstadiaServiceFactory;
 use UnicaenApp\Service\Mailer\MailerService;
 use UnicaenApp\Service\Mailer\MailerServiceFactory;
@@ -276,7 +278,7 @@ return [
         'view_manager' => [
             'display_not_found_reason' => true,
             'display_exceptions'       => true,
-        ]
+        ],
     ],
     'service_manager' => [
         'factories'          => [
@@ -326,7 +328,7 @@ return [
             MailerService::class    => MailerServiceFactory::class,
             RunSQLService::class    => RunSQLServiceFactory::class,
             'instadia'              => InstadiaServiceFactory::class,
-            HostLocalizationUnicaen::class => HostLocalizationUnicaenFactory::class,
+            HostLocalization::class => HostLocalizationFactory::class,
         ],
         'shared'             => [
             'MouchardListenerErrorHandler' => false,
@@ -349,8 +351,8 @@ return [
         ],
         'aliases'            => [
             'MailerService'    => MailerService::class,
-            'HostLocalization'  => HostLocalizationUnicaen::class,
-        ]
+            'HostLocalization' => HostLocalization::class,
+        ],
     ],
     'form_elements'   => [
         'invokables'   => [
@@ -458,7 +460,7 @@ return [
             'UnicaenApp\Controller\Cache'    => CacheControllerFactory::class,
             'UnicaenApp\Controller\Instadia' => InstadiaControllerFactory::class,
             ConsoleController::class         => ConsoleControllerFactory::class,
-        ]
+        ],
     ],
 
     'doctrine' => [
@@ -477,6 +479,14 @@ return [
                 ],
             ],
         ],
+        'configuration' => [
+            'orm_default' => [
+                'string_functions' => [
+                    'compriseEntre' => CompriseEntre::class,
+                    'pasHistorise' => PasHistorise::class,
+                ],
+            ],
+        ],
     ],
 
     'view_manager' => [

config/unicaen-app.local.php.dist

@@ -7,6 +7,24 @@
  */
 return [
     'unicaen-app' => [
+         /**
+         * Paramétrage pour utilisation pour autorisation ou non à la connexion à
+         * une app de l'exterieur de l'établissement
+         */
+         hostlocalization'       => [
+             'activated' => false,
+
+             'proxies' => [
+                 //xxx.xx.xx.xxx
+             ],
+
+             'reverse-proxies' => [
+                 //xxx.xx.xx.xxx
+             ],
+
+             'masque-ip' => '',
+
+         ],
         /**
          * Connexion à l'annuaire LDAP (NB: compte admin requis)
          */
@@ -59,5 +77,7 @@ return [
                 // ['195.220.135.97', '194.199.107.33'], // Via proxy
             ],
         ],
+
+
     ],
 ];
\ No newline at end of file

doc/Services.md

@@ -6,8 +6,7 @@ LDAP
 
 ### Group
 
-Service d\'accès aux groupes de l\'annuaire LDAP. Avec le gestionnaire
-de service, ce service est accessible ainsi :
+Service d\'accès aux groupes de l\'annuaire LDAP. Avec le gestionnaire de service, ce service est accessible ainsi :
 
 ``` {.php}
 $service = $sm->get('ldap_group_service'); /* @var $service \UnicaenApp\Service\Ldap\Group */
@@ -15,8 +14,7 @@ $service = $sm->get('ldap_group_service'); /* @var $service \UnicaenApp\Service\
 
 ### People
 
-Service d\'accès aux individus de l\'annuaire LDAP. Avec le gestionnaire
-de service, ce service est accessible ainsi :
+Service d\'accès aux individus de l\'annuaire LDAP. Avec le gestionnaire de service, ce service est accessible ainsi :
 
 ``` {.php}
 $service = $sm->get('ldap_people_service'); /* @var $service \UnicaenApp\Service\Ldap\People */
@@ -24,8 +22,7 @@ $service = $sm->get('ldap_people_service'); /* @var $service \UnicaenApp\Service
 
 ### Structure
 
-Service d\'accès aux structures de l\'annuaire LDAP. Avec le
-gestionnaire de service, ce service est accessible ainsi :
+Service d\'accès aux structures de l\'annuaire LDAP. Avec le gestionnaire de service, ce service est accessible ainsi :
 
 ``` {.php}
 $service = $sm->get('ldap_structure_service'); /* @var $service \UnicaenApp\Service\Ldap\Structure */
@@ -35,16 +32,17 @@ $service = $sm->get('ldap_structure_service'); /* @var $service \UnicaenApp\Serv
 
 Le service HostLocalization a pour but de déterminer si le poste client est situé dans le réseau de l'Etablissement ou non.
 
-En standard, seul un mécanisme de détection pour l'Université de Caen est disponible.
-Pour Caen, l'idée est la suivante : la méthode inEtablissement du service renverra true si la machine de l'utilisateur est dans les cas suivants :
+En standard, seul un mécanisme de détection pour l'Université de Caen est disponible. Pour Caen, l'idée est la suivante : la
+méthode inEtablissement du service renverra true si la machine de l'utilisateur est dans les cas suivants :
+
 * En direct et en interne sans passer par le proxy (pas de redirection)
 * En interne, que l'on sort puis on re-rentre en passant par le reverse proxy
-* En interne, que l'on passe par le proxy en interne
-Sinon false.
+* En interne, que l'on passe par le proxy en interne Sinon false.
 
 Il est cependant possible d'ajouter votre propre mécanisme de localisation.
 
-Attention : en mode développement, sur votre machine et avec l'appli en localhost, vous ne serez pas considéré comme sur le réseau de l'établissement.
+Attention : en mode développement, sur votre machine et avec l'appli en localhost, vous ne serez pas considéré comme sur le
+réseau de l'établissement.
 
 #### Récupération et utilisation du service
 
@@ -63,14 +61,15 @@ if ($hl->inEtablissement()){
 }
 ```
 
-Un Trait [HostLocalizationAwareTrait](../src/UnicaenApp/HostLocalization/HostLocalizationAwareTrait.php) est dispo avec un getter et un setter pour pouvoir injecter proprement la dépendance dans votre code.
+Un Trait [HostLocalizationAwareTrait](../src/UnicaenApp/HostLocalization/HostLocalizationAwareTrait.php) est dispo avec un
+getter et un setter pour pouvoir injecter proprement la dépendance dans votre code.
 
 ### Créer votre propre système de localization
 
 Pour utiliser votre propre système de HostLocalization, il vous faut :
 
 Créer une classe implémentant [HostLocalizationInterface](../src/UnicaenApp/HostLocalization/HostLocalizationInterface.php).
-La rendre dispo comme service dans votre config du ServiceManager.
-Modifier l'alias de service 'HostLocalization' pour le faire pointer sur votre propre classe.
+La rendre dispo comme service dans votre config du ServiceManager. Modifier l'alias de service 'HostLocalization' pour le
+faire pointer sur votre propre classe.
 
-Vous pourrez au besoin vous inspirer de [HostLocalizationUnicaen](../src/UnicaenApp/HostLocalization/HostLocalizationUnicaen.php).
\ No newline at end of file
+Vous pourrez au besoin vous inspirer de [HostLocalization](../src/UnicaenApp/HostLocalization/HostLocalization.php).
\ No newline at end of file

src/UnicaenApp/Exporter/Pdf.php

@@ -584,7 +584,7 @@ class Pdf implements ExporterInterface
                     $this->marginBottom,
                     $this->marginHeader,
                     $this->marginFooter,
-                    'tempDir' => sys_get_temp_dir(),
+                    'tempDir' => sys_get_temp_dir() . '/mpdf',
             ]);
 
             $this->mpdf->useSubstitutions = false;

src/UnicaenApp/Form/View/Helper/FormControlGroup.php

@@ -254,6 +254,12 @@ class FormControlGroup extends AbstractHelper
             $helper = $this->getView()->plugin('formSearchAndSelect');
             $helper->setAutocompleteMinLength(2);
             $html = $helper($element);
+        } elseif($element instanceof \Zend\Form\Element\Date) {
+            $helper = $this->getView()->plugin(\Zend\Form\View\Helper\FormDate::class);
+            $html = $helper($element);
+        } elseif($element instanceof \Zend\Form\Element\Time) {
+            $helper = $this->getView()->plugin(\Zend\Form\View\Helper\FormTime::class);
+            $html = $helper($element);
         } elseif($element instanceof DateTime) {
             $helper = $this->getView()->plugin('formDateTime');
             $html = $helper($element);

src/UnicaenApp/HostLocalization/HostLocalizationUnicaen.php → src/UnicaenApp/HostLocalization/HostLocalization.php

@@ -2,43 +2,58 @@
 
 namespace UnicaenApp\HostLocalization;
 
-class HostLocalizationUnicaen implements HostLocalizationInterface
+
+class HostLocalization implements HostLocalizationInterface
 {
+
+    protected $activated;
+
+    protected $proxies;
+
+    protected $reverseProxies;
+
+    protected $masqueIp;
+
+
+
     /**
      * Renvoie true si le'utilisateur utilise une machine connectée au réseau de l'établissement
      * ou bien si la machine est hors réseau
      *
      * @return bool
      */
-    public function inEtablissement(): bool
+
+    public function __construct($activated, $masqueIp, $proxies, $reverseProxies)
     {
-        $ip        = isset($_SERVER['REMOTE_ADDR']) ? $_SERVER['REMOTE_ADDR'] : '';
-        $forwarded =  isset($_SERVER['HTTP_X_FORWARDED_FOR']) ? $_SERVER['HTTP_X_FORWARDED_FOR'] : '';
+        $this->activated      = $activated;
+        $this->proxies        = $proxies;
+        $this->reverseProxies = $reverseProxies;
+        $this->masqueIp       = $masqueIp;
+    }
 
-        $ipInterne = '10.';
 
-        $proxies = [
-            '10.14.128.39',
-            '10.14.128.100',
-            '10.14.128.100',
-            '10.14.128.101',
-            '10.14.128.137',
-        ];
 
-        $reverseProxies = [
-            '193.55.120.23',
-            '193.55.120.24',
-            '193.55.120.25',
-        ];
+    /**
+     * Retourne selon les règles définies en config si on est dans l'établissement ou à l'extérieur.
+     *
+     * @return bool
+     */
+    public function inEtablissement(): bool
+    {
+        if (!$this->activated || empty($this->masqueIp)) {
+            return true;
+        }
+        $ip        = isset($_SERVER['REMOTE_ADDR']) ? $_SERVER['REMOTE_ADDR'] : '';
+        $forwarded = isset($_SERVER['HTTP_X_FORWARDED_FOR']) ? $_SERVER['HTTP_X_FORWARDED_FOR'] : '';
 
         // Si on est en direct et en interne sans passer par le proxy (pas de redirection)
-        if (0 === strpos($ip,$ipInterne) && $forwarded === '') return true;
+        if (0 === strpos($ip, $this->masqueIp) && $forwarded === '') return true;
 
         // Si on est en interne, que l'on sort puis on re-rentre en passant par le reverse proxy
-        if (in_array($ip,$reverseProxies) && 0 === strpos($forwarded,$ipInterne)) return true;
+        if (in_array($ip, $this->reverseProxies) && 0 === strpos($forwarded, $this->masqueIp)) return true;
 
         // Si on est en interne, que l'on passe par le proxy en interne
-        if (in_array($ip,$proxies) && 0 === strpos($forwarded,$ipInterne)) return true;
+        if (in_array($ip, $this->proxies) && 0 === strpos($forwarded, $this->masqueIp)) return true;
 
         // Sinon, on vient de l'extérieur
         return false;

src/UnicaenApp/HostLocalization/HostLocalizationUnicaenFactory.php → src/UnicaenApp/HostLocalization/HostLocalizationFactory.php

@@ -4,11 +4,18 @@ namespace UnicaenApp\HostLocalization;
 
 use Interop\Container\ContainerInterface;
 
-class HostLocalizationUnicaenFactory
+class HostLocalizationFactory
 {
     public function __invoke(ContainerInterface $container, $requestedName, array $options = null)
     {
-        $service = new HostLocalizationUnicaen();
+        $config         = $container->get('config');
+        $activated      = $config['unicaen-app']['hostlocalization']['activated'];
+        $proxies        = $config['unicaen-app']['hostlocalization']['proxies'];
+        $reverseProxies = $config['unicaen-app']['hostlocalization']['reverse-proxies'];
+        $masqueIp       = $config['unicaen-app']['hostlocalization']['masque-ip'];
+
+        $service = new HostLocalization($activated, $masqueIp, $proxies, $reverseProxies);
+
 
         return $service;
     }

src/UnicaenApp/ORM/Query/Functions/CompriseEntre.php

+<?php
+
+namespace UnicaenApp\ORM\Query\Functions;
+
+use Doctrine\ORM\Query\AST\Functions\FunctionNode;
+use Doctrine\ORM\Query\Lexer;
+use Doctrine\ORM\Query\Parser;
+use Doctrine\ORM\Query\SqlWalker;
+
+class CompriseEntre extends FunctionNode
+{
+    public $histoCreation;
+    public $histoDestruction;
+    public $dateObs;
+
+    public function getSql(SqlWalker $sqlWalker)
+    {
+        $sql = sprintf('comprise_entre(%s, %s',
+                $this->histoCreation->dispatch($sqlWalker),
+                $this->histoDestruction->dispatch($sqlWalker));
+        
+        if ($this->dateObs){
+            $sql .= ','.$this->dateObs->dispatch($sqlWalker);
+        }
+        $sql .= ')';
+        return $sql;
+    }
+
+    /**
+     * NB: le format DQL attendu est "compriseEntre"
+     * (et non pas "UNICAEN_ORACLE.COMPRISE_ENTRE").
+     * 
+     * @param Parser $parser
+     */
+    public function parse(Parser $parser)
+    {
+        $lexer = $parser->getLexer();
+        $parser->match(Lexer::T_IDENTIFIER);
+        $parser->match(Lexer::T_OPEN_PARENTHESIS);
+        $this->histoCreation = $parser->ArithmeticPrimary();
+        if(Lexer::T_COMMA === $lexer->lookahead['type']){
+            $parser->match(Lexer::T_COMMA);
+            $this->histoDestruction = $parser->ArithmeticPrimary();
+        }
+        if(Lexer::T_COMMA === $lexer->lookahead['type']){
+            $parser->match(Lexer::T_COMMA);
+            $this->dateObs = $parser->ArithmeticPrimary();
+        }
+        $parser->match(Lexer::T_CLOSE_PARENTHESIS);
+    }
+}
\ No newline at end of file

src/UnicaenApp/ORM/Query/Functions/PasHistorise.php

+<?php
+
+namespace UnicaenApp\ORM\Query\Functions;
+
+use Doctrine\ORM\Query\AST\Functions\FunctionNode;
+use Doctrine\ORM\Query\Lexer;
+use Doctrine\ORM\Query\Parser;
+use Doctrine\ORM\Query\SqlWalker;
+
+/**
+ * Fonction DQL maison "pasHistorise()".
+ * 
+ * NB: le format DQL attendu est "pasHistorise(EntityExpression)". 
+ * Exemples :
+ * <pre>
+ * $qb->andWhere("1 = pasHistorise(sr)")
+ * $qb->andWhere("1 = pasHistorise(int.serviceReferentiel)")
+ * </pre>
+ * 
+ * Fait appel à la fonction Oracle UNICAEN_ORACLE.COMPRISE_ENTRE().
+ */
+class PasHistorise extends FunctionNode
+{
+    public $alias;
+    public $dateObservation;
+
+    /**
+     * Parsing.
+     * 
+     * NB: le format DQL attendu est "pasHistorise(EntityExpression)".
+     * Exemple :
+     *  pasHistorise(s)
+     *  pasHistorise(int.serviceReferentiel)
+     * 
+     * @param Parser $parser
+     */
+    public function parse(Parser $parser)
+    {
+        $lexer = $parser->getLexer();
+        $parser->match(Lexer::T_IDENTIFIER);
+        $parser->match(Lexer::T_OPEN_PARENTHESIS);
+        $this->alias = $parser->EntityExpression();
+        if(Lexer::T_COMMA === $lexer->lookahead['type']){
+            $parser->match(Lexer::T_COMMA);
+            $this->dateObservation = $parser->ArithmeticPrimary();
+        }
+        $parser->match(Lexer::T_CLOSE_PARENTHESIS);
+    }
+
+    /**
+     * Génère le code SQL.
+     * 
+     * @param SqlWalker $sqlWalker
+     * @return string
+     */
+    public function getSql(SqlWalker $sqlWalker)
+    {
+        $expr1 = clone($this->alias);
+        $expr2 = clone($this->alias);
+        
+        $expr1->field = 'histoCreation';
+        $expr2->field = 'histoDestruction';
+        
+        $sql = sprintf('comprise_entre(%s, %s, %s)',
+                $expr1->dispatch($sqlWalker),
+                $expr2->dispatch($sqlWalker),
+                $this->dateObservation ? $this->dateObservation->dispatch($sqlWalker) : 'null');
+        
+        return $sql;
+    }
+}
\ No newline at end of file

src/UnicaenApp/Util.php

@@ -338,11 +338,20 @@ class Util
         if (strlen($string) <= $length) {
             return $string;
         }
+
+        $trunc = substr($string, 0, $length);
         if ($string[$length] === ' ') {
-            return substr($string, 0, $length) . $appended;
+            $kept = $trunc;
+        } else {
+            $found = strrpos($trunc, ' '); // position du dernier espace
+            if ($found === false) {
+                $kept = $trunc;
+            } else {
+                $kept = substr($string, 0, $found); // on tronque à l'espace trouvé
+            }
         }
 
-        return substr($string, 0, strrpos(substr($string, 0, $length), ' ')) . $appended;
+        return $kept . $appended;
     }