dev #70488: advanced search configuration : rename some column names (file...

dev #70488: advanced search configuration : rename some column names  (file and visible)   and make visible/unvisible button

application/modules/admin/controllers/AdvancedSearchFormController.php

@@ -36,6 +36,15 @@ class Admin_AdvancedSearchFormController extends ZendAfi_Controller_Action {
 
   }
 
+  public function makevisibleAction() {
+    if (!$search_form = Class_SearchForm::getLoader()->find((int)$this->_getParam('id'))) {
+      $this->_redirect('admin/advanced-search-form');
+      return;
+    }
+    $search_form->setVisible(!$search_form->getVisible())->save();
+    return $this->_redirectToIndex();
+
+  }
 
 
 }

application/modules/admin/views/scripts/advanced-search-form/index.phtml

@@ -7,7 +7,6 @@ echo $this->Button_New((new Class_Entity())
 $description = (new Class_TableDescription('searchforms'))
   ->addColumn($this->_('Libellé'), 'label')
   ->addColumn($this->_('fichier'), 'filename')
-  ->addColumn($this->_('Visible'), 'visible')
   ->addRowAction(function($model) { return $this->renderPluginsActions($model); });
 
 echo $this->renderTable($description, Class_SearchForm::findAll(), ['sorter' => true]);

library/ZendAfi/Controller/Plugin/Manager/SearchForm.php

@@ -24,7 +24,7 @@ class ZendAfi_Controller_Plugin_Manager_SearchForm extends ZendAfi_Controller_Pl
   public function getActions($model) {
     return [
             ['url' => '/admin/advanced-search-form/view/id/%s',
-             'icon' => 'test',
+             'icon' => 'view',
              'label' => $this->_('')
             ],
             ['url' => '/admin/advanced-search-form/edit/id/%s',
@@ -36,7 +36,7 @@ class ZendAfi_Controller_Plugin_Manager_SearchForm extends ZendAfi_Controller_Pl
                        'controller' => 'advanced-search-form',
                        'action' => 'makevisible',
                        'id' =>  '%s'],
-             'icon' => function($model) {return $model->getVisible() ? 'view' : 'hide';},
+             'icon' => function($model) {return $model->getVisible() ? 'show' : 'hide';},
              'label' => $this->_('Visualisation de l\album')],
 
             ['url' => '/admin/advanced-search-form/delete/id/%s',

library/ZendAfi/Form/AdvancedSearch.php

@@ -33,11 +33,11 @@ class ZendAfi_Form_AdvancedSearch extends ZendAfi_Form {
                     'placeholder' => $this->_('Libellé de la configuration')])
       ->addElement('userfile', 'filename')
       ->addElement('checkbox',
-                      'visible',
-                      ['label' => $this->_('Le formulaire est visible'),
-                       'required' => true,
-                       'allowEmpty' => false])
-      ->addDisplayGroup(['label','filename', 'visible'],
+                   'visible',
+                   ['label' => $this->_('Le formulaire est visible'),
+                    'required' => true,
+                    'allowEmpty' => false])
+      ->addDisplayGroup(['filename', 'label','visible'],
                         'all',
                         ['legend' => $this->_('Configuration du formulaire')]);
 

tests/scenarios/AdvancedSearch/AdminConfigurationTest.php

@@ -76,13 +76,25 @@ class AdvancedSearch_AdminConfigurationActiveTest extends   Admin_AbstractContro
     );
   }
 
+  /** @test */
+  public function makeVisibleShouldRenderVisibleIfUnvisible() {
+    $this->dispatch('/admin/advanced-search-form/makevisible/id/1');
+    $this->assertEquals(false, Class_SearchForm::find(1)->getVisible());
+  }
 
 
   /** @test */
-  public function addShouldDisplayForms() {
-    $this->dispatch('/admin/advanced-search-form/add');
-    $this->assertXPath('//toto',$this->_response->getBody()
-    );
+  public function makeVisibleShouldRenderInVisibleIfVisible() {
+    Class_SearchForm::find(1)->setVisible(false)->save();
+    $this->dispatch('/admin/advanced-search-form/makevisible/id/1');
+    $this->assertEquals(true, Class_SearchForm::find(1)->getVisible());
+  }
+
+
+  /** @test */
+  public function deleteShouldDeleteForm() {
+    $this->dispatch('/admin/advanced-search-form/delete/id/1');
+    $this->assertNull(Class_SearchForm::find(1));
   }