From 5dc283b59d822318df8bca5a9b7e15906ef8e527 Mon Sep 17 00:00:00 2001
From: Tom Darneix <tomdar87@outlook.com>
Date: Thu, 9 Jun 2022 14:26:23 +0200
Subject: [PATCH] merge dev-20.10.x (#11193)

* merge dev-20.10.x

* Attempt to fix the build
---
 Jenkinsfile                                   |  1 -
 www/api/class/centreon_ceip.class.php         |  2 +-
 www/front_src/src/Footer/Links.tsx            |  7 +--
 www/front_src/src/Resources/Filter/index.tsx  |  2 +-
 .../Resources/Listing/columns/State/index.tsx |  2 +-
 .../Resources/Listing/columns/Url/index.tsx   |  2 +-
 .../src/Resources/Listing/useListing.ts       |  9 ----
 www/front_src/src/components/Hook/index.tsx   | 13 ++---
 .../src/components/ReactRouter/index.tsx      | 11 ++--
 .../configObject/contact/formContact.php      | 53 ++++++++++---------
 10 files changed, 43 insertions(+), 59 deletions(-)

diff --git a/Jenkinsfile b/Jenkinsfile
index 3e7fcfcf227..e2606058add 100644
--- a/Jenkinsfile
+++ b/Jenkinsfile
@@ -292,5 +292,4 @@ try {
             "*COMMIT*: <https://github.com/centreon/centreon/commit/${source.COMMIT}|here> by ${source.COMMITTER}\n" +
             "*INFO*: ${e}"
   }
-
 }
diff --git a/www/api/class/centreon_ceip.class.php b/www/api/class/centreon_ceip.class.php
index ddafa758077..46e1a59e7c5 100644
--- a/www/api/class/centreon_ceip.class.php
+++ b/www/api/class/centreon_ceip.class.php
@@ -229,7 +229,7 @@ private function getLicenseInformation(): array
                 }
             }
         } catch (\Exception $exception) {
-            $this->logger->error($exception->getMessage, ['context' => $exception]);
+            $this->logger->error($exception->getMessage(), ['context' => $exception]);
         }
 
         return [
diff --git a/www/front_src/src/Footer/Links.tsx b/www/front_src/src/Footer/Links.tsx
index 688f61cd1ae..1ef818351f1 100644
--- a/www/front_src/src/Footer/Links.tsx
+++ b/www/front_src/src/Footer/Links.tsx
@@ -1,6 +1,6 @@
 import * as React from 'react';
 
-import { dec, equals, length, not, pipe } from 'ramda';
+import { equals, not, dec } from 'ramda';
 
 import { Link, makeStyles, Typography } from '@material-ui/core';
 
@@ -43,10 +43,7 @@ const links: Array<FooterLink> = [
   },
 ];
 
