hotline #60951 : fix recursive call on 2 columns box (when col_*_module_id...

hotline #60951 : fix recursive call on 2 columns box (when  col_*_module_id preferences is the same as id_module)

VERSIONS_HOTLINE/60951

+ - ticket #60951 : Récursivité lorsqu'une boite 2 colonnes fait appel à son id pour les attributs col_gauche_module_id ou col_droite_module_id
\ No newline at end of file

library/ZendAfi/View/Helper/Accueil/ConteneurDeuxColonnes.php

@@ -43,6 +43,9 @@ class ZendAfi_View_Helper_Accueil_ConteneurDeuxColonnes extends ZendAfi_View_Hel
         $module_id = $this->preferences[$id_key];
         if (!ZendAfi_View_Helper_Accueil_Base::getModuleHelper($module_id, $this->view))
           $module_id = $this->createModuleForCol($colonne);
+
+        if ($module_id == $this->id_module)
+          continue;
       }
 
       $helper =  ZendAfi_View_Helper_Accueil_Base::getModuleHelper($module_id, $this->view);

tests/application/modules/admin/controllers/AccueilControllerConteneurDeuxColonnesTest.php

@@ -283,6 +283,20 @@ class AccueilControllerConteneurDeuxColonnesNewTest extends Admin_AbstractContro
     $this->assertXPathContentContains('//div', 'Le catalogue contient');
     $this->assertXPath('//div[contains(@class,"conteneur_deux_colonnes")]//div[contains(@class,"rech_guide")]');
   }
+
+
+  /** @test */
+  public function withSameModuleIdOneModuleShouldBeCreated() {
+    $this->boite2cols['preferences'] = ['col_gauche_module_id' => 3,
+                                        'col_gauche_type' => 'COMPTEURS',
+                                        'col_droite_module_id' => 235,
+                                        'col_droite_type' => 'RECH_GUIDEE'];
+
+    $this->profil_biologie->updateModuleConfigAccueil(3, $this->boite2cols);
+    $this->dispatch('/opac?id_profil=34');
+    $this->assertXPath('//div[contains(@class,"conteneur_deux_colonnes")]//div[contains(@class,"rech_guide")]');
+  }
+
 }