From 6ca9f8ad31c3c34e5a2ffb8bc98e9081c8d7b25e Mon Sep 17 00:00:00 2001 From: Bruno Da Silva Date: Fri, 6 Apr 2018 12:06:20 +0000 Subject: [PATCH] advanced search: removal of legacy_search_drawer_open associated to this change, those wiki pages are altered : - latest:admin:itop_configuration_file (`legacy_search_drawer_open` removal) - latest:customization:xml_reference (`search_form_open` default value changed) SVN:trunk[5626] --- application/menunode.class.inc.php | 11 ++--------- application/ui.extkeywidget.class.inc.php | 2 -- application/ui.linksdirectwidget.class.inc.php | 2 -- application/ui.linkswidget.class.inc.php | 2 -- .../ui.searchformforeignkeys.class.inc.php | 2 -- core/config.class.inc.php | 8 -------- pages/UI.php | 7 +++++-- setup/compiler.class.inc.php | 2 +- .../search/searchform.class.inc.php | 18 ++++++++++++------ 9 files changed, 20 insertions(+), 34 deletions(-) diff --git a/application/menunode.class.inc.php b/application/menunode.class.inc.php index 06cf0cf8c..74740b51e 100644 --- a/application/menunode.class.inc.php +++ b/application/menunode.class.inc.php @@ -824,14 +824,7 @@ class OQLMenuNode extends MenuNode $this->sPageTitle = "Menu:$sMenuId+"; $this->sOQL = $sOQL; $this->bSearch = $bSearch; - if ($bSearchFormOpen == null) - { - $this->bSearchFormOpen = MetaModel::GetConfig()->Get('legacy_search_drawer_open'); - } - else - { - $this->bSearchFormOpen = $bSearchFormOpen; - } + $this->bSearchFormOpen = $bSearchFormOpen; $this->m_aParams = array(); $this->aReflectionProperties['oql'] = $sOQL; $this->aReflectionProperties['do_search'] = $bSearch; @@ -966,7 +959,7 @@ class SearchMenuNode extends MenuNode $oPage->SetBreadCrumbEntry("menu-".$this->sMenuId, $this->GetTitle(), '', '', utils::GetAbsoluteUrlAppRoot().'images/search.png'); $oSearch = new DBObjectSearch($this->sClass); - $aParams = array_merge(array('open' => true, 'table_id' => 'Menu_'.utils::GetSafeId($this->GetMenuId())), $aExtraParams); + $aParams = array_merge(array('table_id' => 'Menu_'.utils::GetSafeId($this->GetMenuId())), $aExtraParams); $oBlock = new DisplayBlock($oSearch, 'search', false /* Asynchronous */, $aParams); $oBlock->Display($oPage, 0); } diff --git a/application/ui.extkeywidget.class.inc.php b/application/ui.extkeywidget.class.inc.php index 5ec9acd5d..e019ea963 100644 --- a/application/ui.extkeywidget.class.inc.php +++ b/application/ui.extkeywidget.class.inc.php @@ -341,13 +341,11 @@ EOF $aParams = array(); $oFilter = new DBObjectSearch($this->sTargetClass); } - $bOpen = MetaModel::GetConfig()->Get('legacy_search_drawer_open'); $oFilter->SetModifierProperty('UserRightsGetSelectFilter', 'bSearchMode', $this->bSearchMode); $oBlock = new DisplayBlock($oFilter, 'search', false, $aParams); $sHTML .= $oBlock->GetDisplay($oPage, $this->iId, array( 'menu' => false, - 'open' => $bOpen, 'currentId' => $this->iId, 'table_id' => "dr_{$this->iId}", 'table_inner_id' => "{$this->iId}_results", diff --git a/application/ui.linksdirectwidget.class.inc.php b/application/ui.linksdirectwidget.class.inc.php index bbbd09745..a4056352c 100644 --- a/application/ui.linksdirectwidget.class.inc.php +++ b/application/ui.linksdirectwidget.class.inc.php @@ -331,11 +331,9 @@ class UILinksWidgetDirect $aArgs = array_merge($oCurrentObj->ToArgs('this'), $oFilter->GetInternalParams()); $oFilter->SetInternalParams($aArgs); } - $bOpen = MetaModel::GetConfig()->Get('legacy_search_drawer_open'); $oBlock = new DisplayBlock($oFilter, 'search', false); $sHtml .= $oBlock->GetDisplay($oPage, "SearchFormToAdd_{$this->sInputid}", array( - 'open' => $bOpen, 'result_list_outer_selector' => "SearchResultsToAdd_{$this->sInputid}", 'table_id' => "add_{$this->sInputid}", 'table_inner_id' => "ResultsToAdd_{$this->sInputid}", diff --git a/application/ui.linkswidget.class.inc.php b/application/ui.linkswidget.class.inc.php index 9055a248c..3f0de757e 100644 --- a/application/ui.linkswidget.class.inc.php +++ b/application/ui.linkswidget.class.inc.php @@ -395,7 +395,6 @@ EOF */ public function GetObjectPickerDialog($oPage, $oCurrentObj, $sJson, $aAlreadyLinkedIds = array(), $aPrefillFormParam = array()) { - $bOpen = MetaModel::GetConfig()->Get('legacy_search_drawer_open'); $sHtml = "
\n"; $oAlreadyLinkedFilter = new DBObjectSearch($this->m_sRemoteClass); @@ -421,7 +420,6 @@ EOF $oBlock = new DisplayBlock($oFilter, 'search', false); $sHtml .= $oBlock->GetDisplay($oPage, "SearchFormToAdd_{$this->m_sAttCode}{$this->m_sNameSuffix}", array( - 'open' => $bOpen, 'menu' => false, 'result_list_outer_selector' => "SearchResultsToAdd_{$this->m_sAttCode}{$this->m_sNameSuffix}", 'table_id' => 'add_'.$this->m_sAttCode, diff --git a/application/ui.searchformforeignkeys.class.inc.php b/application/ui.searchformforeignkeys.class.inc.php index 1b97ac808..0bb7c4703 100644 --- a/application/ui.searchformforeignkeys.class.inc.php +++ b/application/ui.searchformforeignkeys.class.inc.php @@ -41,7 +41,6 @@ class UISearchFormForeignKeys */ public function ShowModalSearchForeignKeys($oPage, $sTitle) { - $bOpen = MetaModel::GetConfig()->Get('legacy_search_drawer_open'); $sHtml = "
\n"; $oFilter = new DBObjectSearch($this->m_sRemoteClass); @@ -49,7 +48,6 @@ class UISearchFormForeignKeys $oBlock = new DisplayBlock($oFilter, 'search', false); $sHtml .= $oBlock->GetDisplay($oPage, "SearchFormToAdd_{$this->m_iInputId}", array( - 'open' => $bOpen, 'menu' => false, 'result_list_outer_selector' => "SearchResultsToAdd_{$this->m_iInputId}", 'table_id' => "add_{$this->m_iInputId}", diff --git a/core/config.class.inc.php b/core/config.class.inc.php index 9d2c4eeb6..e079e9cb6 100644 --- a/core/config.class.inc.php +++ b/core/config.class.inc.php @@ -1053,14 +1053,6 @@ class Config 'source_of_value' => '', 'show_in_conf_sample' => false, ), - 'legacy_search_drawer_open' => array( - 'type' => 'bool', - 'description' => 'Whether or not to display the "search drawer" open by default as in previous versions of iTop.', - 'default' => false, - 'value' => '', - 'source_of_value' => '', - 'show_in_conf_sample' => false, - ), 'obsolescence.show_obsolete_data' => array( 'type' => 'bool', 'description' => 'Default value for the user preference "show obsolete data"', diff --git a/pages/UI.php b/pages/UI.php index db6ff4a52..e0a62977d 100644 --- a/pages/UI.php +++ b/pages/UI.php @@ -130,7 +130,10 @@ function DisplayDetails($oP, $sClass, $oObj, $id) $sClassLabel = MetaModel::GetName($sClass); $oSearch = new DBObjectSearch($sClass); $oBlock = new DisplayBlock($oSearch, 'search', false); - $oBlock->Display($oP, 0, array('table_id' => 'search-widget-results-outer')); + $oBlock->Display($oP, 0, array( + 'table_id' => 'search-widget-results-outer', + 'open' => false, + )); // The object could be listed, check if it is actually allowed to view it $oSet = CMDBObjectSet::FromObject($oObj); @@ -194,7 +197,7 @@ function SetObjectBreadCrumbEntry(DBObject $oObj, WebPage $oPage) * @param $bDoSearch bool True to display the search results below the search form * @param $bSearchFormOpen bool True to display the search form fully expanded (only if $bSearchForm of course) */ -function DisplaySearchSet($oP, $oFilter, $bSearchForm = true, $sBaseClass = '', $sFormat = '', $bDoSearch = true, $bSearchFormOpen = false) +function DisplaySearchSet($oP, $oFilter, $bSearchForm = true, $sBaseClass = '', $sFormat = '', $bDoSearch = true, $bSearchFormOpen = true) { if ($bSearchForm) { diff --git a/setup/compiler.class.inc.php b/setup/compiler.class.inc.php index 53e54986c..db7886958 100644 --- a/setup/compiler.class.inc.php +++ b/setup/compiler.class.inc.php @@ -1897,7 +1897,7 @@ EOF; break; default: - $sSearchFormOpen = 'null'; // Actual open/close status depend on the config + $sSearchFormOpen = 'true'; } $sNewMenu = "new OQLMenuNode('$sMenuId', $sOQL, $sParentSpec, $fRank, $bSearch {$sOptionalEnableParams}, $sSearchFormOpen);"; break; diff --git a/sources/application/search/searchform.class.inc.php b/sources/application/search/searchform.class.inc.php index c1b301dd2..9587725d6 100644 --- a/sources/application/search/searchform.class.inc.php +++ b/sources/application/search/searchform.class.inc.php @@ -84,6 +84,15 @@ class SearchForm { $sRootClass = $sClassName; } + //should the search be opend on load? + if (isset($aExtraParams['open'])) + { + $bOpen = $aExtraParams['open']; + } + else + { + $bOpen = true; + } $sJson = utils::ReadParam('json', '', false, 'raw_data'); if (!empty($sJson)) @@ -124,7 +133,7 @@ class SearchForm $sClassesCombo = MetaModel::GetName($sClassName); } $sAction = (isset($aExtraParams['action'])) ? $aExtraParams['action'] : utils::GetAbsoluteUrlAppRoot().'pages/UI.php'; - $sStyle = (isset($aExtraParams['open']) && ($aExtraParams['open'] == 'true')) ? '' : 'closed'; + $sStyle = ($bOpen == 'true') ? '' : 'closed'; $sHtml .= "
\n"; // Don't use $_SERVER['SCRIPT_NAME'] since the form may be called asynchronously (from ajax.php) $sHtml .= "

" . Dict::Format('UI:SearchFor_Class_Objects', $sClassesCombo) . "

\n"; $sHtml .= "
\n"; @@ -173,11 +182,8 @@ class SearchForm { $aListParams['table_inner_id'] = "table_inner_id_{$sSearchFormId}"; } - $bOpen = false; - if (isset($aExtraParams['open'])) - { - $bOpen = $aExtraParams['open']; - } + + $sDebug = utils::ReadParam('debug', 'false', false, 'parameter'); if ($sDebug == 'true')