-const numbersOfLinks = pipe<Array<FooterLink>, number, number>(
-  length,
-  dec,
-)(links);
+const numbersOfLinks = dec(links.length);
 
 const Links = (): JSX.Element => {
   const classes = useStyles();
diff --git a/www/front_src/src/Resources/Filter/index.tsx b/www/front_src/src/Resources/Filter/index.tsx
index e37af72ab3a..7127b370fa9 100644
--- a/www/front_src/src/Resources/Filter/index.tsx
+++ b/www/front_src/src/Resources/Filter/index.tsx
@@ -280,7 +280,7 @@ const Filter = (): JSX.Element => {
           />
           <Button
             color="primary"
-            data-testid={labelClear}
+            data-testid={labelClearAll}
             onClick={clearAllFilters}
           >
             {t(labelClearAll)}
diff --git a/www/front_src/src/Resources/Listing/columns/State/index.tsx b/www/front_src/src/Resources/Listing/columns/State/index.tsx
index d4c2fa7aadf..403af20cb1e 100644
--- a/www/front_src/src/Resources/Listing/columns/State/index.tsx
+++ b/www/front_src/src/Resources/Listing/columns/State/index.tsx
@@ -17,7 +17,7 @@ import DowntimeDetailsTable from './DetailsTable/Downtime';
 
 interface StateChipProps {
   Chip: () => JSX.Element;
-  DetailsTable: React.SFC<{ endpoint: string }>;
+  DetailsTable: React.FC<{ endpoint: string }>;
   endpoint: string;
   label: string;
 }
diff --git a/www/front_src/src/Resources/Listing/columns/Url/index.tsx b/www/front_src/src/Resources/Listing/columns/Url/index.tsx
index 0c07f30b0a2..381d66d0424 100644
--- a/www/front_src/src/Resources/Listing/columns/Url/index.tsx
+++ b/www/front_src/src/Resources/Listing/columns/Url/index.tsx
@@ -60,7 +60,7 @@ const UrlColumn = ({
         <IconButton
           ariaLabel={title}
           data-testid={title || endpoint}
-          title={title || endpoint}
+          title={(title || endpoint) as string}
           onClick={(): null => {
             return null;
           }}
diff --git a/www/front_src/src/Resources/Listing/useListing.ts b/www/front_src/src/Resources/Listing/useListing.ts
index b097b13861b..778dae82d8d 100644
--- a/www/front_src/src/Resources/Listing/useListing.ts
+++ b/www/front_src/src/Resources/Listing/useListing.ts
@@ -1,14 +1,10 @@
 import * as React from 'react';
 
-import { ifElse, pathEq, always, pathOr } from 'ramda';
-
 import { useRequest } from '@centreon/ui';
 
 import { ResourceListing } from '../models';
-import { labelSomethingWentWrong } from '../translatedLabels';
 
 import { defaultSortOrder, defaultSortField } from './columns';
-import ApiNotFoundMessage from './ApiNotFoundMessage';
 import { listResources } from './api';
 
 type SortOrder = 'asc' | 'desc';
@@ -41,11 +37,6 @@ const useListing = (): ListingState => {
   const [enabledAutorefresh, setEnabledAutorefresh] = React.useState(true);
 
   const { sendRequest, sending } = useRequest<ResourceListing>({
-    getErrorMessage: ifElse(
-      pathEq(['response', 'status'], 404),
-      always(ApiNotFoundMessage),
-      pathOr(labelSomethingWentWrong, ['response', 'data', 'message']),
-    ),
     request: listResources,
   });
 
diff --git a/www/front_src/src/components/Hook/index.tsx b/www/front_src/src/components/Hook/index.tsx
index 1a717fe720a..d5f913a545e 100644
--- a/www/front_src/src/components/Hook/index.tsx
+++ b/www/front_src/src/components/Hook/index.tsx
@@ -1,24 +1,19 @@
 import React, { Suspense } from 'react';
 
 import { connect } from 'react-redux';
-import { withRouter } from 'react-router-dom';
+import { useHistory } from 'react-router';
 import isEqual from 'lodash/isEqual';
 
 import { dynamicImport } from '../../helpers/dynamicImport';
 import centreonAxios from '../../axios';
 
 interface Props {
-  history;
   hooks;
   path;
 }
 
-const LoadableHooks = ({
-  history,
-  hooks,
-  path,
-  ...rest
-}: Props): JSX.Element => {
+const LoadableHooks = ({ hooks, path, ...rest }: Props): JSX.Element => {
+  const history = useHistory();
   const basename = history.createHref({
     hash: '',
     pathname: '/',
@@ -58,4 +53,4 @@ const mapStateToProps = ({ externalComponents }): Record<string, unknown> => ({
   hooks: externalComponents.hooks,
 });
 
-export default connect(mapStateToProps)(withRouter(Hook));
+export default connect(mapStateToProps)(Hook);
diff --git a/www/front_src/src/components/ReactRouter/index.tsx b/www/front_src/src/components/ReactRouter/index.tsx
index 6347ce40944..b4bf6feaa1a 100644
--- a/www/front_src/src/components/ReactRouter/index.tsx
+++ b/www/front_src/src/components/ReactRouter/index.tsx
@@ -1,7 +1,7 @@
 import React, { Suspense } from 'react';
 
 import { connect } from 'react-redux';
-import { Switch, Route, withRouter } from 'react-router-dom';
+import { Switch, Route, useHistory } from 'react-router';
 import isEqual from 'lodash/isEqual';
 
 import { styled } from '@material-ui/core';
@@ -59,12 +59,13 @@ const getExternalPageRoutes = ({
 interface Props {
   allowedPages: Array<string>;
   externalPagesFetched: boolean;
-  history;
   pages: Record<string, unknown>;
 }
 
 const ReactRouter = React.memo<Props>(
-  ({ allowedPages, history, pages, externalPagesFetched }: Props) => {
+  ({ allowedPages, pages, externalPagesFetched }: Props) => {
+    const history = useHistory();
+
     if (!externalPagesFetched) {
       // eslint-disable-next-line react/jsx-no-undef
       return <PageContainer />;
@@ -104,10 +105,10 @@ const ReactRouter = React.memo<Props>(
     isEqual(previousProps.externalPagesFetched, nextProps.externalPagesFetched),
 );
 
-const mapStateToProps = (state): Record<string, unknown> => ({
+const mapStateToProps = (state): Props => ({
   allowedPages: allowedPagesSelector(state),
   externalPagesFetched: state.externalComponents.fetched,
   pages: state.externalComponents.pages,
 });
 
-export default connect(mapStateToProps)(withRouter(ReactRouter));
+export default connect(mapStateToProps)(ReactRouter);
diff --git a/www/include/configuration/configObject/contact/formContact.php b/www/include/configuration/configObject/contact/formContact.php
index eaf4f677f19..9b130d13125 100644
--- a/www/include/configuration/configObject/contact/formContact.php
+++ b/www/include/configuration/configObject/contact/formContact.php
@@ -1,4 +1,5 @@
 <?php
+
 /*
  * Copyright 2005-2019 Centreon
  * Centreon is developed by : Julien Mathis and Romain Le Merlus under
@@ -84,7 +85,7 @@
 $dbResult->closeCursor();
 
 $cct = array();
-if (($o == "c" || $o == "w") && $contactId) {
+if (($o == MODIFY_CONTACT || $o == WATCH_CONTACT) && $contactId) {
     /**
      * Init Tables informations
      */
@@ -146,7 +147,7 @@
  */
 $langs = array();
 $langs = getLangs();
-if ($o == "mc") {
+if ($o == MASSIVE_CHANGE) {
     array_unshift($langs, null);
 }
 
@@ -236,7 +237,7 @@
  * @var $moduleFormManager \Centreon\Domain\Service\ModuleFormManager
  */
 
-if ($o == "a") {
+if ($o == ADD_CONTACT) {
     $form->addElement('header', 'title', _("Add a User"));
 
     $eventDispatcher->notify(
@@ -248,7 +249,7 @@
             'contact_id' => $contactId
         ]
     );
-} elseif ($o == "c") {
+} elseif ($o == MODIFY_CONTACT) {
     $form->addElement('header', 'title', _("Modify a User"));
 
     $eventDispatcher->notify(
@@ -260,7 +261,7 @@
             'contact_id' => $contactId
         ]
     );
-} elseif ($o == "w") {
+} elseif ($o == WATCH_CONTACT) {
     $form->addElement('header', 'title', _("View a User"));
 
     $eventDispatcher->notify(
@@ -272,7 +273,7 @@
             'contact_id' => $contactId
         ]
     );
-} elseif ($o == "mc") {
+} elseif ($o == MASSIVE_CHANGE) {
     $form->addElement('header', 'title', _("Massive Change"));
 
     $eventDispatcher->notify(
@@ -301,7 +302,7 @@
  * Don't change contact name and alias in massif change
  * Don't change contact name, alias or autologin key in massive change
  */
-if ($o != "mc") {
+if ($o != MASSIVE_CHANGE) {
     $form->addElement('text', 'contact_name', _("Full Name"), $attrsTextDescr);
     $form->addElement('text', 'contact_alias', _("Alias / Login"), $attrsText);
     $form->addElement('text', 'contact_autologin_key', _("Autologin Key"), array("size" => "90", "id" => "aKey"));
@@ -328,7 +329,7 @@
  * Contact Groups Field
  */
 $form->addElement('header', 'groupLinks', _("Group Relations"));
-if ($o == "mc") {
+if ($o == MASSIVE_CHANGE) {
     $mc_mod_cg = array();
     $mc_mod_cg[] = $form->createElement('radio', 'mc_mod_cg', null, _("Incremental"), '0');
     $mc_mod_cg[] = $form->createElement('radio', 'mc_mod_cg', null, _("Replacement"), '1');
@@ -396,7 +397,7 @@
 /**
  * ACL configurations
  */
-if ($o == "mc") {
+if ($o == MASSIVE_CHANGE) {
     $mc_mod_cg = array();
     $mc_mod_cg[] = $form->createElement('radio', 'mc_mod_acl', null, _("Incremental"), '0');
     $mc_mod_cg[] = $form->createElement('radio', 'mc_mod_acl', null, _("Replacement"), '1');
@@ -432,7 +433,7 @@
 );
 $form->addElement('select2', 'contact_location', _("Timezone / Location"), array(), $attrTimezones);
 
-if ($o != "mc") {
+if ($o != MASSIVE_CHANGE) {
     $auth_type = array();
 } else {
     $auth_type = array(null => null);
@@ -446,7 +447,7 @@
         $dnElement->freeze();
     }
 }
-if ($o != "mc") {
+if ($o != MASSIVE_CHANGE) {
     $form->setDefaults(array(
         'contact_oreon' => '1',
         'contact_admin' => '0',
@@ -466,7 +467,7 @@
 $tab[] = $form->createElement('radio', 'contact_enable_notifications', null, _("No"), '0');
 $tab[] = $form->createElement('radio', 'contact_enable_notifications', null, _("Default"), '2');
 $form->addGroup($tab, 'contact_enable_notifications', _("Enable Notifications"), '&nbsp;');
-if ($o != "mc") {
+if ($o != MASSIVE_CHANGE) {
     $form->setDefaults(array('contact_enable_notifications' => '2'));
 }
 
@@ -529,7 +530,7 @@
 
 unset($hostNotifOpt);
 
-if ($o == "mc") {
+if ($o == MASSIVE_CHANGE) {
     $mc_mod_hcmds = array();
     $mc_mod_hcmds[] = $form->createElement('radio', 'mc_mod_hcmds', null, _("Incremental"), '0');
     $mc_mod_hcmds[] = $form->createElement('radio', 'mc_mod_hcmds', null, _("Replacement"), '1');
@@ -615,7 +616,7 @@
 );
 $form->addElement('select2', 'timeperiod_tp_id2', _("Service Notification Period"), array(), $attrTimeperiod2);
 
-if ($o == "mc") {
+if ($o == MASSIVE_CHANGE) {
     $mc_mod_svcmds = array();
     $mc_mod_svcmds[] = $form->createElement('radio', 'mc_mod_svcmds', null, _("Incremental"), '0');
     $mc_mod_svcmds[] = $form->createElement('radio', 'mc_mod_svcmds', null, _("Replacement"), '1');
@@ -645,7 +646,7 @@
 $cctActivation[] = $form->createElement('radio', 'contact_activate', null, _("Disabled"), '0');
 $form->addGroup($cctActivation, 'contact_activate', _("Status"), '&nbsp;');
 $form->setDefaults(array('contact_activate' => '1'));
-if ($o == "c" && $centreon->user->get_id() == $cct["contact_id"]) {
+if ($o == MODIFY_CONTACT && $centreon->user->get_id() == $cct["contact_id"]) {
     $form->freeze('contact_activate');
 }
 
@@ -683,7 +684,7 @@ function myReplace()
 $form->applyFilter('__ALL__', 'myTrim');
 $form->applyFilter('contact_name', 'myReplace');
 $from_list_menu = false;
-if ($o != "mc") {
+if ($o != MASSIVE_CHANGE) {
     $ret = $form->getSubmitValues();
     $form->addRule('contact_name', _("Compulsory Name"), 'required');
     $form->addRule('contact_alias', _("Compulsory Alias"), 'required');
@@ -730,7 +731,7 @@ function myReplace()
         _("You have to keep at least one contact to access to Centreon"),
         'keepOneContactAtLeast'
     );
-} elseif ($o == "mc") {
+} elseif ($o == MASSIVE_CHANGE) {
     if ($form->getSubmitValue("submitMC")) {
         $from_list_menu = false;
     } else {
@@ -753,7 +754,7 @@ function myReplace()
     $helptext .= '<span style="display:none" id="help:' . $key . '">' . $text . '</span>' . "\n";
 }
 $tpl->assign("helptext", $helptext);
-if ($o == "w") {
+if ($o == WATCH_CONTACT) {
     # Just watch a contact information
     if ($centreon->user->access->page($p) != 2) {
         $form->addElement(
@@ -765,16 +766,16 @@ function myReplace()
     }
     $form->setDefaults($cct);
     $form->freeze();
-} elseif ($o == "c") {
+} elseif ($o == MODIFY_CONTACT) {
     # Modify a contact information
     $subC = $form->addElement('submit', 'submitC', _("Save"), array("class" => "btc bt_success"));
     $res = $form->addElement('reset', 'reset', _("Reset"), array("class" => "btc bt_default"));
     $form->setDefaults($cct);
-} elseif ($o == "a") {
+} elseif ($o == ADD_CONTACT) {
     # Add a contact information
     $subA = $form->addElement('submit', 'submitA', _("Save"), array("class" => "btc bt_success"));
     $res = $form->addElement('reset', 'reset', _("Reset"), array("class" => "btc bt_default"));
-} elseif ($o == "mc") {
+} elseif ($o == MASSIVE_CHANGE) {
     # Massive Change
     $subMC = $form->addElement('submit', 'submitMC', _("Save"), array("class" => "btc bt_success"));
     $res = $form->addElement('reset', 'reset', _("Reset"), array("class" => "btc bt_default"));
@@ -791,7 +792,7 @@ function myReplace()
 
 if ($form->validate() && $from_list_menu == false) {
     $cctObj = $form->getElement('contact_id');
-    if (!$centreon->user->admin && $contact_id) {
+    if (!$centreon->user->admin && $contactId) {
         $form->removeElement('contact_admin');
         $form->removeElement('reach_api');
         $form->removeElement('reach_api_rt');
@@ -821,16 +822,16 @@ function myReplace()
         );
     } elseif ($form->getSubmitValue("submitMC")) {
         $select = explode(",", $select);
-        foreach ($select as $key => $contactId) {
-            if ($contactId) {
-                updateContactInDB($contactId, true);
+        foreach ($select as $key => $selectedContactId) {
+            if ($selectedContactId) {
+                updateContactInDB($selectedContactId, true);
 
                 $eventDispatcher->notify(
                     'contact.form',
                     EventDispatcher::EVENT_UPDATE,
                     [
                         'form' => $form,
-                        'contact_id' => $contactId
+                        'contact_id' => $selectedContactId
                     ]
                 );
             }