Merge branch 'hotline#76992_dysfonctionnement_du_refus_dans_le_workflow' into 'hotline'

hotline #76992 improve refuse message in article workflow

See merge request !2726

VERSIONS_HOTLINE/76992

+ - ticket #76992 : Article : amélioration du message de refus dans le workflow.
+ 
\ No newline at end of file

library/Class/Article.php

@@ -719,9 +719,9 @@ class Class_Article extends Storm_Model_Abstract {
 
 
   public function getRefusMessage() {
-    if (!parent::_get('refus_message'))
-      return Class_AdminVar::getWorkflowTextMailArticleRefused();
-    return parent::_get('refus_message');
+    return implode("\n",
+                   array_filter([Class_AdminVar::getWorkflowTextMailArticleRefused(),
+                                 parent::_get('refus_message')]));
   }
 
 

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

@@ -1929,7 +1929,8 @@ class CmsControllerWorkflowArticleRefusedTest extends CmsControllerWorkflowTestC
               'description' => '',
               'id_cat' => 23,
               'contenu' => 'Ne manquez pas cet évènement.',
-              'status' => Class_Article::STATUS_REFUSED
+              'status' => Class_Article::STATUS_REFUSED,
+              'refus_message' => 'Votre article laisse à désirer, à revoir de fond en comble immédiatement.'
     ];
 
     $this->postDispatch('/admin/cms/edit/id/4',
@@ -1979,11 +1980,19 @@ class CmsControllerWorkflowArticleRefusedTest extends CmsControllerWorkflowTestC
 
   /** @test */
   public function sentMailToUserWhenRefusedShouldContainsDefaultBody() {
+    Class_AdminVar::set('WORKFLOW_TEXT_MAIL_ARTICLE_REFUSED', 'L\'article "TITRE_ARTICLE" "URL_ARTICLE" a été refusé par "SAVED_BY_ARTICLE".');
     $this->postArticleRefuser();
-    $this->assertEquals('L\'article a été refusé.',quoted_printable_decode($this->mock_transport->getSentMails()[0]->getBodyText()->getContent()));
+    $body_mail = quoted_printable_decode($this
+                                         ->mock_transport
+                                         ->getSentMails()[0]
+                                         ->getBodyText()
+                                         ->getContent());
+    $this->assertContains('L\'article "Katsuhiro Otomo en dédicace !', $body_mail);
+    $this->assertContains('/cms/articleview/id/4', $body_mail);
+    $this->assertContains('a été refusé par "Mario Super".
+Votre article laisse à désirer, à revoir de fond en comble immédiatement.',
+                          $body_mail);
   }
-
-
 }