Merge branch 'hotline#37032_domain_bookmark' into 'stable'

Hotline#37032 domain bookmark

See merge request !1422

VERSIONS_HOTLINE/37032

+ - ticket #37032 : Correction du lien de gestion des favoris situé sur l'icone. Ajout d'un message dans le formulaire des domaines sur la case à cocher "Peut être un favori utilisateur" indiquant la nécessité d'une intégration pour que le paramètrage fonctionne.
\ No newline at end of file

application/modules/admin/views/scripts/catalogue/form.phtml

@@ -46,7 +46,7 @@
             <td class="gauche"><?php echo $this->formCheckbox('indexer',
                                                               (int)$this->catalogue->getIndexer(),
                                                               null,
-                                                              [1, 0]); ?></td>
+                                                              [1, 0]); ?><span style="font-weight: bold; font-size: 0.9em;"><?php echo $this->traduire('Nécessite une intégration du catalogue pour être actif'); ?></span></td>
           </tr>
         </table>
       </fieldset>

library/ZendAfi/View/Helper/Abonne/Abstract.php

@@ -19,13 +19,13 @@
  * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301  USA
  */
 abstract class ZendAfi_View_Helper_Abonne_Abstract extends ZendAfi_View_Helper_BaseHelper {
-  public function tagFicheAbonne($html, $icone='', $url='') {
+  public function tagFicheAbonne($html, $icone='', $url='', $attribs = []) {
     if ($icone && $url) {
       $html = $this->view->tagAnchor($url,
                                      $this->view->tagImg(Class_Profil::getCurrentProfil()
                                                          ->getUrlImage('/abonnes/'.$icone.'.png'),
                                                          ['alt' => $icone]) . $html,
-                                     ['class' => $icone]);
+                                     array_merge(['class' => $icone], $attribs));
     }
     return '<div class="abonneFiche '.$icone.'">'.$html.'<div class="clear"></div></div>';
   }

library/ZendAfi/View/Helper/Abonne/Settings.php

@@ -32,7 +32,8 @@ class ZendAfi_View_Helper_Abonne_Settings extends ZendAfi_View_Helper_Abonne_Abs
                                          $action_url,
                                          $this->view->_('Gérer mes favoris')),
                                  'settings',
-                                 $action_url);
+                                 $action_url,
+                                 ['data-popup' => 'true']);
   }
 }
 ?>
\ No newline at end of file

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

@@ -268,6 +268,13 @@ class CatalogueControllerWithAdminBibAndRightTotalAccessTest extends AdminCatalo
     $this->dispatch('/admin/catalogue/edit/id_catalogue/300');
     $this->assertXPathContentContains('//td', 'Valider');
   }
+
+
+  /** @test */
+  public function editCatalogueShouldAddToBookmarksCheckboxBePresent() {
+    $this->dispatch('/admin/catalogue/edit/id_catalogue/300');
+    $this->assertXPathContentContains('//td//input[@type="checkbox"][@id="indexer"]/following-sibling::span', 'Nécessite une intégration');
+  }
 }
 
 

tests/application/modules/opac/controllers/AbonneControllerFicheTest.php

@@ -158,4 +158,15 @@ class AbonneControllerFicheAsAdminDisableSuggestTest extends AbstractAbonneContr
     $this->assertNotXPathContentContains('//a[contains(@href, "/abonne/suggestion-achat")]',
                                          'Suggérer un achat');
   }
+
+  /** @test */
+  public function settingsLinkShouldBePopup() {
+    $this->assertXPath('//div//a[2][@data-popup="true"]');
+  }
+
+
+  /** @test */
+  public function settingsLinkWithImageShouldBePopup() {
+    $this->assertXPath('//div//a[@data-popup="true"]/img[contains(@src, "abonnes/settings")]');
+  }
 }