From 49f7b8dc894e8c48f91ebcfc3a0e26725032d695 Mon Sep 17 00:00:00 2001
From: Jean-Philippe ARGAUD <jean-philippe.argaud@edf.fr>
Date: Mon, 30 May 2022 15:43:07 +0200
Subject: [PATCH] Minor documentation and code review corrections (30)

Follow change in SALOME environment variables
---
 doc/en/advanced.rst                    | 8 +++++---
 doc/fr/advanced.rst                    | 8 +++++---
 src/daComposant/daCore/Interfaces.py   | 2 +-
 src/daComposant/daCore/PlatformInfo.py | 6 +++---
 4 files changed, 14 insertions(+), 10 deletions(-)

diff --git a/doc/en/advanced.rst b/doc/en/advanced.rst
index 695c07d..8be42b1 100644
--- a/doc/en/advanced.rst
+++ b/doc/en/advanced.rst
@@ -576,9 +576,11 @@ Switching from a version of ADAO to a newer one
 .. index:: single: Version
 
 The ADAO module and its ".comm" case files are identified by versions, with
-"Major", "Minor" and "Revision" characteristics. A particular version is
-numbered as "Major.Minor.Revision", with strong link with the numbering of the
-SALOME platform.
+"Major", "Minor", "Revision" and optionally "Installation" characteristics. A
+particular version is numbered as "Major.Minor.Revision", with strong link with
+the numbering of the SALOME platform. The optional indication of a fourth
+number indicates a difference in the installation method, not in the content of
+the version.
 
 Each version "Major.Minor.Revision" of the ADAO module can read ADAO case files
 of the previous minor version "Major.Minor-1.*". In general, it can also read
diff --git a/doc/fr/advanced.rst b/doc/fr/advanced.rst
index 896fc5d..d613f97 100644
--- a/doc/fr/advanced.rst
+++ b/doc/fr/advanced.rst
@@ -628,9 +628,11 @@ Passer d'une version d'ADAO à une nouvelle
 .. index:: single: Version
 
 Le module ADAO et ses fichiers de cas ".comm" sont identifiés par des versions,
-avec des caractéristiques "Major", "Minor" et "Revision". Une version
-particulière est numérotée "Major.Minor.Revision", avec un lien fort avec la
-numérotation de la plateforme SALOME.
+avec des caractéristiques "Major", "Minor", "Revision" et optionnellement
+"Installation". Une version particulière est numérotée "Major.Minor.Revision",
+avec un lien fort avec la numérotation de la plateforme SALOME. L'indication
+optionnelle d'un quatrième numéro désigne une différence dans le mode
+d'installation, pas dans le contenu de la version.
 
 Chaque version "Major.Minor.Revision" du module ADAO peut lire les fichiers de
 cas ADAO de la précédente version mineure "Major.Minor-1.*". En général, elle
diff --git a/src/daComposant/daCore/Interfaces.py b/src/daComposant/daCore/Interfaces.py
index 175b9b6..9a7d54e 100644
--- a/src/daComposant/daCore/Interfaces.py
+++ b/src/daComposant/daCore/Interfaces.py
@@ -609,7 +609,7 @@ class _YACSViewer(GenericCaseViewer):
         if not PlatformInfo.has_salome or \
             not PlatformInfo.has_adao:
             raise ImportError(
-                "Unable to get SALOME or ADAO environnement for YACS conversion.\n"+\
+                "Unable to get SALOME (%s) or ADAO (%s) environnement for YACS conversion.\n"%(PlatformInfo.has_salome,PlatformInfo.has_adao)+\
                 "Please load the right SALOME environnement before trying to use it.")
         else:
             from daYacsSchemaCreator.run import create_schema_from_content
diff --git a/src/daComposant/daCore/PlatformInfo.py b/src/daComposant/daCore/PlatformInfo.py
index 1bb6937..d3f9ea5 100644
--- a/src/daComposant/daCore/PlatformInfo.py
+++ b/src/daComposant/daCore/PlatformInfo.py
@@ -259,9 +259,9 @@ try:
 except ImportError:
     has_sdf = False
 
-has_salome = bool( "ROOT_SALOME"   in os.environ )
-has_yacs   = bool( "YACS_ROOT_DIR" in os.environ )
-has_adao   = bool( "ADAO_ROOT_DIR" in os.environ )
+has_salome = bool( "SALOME_ROOT_DIR" in os.environ )
+has_yacs   = bool(   "YACS_ROOT_DIR" in os.environ )
+has_adao   = bool(   "ADAO_ROOT_DIR" in os.environ )
 has_eficas = bool( "EFICAS_ROOT_DIR" in os.environ )
 
 # ==============================================================================
-- 
2.39.2