dev #48675 add authentificaion restriction to send mail

library/ZendAfi/Controller/Action.php

@@ -293,9 +293,9 @@ class ZendAfi_Controller_Action extends Zend_Controller_Action {
                       {
                         return $this->_setParam($key, $value, $default);
                       })
-      ->visitForward(function($action)
+      ->visitForward(function($action, $controller = null, $module = null, $params = [])
                      {
-                       return $this->_forward($action);
+                       return $this->_forward($action, $controller, $module, $params);
                      })
       ->visitRedirectToReferer(function()
                                {

library/ZendAfi/Controller/Plugin/Abstract.php

@@ -207,8 +207,11 @@ abstract class ZendAfi_Controller_Plugin_Abstract {
   }
 
 
-  protected function _forward($action) {
-    return call_user_func($this->_forward, $action);
+  protected function _forward($action, $controller = null, $module = null, $params = []) {
+    return call_user_func_array($this->_forward, [$action,
+                                                  $controller,
+                                                  $module,
+                                                  $params]);
   }
 
 

library/ZendAfi/Controller/Plugin/Mailer/ModelFusion.php

@@ -23,6 +23,9 @@
 class ZendAfi_Controller_Plugin_Mailer_ModelFusion extends ZendAfi_Controller_Plugin_Abstract {
 
   public function sendMailAction() {
+    if(!Class_Users::getIdentity())
+      return $this->_forward('popup-login', 'auth', 'opac', ['redirect' => $this->_view->absoluteUrl()]);
+
     $this->_view->titre = $this->_('Envoyer par Email');
     $this->_view->form = ZendAfi_Form_SendMail::newWith(['subject' => $this->_getSubject(),
                                                          'content' => $this->_gethtml()]);

tests/scenarios/Mailer/MailerTest.php

@@ -20,29 +20,38 @@
  */
 
 
-class MailerSearchResultSimpleTest extends Admin_AbstractControllerTestCase {
+class MailerSearchResultSimpleTest extends AbstractControllerTestCase {
 
   protected $_storm_default_to_volatile = true;
 
   public function setup() {
     parent::setUp();
 
-    $this->fixture('Class_ModeleFusion',
-                   ['id' => 6,
-                   ]);
+    $guest = $this->fixture('Class_Users',
+                            ['id' => 13,
+                             'login' => 'XIII',
+                             'password' => 'BD'])
+                  ->beInvite();
 
-    $this->dispatch('/recherche/send-mail/expressionRecherche/il+%C3%A9tait+une+fois+dans+l%27ouest/tri/%2A/code_rebond/A26874/ids/6731%3B6877%3B6917%3B7528%3B7574%3B8648%3B11793%3B11972%3B13994%3B14010/strategy/Notice_List/modele_fusion/6', true);
+    ZendAfi_Auth::getInstance()->logUser($guest);
+
+    $this->fixture('Class_ModeleFusion',
+                   ['id' => 6]);
   }
 
 
   /** @test */
-  public function formShouldBePresent() {
-    $this->assertXPath('//form');
+  public function authLoginformShouldBePresent() {
+    ZendAfi_Auth::getInstance()->clearIdentity();
+    $this->dispatch('/recherche/send-mail/expressionRecherche/il+%C3%A9tait+une+fois+dans+l%27ouest/tri/%2A/code_rebond/A26874/ids/6731%3B6877%3B6917%3B7528%3B7574%3B8648%3B11793%3B11972%3B13994%3B14010/strategy/Notice_List/modele_fusion/6', true);
+
+    $this->assertContains('<input type=\"submit\" name=\"login\"', $this->_response->getBody());
   }
 
 
   /** @test */
   public function subjectShouldBeIlEtaitUneFoisDansLOuest() {
+    $this->dispatch('/recherche/send-mail/expressionRecherche/il+%C3%A9tait+une+fois+dans+l%27ouest/tri/%2A/code_rebond/A26874/ids/6731%3B6877%3B6917%3B7528%3B7574%3B8648%3B11793%3B11972%3B13994%3B14010/strategy/Notice_List/modele_fusion/6', true);
     $this->assertXPath('//form//input[@type="text"][@value="Résultat de recherche : il était une fois dans l\'ouest"]');
   }
 }